Messages in this thread | | | Subject | Re: [PATCH 1/2] block: add initial kdoc over the request_queue | From | Bart Van Assche <> | Date | Sun, 28 Jun 2020 13:53:50 -0700 |
| |
On 2020-06-23 15:03, Luis Chamberlain wrote: > +/** > + * struct request_queue - block device driver request queue
Since request queues are used as a communication channel between the block layer core and block drivers and since the block layer core creates request queues, I propose to leave out the word "driver".
> + * @debugfs_mutex: used to protect access to the @debugfs_dir > + * @blk_trace: used by blktrace to keep track of setup / tracing > + * @debugfs_dir: directory created to place debugfs information. This is always > + * created for make_request and request-based block drivers upon > + * initialization. blktrace requires for this directory to be created, > + * and so it will be created on demand if its block driver type does not > + * create it opon initialization. > + * > + * The request_queue is used to manage incoming block layer device driver > + * requests. We have three main type of block driver types which end up making > + * use of the request_queue:
"incoming" is correct from the perspective of the block driver but not from the perspective of the block layer core. How about describing request queues as a communication channel between the block layer core and block drivers?
Thanks,
Bart.
| |