Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1359470

Re: [v3,1/8] powerpc: Create a helper for getting the kernel toc value

From Balbir Singh <bsingharora@gmail.com>
Newsgroups linux.kernel
Subject Re: [v3,1/8] powerpc: Create a helper for getting the kernel toc value
Date 2016-03-17 01:00 +0100
Message-ID <rdtw7-7MK-25@gated-at.bofh.it> (permalink)
References <rcwZ3-1Qb-3@gated-at.bofh.it> <rcLY6-3nY-9@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw



On 15/03/16 12:27, Jiri Kosina wrote:
> On Mon, 14 Mar 2016, Michael Ellerman wrote:
>
>>> Move the logic to work out the kernel toc pointer into a header. This is
>>> a good cleanup, and also means we can use it elsewhere in future.
>>>
>>> Reviewed-by: Kamalesh Babulal <kamalesh@linux.vnet.ibm.com>
>>> Reviewed-by: Torsten Duwe <duwe@suse.de>
>>> Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
>>> Tested-by: Kamalesh Babulal <kamalesh@linux.vnet.ibm.com>
>> Series applied to powerpc next.
>>
>> https://git.kernel.org/powerpc/c/a5cab83cd3d2d75d3893276cb5
> Thanks Michael; this is an excellent basis for ppc live patching, but FYI 
> I am not merging that one to my tree just yet.
>
> The solution (*) for functions with non-trivial argument list is not there 
> yet, and it's my requirement for this to be taken care of in a way that's 
> not prone to easily-done human errors on the patch-producer side.
>
> (*) both "making it work" or "making it so broken that it's guaranteed 
>     that noone would ever produce a patch that brings the kernel down" is 
>     okay, but I really don't feel that just documenting the limitation is
>     sufficient and safe in this case; kudos to Torsten here for
>     idenfitfying the problem before it actually became The Problem
To be honest I think my v6 works well, but I don't have complete confidence due to the lack of proper testing. livepatch samples plus some others I wrote and I one Petr wrote all work (calling patched from within patched), but we need more confidence with good tests or an alternative approach that is easier to review and be satisfied with
>
> Thanks,
>

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

Re: [v3,1/8] powerpc: Create a helper for getting the kernel toc value Michael Ellerman <mpe@ellerman.id.au> - 2016-03-14 10:30 +0100
  Re: [v3,1/8] powerpc: Create a helper for getting the kernel toc  value Jiri Kosina <jikos@kernel.org> - 2016-03-15 02:30 +0100
    Re: [v3,1/8] powerpc: Create a helper for getting the kernel toc  value Michael Ellerman <mpe@ellerman.id.au> - 2016-03-16 11:30 +0100
      Re: [v3,1/8] powerpc: Create a helper for getting the kernel toc  value Torsten Duwe <duwe@lst.de> - 2016-03-16 16:00 +0100
        Re: [v3,1/8] powerpc: Create a helper for getting the kernel toc  value Balbir Singh <bsingharora@gmail.com> - 2016-03-17 01:00 +0100
    Re: [v3,1/8] powerpc: Create a helper for getting the kernel toc  value Balbir Singh <bsingharora@gmail.com> - 2016-03-17 01:00 +0100
      Re: [v3,1/8] powerpc: Create a helper for getting the kernel toc  value Torsten Duwe <duwe@lst.de> - 2016-03-17 17:00 +0100

csiph-web