On Fri, Mar 18, 2022 at 2:59 PM Jens Axboe <axboe@xxxxxxxxx> 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
