On Fri, Jun 16, 2023 at 08:55:59PM +0530, Prathu Baronia wrote: > In various places, string buffers of a fixed size are allocated, and > filled using snprintf() with the same fixed size, which is error-prone. > > Replace this by calling devm_kasprintf() instead, which always uses the > appropriate size. Allocate the device name with a unique identifier > instead of a kernel address. You are doing two different things here, one is changing the allocation way, and the other is the name. If one of those things turns out to break something, we have to revert this whole thing. So please make this two different changes, one to change to use devm_kasprintf() and the second to change the naming scheme, ESPECIALLY as you do not mention the name change in the subject line. And that's going to be a user-visible change, so you need to make that VERY obvious. thanks, greg k-h
