Re: [PATCH 05/10] paravirt: add new PARAVIRT_FULL config item
From: Boris Ostrovsky <boris.ostrovsky@oracle.com>
Date: 2017-05-24 15:41:20
Also in:
lkml
From: Boris Ostrovsky <boris.ostrovsky@oracle.com>
Date: 2017-05-24 15:41:20
Also in:
lkml
On 05/19/2017 11:47 AM, Juergen Gross wrote:
Add a new config item PARAVIRT_FULL. It will be used to guard the pv_*_ops functions used by fully paravirtualized guests (Xen pv-guests and lguest) only. Kernels not meant to support those guest types will be able to use many operations without paravirt abstraction while still supporting all the other paravirt features. For now just add the new Kconfig option and select it for XEN_PV and LGUEST_GUEST. Add paravirt_full.c, paravirt_full.h and paravirt_types_full.h which will contain the necessary implementation parts of the pv guest specific paravirt functions.
Is it not possible to just 'ifdef CONFIG_PARAVIT_FULL' the (ir)relevant parts of paravirt.[ch] and paravirt_types.c? Separating structures and files into pv and pvfull seems somewhat arbitrary (.flush_tlb_others in patch 8 being a good example of one type of guest deciding to use something that normally would be considered part of a pvfull-type structure). -boris