[dm-devel] [PATCH V2] blk-mq: introduce BLK_STS_DEV_RESOURCE
Ming Lei
ming.lei at redhat.com
Tue Jan 23 14:16:27 UTC 2018
On Tue, Jan 23, 2018 at 08:30:41AM -0500, Martin K. Petersen wrote:
>
> Ming,
>
> > + * Block layer and block driver specific status, which is ususally returnd
> ^^^^^^^
> > + * from driver to block layer in IO path.
>
> Given that the comment blurb is long and the flag not defined until
> later, it is not entirely obvious that you are documenting
> BLK_STS_DEV_RESOURCE. So please make that clear at the beginning of the
> comment.
>
> > + * If driver isn't sure if the queue can be run again for dealing with the
> > + * current request after this kind of resource is available, please return
> > + * BLK_STS_SOURCE, for example, when memory allocation, DMA Mapping or other
> ^^^^^^^^^^^^^^
This one is correct, just try to describe their difference.
--
Ming
More information about the dm-devel
mailing list