Re: [PATCH RFC V2 5/5] kvm guest : pv-ticketlocks support for linuxguests running on KVM hypervisor

[Raghavendra K T]

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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/