Re: [PATCH v10 08/19] qspinlock: Make a new qnode structure to support virtualization

[Waiman Long]

On 05/08/2014 03:04 PM, Peter Zijlstra wrote:
> On Wed, May 07, 2014 at 11:01:36AM -0400, Waiman Long wrote:
> >   /*
> > + * To have additional features for better virtualization support, it is
> > + * necessary to store additional data in the queue node structure. So
> > + * a new queue node structure will have to be defined and used here.
> > + */
> > +struct qnode {
> > +	struct mcs_spinlock mcs;
> > +};
> You can ditch this entire patch; its pointless, just add a new
> DEFINE_PER_CPU for the para-virt muck.

Yes, I can certainly merge it to the next one in the series. I break it 
out to make each individual patch smaller, more single-purpose and 
easier to review.

-Longman
--
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/