Rusty Russell wrote: >> Also, since this is a pretty generic patch with no earlier dependencies, >> we can push it near the front of the patch queue. > > Actually, I think that it should be something like: > #ifdef CONFIG_PARAVIRT > #define __FIXADDR_TOP (paravirt_ops.fixaddr_top) > #endif > > Then paravirt_ops.init will set this value in the paravirt_ops struct > (just as it will set kernel_rpl). What is wrong with simply calling set_fixaddr_top() in paravirt_ops.init? I don't see the point of moving that into the paravirt_ops struct ... cheers, Gerd -- Gerd Hoffmann <kraxel at suse.de> http://www.suse.de/~kraxel/julika-dora.jpeg
