[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v2 19/23] x86: PIT emulation is common to both PV and HVM



On Tue, Aug 28, 2018 at 03:48:07PM +0100, Wei Liu wrote:
> On Tue, Aug 28, 2018 at 08:36:37AM -0600, Jan Beulich wrote:
> > >>> On 28.08.18 at 15:19, <wei.liu2@xxxxxxxxxx> wrote:
> > > On Tue, Aug 28, 2018 at 05:44:51AM -0600, Jan Beulich wrote:
> > >> >>> On 26.08.18 at 14:19, <wei.liu2@xxxxxxxxxx> wrote:
> > >> > Move the file to x86 common code and change its name to emul-i8254.c.
> > >> > 
> > >> > Put HVM only code under CONFIG_HVM or is_hvm_domain.
> > >> > 
> > >> > Signed-off-by: Wei Liu <wei.liu2@xxxxxxxxxx>
> > >> > ---
> > >> > v2: move the whole file.
> > >> > ---
> > >> >  xen/arch/x86/Makefile     |   1 +-
> > >> >  xen/arch/x86/emul-i8254.c | 609 
> > >> > +++++++++++++++++++++++++++++++++++++++-
> > >> >  xen/arch/x86/hvm/Makefile |   1 +-
> > >> >  xen/arch/x86/hvm/i8254.c  | 597 
> > >> > +--------------------------------------
> > >> >  4 files changed, 610 insertions(+), 598 deletions(-)
> > >> >  create mode 100644 xen/arch/x86/emul-i8254.c
> > >> >  delete mode 100644 xen/arch/x86/hvm/i8254.c
> > >> 
> > >> On the assumption that the changes to the file are only CONFIG_HVM
> > >> additions
> > > 
> > > No only that. There are a few is_hvm_domain calls as well.
> > > 
> > >> Acked-by: Jan Beulich <jbeulich@xxxxxxxx>
> > > 
> > > Does this still stand?
> > 
> > Yes, albeit looking at the full diff of the files I'm then a little
> > puzzled about is_hvm_domain() uses: Why does pit_load_count()
> > need an #ifdef added at all, with the is_hvm_vcpu() check
> > right before it?
> 
> Because if I don't do that we get "defined but not used" warnings.

This is nonsense. I misread the function name.

Wei.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.