[PATCH V3 1/7] nvme-core: use I/O timeout in submit sync cmd

Chaitanya Kulkarni Chaitanya.Kulkarni at wdc.com
Wed Sep 16 16:46:23 EDT 2020


On 9/16/20 00:09, Christoph Hellwig wrote:
>> +	req->timeout = timeout;
> The void *queuedata thing is really weird.
> 
> I'd keep the check for an existing timeout in the callers and just
> write this as:
> 
> static inline void nvme_req_set_default_timeout(struct request *req)
> {
> 	if (req->queuedata)
> 		req->timeout = NVME_IO_TIMEOUT
> 	else /* no queuedata implies admin queue */
> 		req->timeout = ADMIN_TIMEOUT;
> }
> 

Okay I'll add in V4.



More information about the Linux-nvme mailing list