Re: [PATCH v3 3/3] nvmet: preserve controller serial number between reboots

From: Christoph Hellwig
Date: Fri Jul 14 2017 - 08:56:17 EST


> + down_write(&nvmet_config_sem);
> subsys->ver = NVME_VS(1, 3, 0); /* NVMe 1.3.0 */
> /* generate a random serial number as our controllers are ephemeral: */
> get_random_bytes(&subsys->serial, sizeof(subsys->serial));
> + up_write(&nvmet_config_sem);

no need for the lock here, the subsystems isn't visible at this point.

Otherwise:

Reviewed-by: Christoph Hellwig <hch@xxxxxx>