On Thu, Oct 24, 2024 at 07:00:53AM +0200, Christoph Hellwig wrote: > On Thu, Oct 24, 2024 at 09:43:24AM +0800, Ming Lei wrote: > > > so that it's clear why the non_owner version is used here. > > > > There are one more such usage: > > > > - freeze in nvme_dev_disable()/apple_nvme_disable() from timeout work, but > > unfreeze in nvme_reset_work() > > Then add it to the comment :) OK, also nvme_passthru_start() and nvme_passthru_end() are always paired, so they are actually not non_owner use. Thanks, Ming