Re: [PATCH] FS-Cache: print hexadecimal value for special cookies type

From: Andy Shevchenko
Date: Tue Apr 25 2017 - 18:04:22 EST


On Wed, Apr 26, 2017 at 12:37 AM, JÃrÃmy Lefaure
<jeremy.lefaure@xxxxxxxxxxxx> wrote:
> On Tue, 25 Apr 2017 00:47:41 +0300
> Andy Shevchenko <andy.shevchenko@xxxxxxxxx> wrote:

> Ok I get it. But changing the format can break the userland tool
> anyway, can't it ? For example if a userland tool expects the value 75,
> it won't read it properpy in BCD.

Maybe.

>> So, if it's okay in this, perhaps paragraph explaining side effects in
>> commit message would be good to have/
>>
> Maybe the best fix would be to increase the size of the _type buffer to
> 4 so the unsigned byte can fit in it. And change the documentation to
> decimal value. What do you think about this solution ?

Yes, it would work.

--
With Best Regards,
Andy Shevchenko