Hi Linus, On 3/21/22 17:13, Linus Torvalds wrote: > 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 > I did that to keep the code uniform in the drivers/nvme/target/configfs.c. In future will avoid submitting such patches. -ck
