Messages in this thread |  | | | Date | Thu, 27 Oct 2011 00:55:55 +0530 | | From | Raghavendra K T <> | | Subject | Re: [PATCH RFC V2 5/5] kvm guest : pv-ticketlocks support for linux guests running on KVM hypervisor |
| |
On 10/26/2011 12:05 AM, Jeremy Fitzhardinge wrote: > On 10/23/2011 12:07 PM, Raghavendra K T wrote: >> >> +#ifdef CONFIG_PARAVIRT_SPINLOCKS >> + >> +#ifdef CONFIG_KVM_DEBUG_FS >> + >> +#include<linux/debugfs.h> >> + >> +#endif /* CONFIG_KVM_DEBUG_FS */ >> + >> +#endif /* CONFIG_PARAVIRT_SPINLOCKS */ > > This is a big mess. Is there any problem with including linux/debugfs.h > unconditionally? Or at least using "#if > defined(CONFIG_PARAVIRT_SPINLOCKS)&& defined(CONFIG_KVM_DEBUG_FS)"? >
I agree. I 'll include unconditionally unless somebody objects.
> J
|  |