On Wed, Jul 08, 2020 at 03:05:03PM +0100, John Garry wrote: > On 08/07/2020 13:27, Ming Lei wrote: > > k; > > - } else if (ret == BLK_STS_ZONE_RESOURCE) { > > + case BLK_STS_RESOURCE: > > + case BLK_STS_DEV_RESOURCE: > > + blk_mq_handle_dev_resource(rq, list); > > + goto out; > > + case BLK_STS_ZONE_RESOURCE: > > /* > > * Move the request to zone_list and keep going through > > * the dispatch list to find more requests the drive can > > question not on this patch specifically: is this supposed to be "driver", > and not "drive"? "driver" is mentioned earlier in the function Hi John, Please focus on change added by this patch instead of existed context code. If you have question on existed code, start a new thread for the discussion. Thanks, Ming