lkml.org 
[lkml]   [2012]   [Aug]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 53/74] x86, lto, paravirt: Make paravirt thunks global
    On Sun, Aug 19, 2012 at 01:27:00AM -0700, Jeremy Fitzhardinge wrote:
    > On 08/18/2012 07:56 PM, Andi Kleen wrote:
    > > From: Andi Kleen <ak@linux.intel.com>
    > >
    > > The paravirt thunks use a hack of using a static reference to a static
    > > function to reference that function from the top level statement.
    > >
    > > This assumes that gcc always generates static function names in a specific
    > > format, which is not necessarily true.
    > >
    > > Simply make these functions global and asmlinkage. This way the
    > > static __used variables are not needed and everything works.
    >
    > I'm not a huge fan of unstaticing all this stuff, but it doesn't
    > surprise me that the current code is brittle in the face of gcc changes.

    Hmm actually reading my own patch again it may be wrong. You need
    regparm(3) here right? asmlinkage forces it to (0). I'll change it to
    __visible. I think I did that earlier for all the 32bit code, but missed
    this one.

    -Andi


    \
     
     \ /
      Last update: 2012-08-19 18:01    [W:5.469 / U:0.020 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site