[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



Hi Jan,


On 07/06/2017 05:22 PM, Jan Beulich wrote:
>>>> 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?
>

Yeap, will do.

Cheers,
~Sergej

_______________________________________________
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®.