Re: [GIT PULL] Block driver updates for 5.18-rc1

[Chaitanya Kulkarni <chaitanyak@nvidia.com>]

Hi Linus,

On 3/21/22 17:13, Linus Torvalds wrote:
> On Fri, Mar 18, 2022 at 2:59 PM Jens Axboe <axboe@kernel.dk> wrote:
>>
>> This will throw a merge conflict in drivers/nvme/target/configfs.c,
>> resolution is just to delete the two discovery helpers and the configfs
>> attribute, basically everything in the conflict section. This is due to
>> conflicting with a last minute revert in 5.17.
> 
> Only because I looked at this conflict did I notice that some of the
> changes are kind of pointless..
> 
> I mean, this is well-meaning, but I'm really not convinced it's
> actually *useful*:
> 
>    -   return sprintf(page, "\n");
>    +   return snprintf(page, PAGE_SIZE, "\n");
> 
> It's not like a two-byte copy can ever overflow PAGE_SIZE.
> 
> Sometimes 'sprintf() -> snprintf()' conversions don't really buy you anything.
> 
>                  Linus
> 

I did that to keep the code uniform in the
drivers/nvme/target/configfs.c.

In future will avoid submitting such patches.

-ck