Re: [PATCH v3] x86/vsmp_64.c: Remove dependency on pv_irq_ops

2018-11-07 Thread Eial Czerwacki
Greetings Ingo, On 11/05/2018 07:59 PM, Ingo Molnar wrote: > > * Eial Czerwacki wrote: > >> +#if defined(CONFIG_PCI) > > This is shorter: > >#ifdef CONFIG_PCI > > Thanks, > > Ingo > you are absolutely right, looks like Thomas have handled it already. Eial.

Re: [PATCH v3] x86/vsmp_64.c: Remove dependency on pv_irq_ops

2018-11-07 Thread Eial Czerwacki
Greetings Ingo, On 11/05/2018 07:59 PM, Ingo Molnar wrote: > > * Eial Czerwacki wrote: > >> +#if defined(CONFIG_PCI) > > This is shorter: > >#ifdef CONFIG_PCI > > Thanks, > > Ingo > you are absolutely right, looks like Thomas have handled it already. Eial.

Re: [PATCH v3] x86/vsmp_64.c: Remove dependency on pv_irq_ops

2018-11-05 Thread Ingo Molnar
* Eial Czerwacki wrote: > +#if defined(CONFIG_PCI) This is shorter: #ifdef CONFIG_PCI Thanks, Ingo

Re: [PATCH v3] x86/vsmp_64.c: Remove dependency on pv_irq_ops

2018-11-05 Thread Ingo Molnar
* Eial Czerwacki wrote: > +#if defined(CONFIG_PCI) This is shorter: #ifdef CONFIG_PCI Thanks, Ingo

[PATCH v3] x86/vsmp_64.c: Remove dependency on pv_irq_ops

2018-11-05 Thread Eial Czerwacki
vSMP dependency on pv_irq_ops has been removed some years ago, but the code still deals with pv_irq_ops. In short, "cap & ctl & (1 << 4)" is always returning 0, so all PARAVIRT/PARAVIRT_XXL code related to that can be removed. However, the rest of the code depends on CONFIG_PCI, so fix it

[PATCH v3] x86/vsmp_64.c: Remove dependency on pv_irq_ops

2018-11-05 Thread Eial Czerwacki
vSMP dependency on pv_irq_ops has been removed some years ago, but the code still deals with pv_irq_ops. In short, "cap & ctl & (1 << 4)" is always returning 0, so all PARAVIRT/PARAVIRT_XXL code related to that can be removed. However, the rest of the code depends on CONFIG_PCI, so fix it