[PATCH 7/7] nvme: Send change uevent when AEN completes

Keith Busch keith.busch at intel.com
Thu Jul 13 09:09:17 PDT 2017


On Tue, Jul 11, 2017 at 10:14:20AM -0700, James Smart wrote:
> On 7/11/2017 7:43 AM, Keith Busch wrote:
> > I agree that usage for dynamic strings is better, but this is a fixed
> > sized string that will always be 20 bytes.
> 
> My scenario was the same - fixed sized, not dynamic.  Just being the
> messenger for consistency.

Wha?! That style point is not one I'm convinced is better. Not only is
it more complicated, the stack overhead of calling kasprintf is higher
than the 20 bytes I'm using here.



More information about the Linux-nvme mailing list