(2014/07/02 15:56), Michael Ellerman wrote: > On Wed, 2014-07-02 at 15:39 +0900, Masami Hiramatsu wrote: >> (2014/07/02 13:41), Michael Ellerman wrote: >>> On Tue, 2014-07-01 at 11:21 +0900, Masami Hiramatsu wrote: >>>> (2014/06/30 20:36), Michael Ellerman wrote: >>>>> On Mon, 2014-06-30 at 12:14 +0900, Masami Hiramatsu wrote: >>>>>> Ping? :) >>>>> >>>>> Yeah sorry. I started looking at this and got dragged into another mess. >>>>> >>>>> You seem to have duplicated the functionality of arch_deref_entry_point(), >>>>> which was also added for kprobes, and for the same reason - ie. because some >>>>> arches have strange function pointers. Is there some reason you can't use it? >>>> >>>> Ah, right! Hmm, it seems some more work to update it. but basically, we can do. >>>> BTW, is there any other users who need to access the actual function entry (for >>>> kallsyms case)? >>> >>> Not that I'm aware of. We have had function descriptors on 64-bit powerpc for >>> ever, so in theory by now we should have already found any cases where we need >>> that sort of wrapper. >> >> OK, then I'll update this patch to use arch_deref_entry_point(), and add additional >> patch which update to support PPC64 ABIv2. > > I've already done the latter: > > https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=2f0143c91d30823f6f6e7d94d7fa818f7ab18a18 Oh, thanks! In that case, my 1/2 patch should be dropped. -- Masami HIRAMATSU Software Platform Research Dept. Linux Technology Research Center Hitachi, Ltd., Yokohama Research Laboratory E-mail: masami.hiramatsu.pt@xxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-tip-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
|