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

Re: [Xen-devel] [PATCH v6 08/14] arm/mem_access: Introduce GENMASK_ULL bit operation



>>> On 06.07.17 at 16:38, <proskurin@xxxxxxxxxxxxx> wrote:
> On 07/06/2017 02:18 PM, Jan Beulich wrote:
>>>>> On 06.07.17 at 13:50, <proskurin@xxxxxxxxxxxxx> wrote:
>>> @@ -128,7 +131,7 @@ static inline int generic_fls64(__u64 x)
>>>  static __inline__ int get_bitmask_order(unsigned int count)
>>>  {
>>>      int order;
>>> -    
>>> +
>>>      order = fls(count);
>>>      return order;   /* We could be slightly more clever with -1 here... */
>>>  }
>> If you really want to do cleanup here, please shrink the function
>> body to a single return statement. But then again I'm unconvinced
>> the function is actually correct (which could easily be the case for
>> an unused one), in particular for power-of-2 counts. Nor can I see
>> how this would be useful with anything more narrow than size_t
>> or unsigned long as parameter type.
> 
> Right. That whitespace elimination was actually unintended. However, if
> you wish, I could do the cleanup in a separate patch.
> 
> Concerning the correctness of this function: I am not sure whether this
> would be the right patch series to address that.

In that case simply leave out the cleanup part?

Jan


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

 


Rackspace

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