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

Re: [Xen-devel] [PATCH] correct mis-conversion set_bit() -> __cpumask_set_cpu() by 4aaca0e9cd



>>> On 23.02.15 at 13:01, <andrew.cooper3@xxxxxxxxxx> wrote:
> On 23/02/15 11:06, Jan Beulich wrote:
>> I have no idea how I came to use __cpumask_set_cpu() there, the
>> conversion should have been set_bit() -> __set_bit(). The wrong
>> construct results in problems on systems with relatively few CPUs.
>>
>> Reported-by: Sander Eikelenboom <linux@xxxxxxxxxxxxxx>
>> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
> 
> Insofar as this clearly corrects the identified regression,
> 
> Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> 
> However,  I am still not convinced that the resulting code is actually
> correct.
> 
> batch_mask is a cpumask_t and used properly as a cpumask in
> cpumask_raise_softirq().  It is wrong to be putting softirq indices into
> it here.

Ah, now I see (somehow I didn't pay close enough attention to
what you said in the morning) - the code was wrong even before
that change. In that case - yes, let's not fix it the wrong way.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

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