* [PATCH 1/8] block/partitions/ldm: Convert a kernel-doc header into a non-kernel-doc header
2019-05-31 0:00 [PATCH 0/8] Improve block layer function documentation Bart Van Assche
@ 2019-05-31 0:00 ` Bart Van Assche
2019-05-31 2:00 ` Chaitanya Kulkarni
2019-05-31 0:00 ` [PATCH 2/8] block: Convert blk_invalidate_devt() " Bart Van Assche
` (7 subsequent siblings)
8 siblings, 1 reply; 19+ messages in thread
From: Bart Van Assche @ 2019-05-31 0:00 UTC (permalink / raw)
To: Jens Axboe; +Cc: linux-block, Christoph Hellwig, Bart Van Assche
This patch avoids that the kernel-doc tool warns about this function
header when building with W=1.
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
---
block/partitions/ldm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/block/partitions/ldm.c b/block/partitions/ldm.c
index 6db573f33219..fe5d970e2e60 100644
--- a/block/partitions/ldm.c
+++ b/block/partitions/ldm.c
@@ -19,7 +19,7 @@
#include "check.h"
#include "msdos.h"
-/**
+/*
* ldm_debug/info/error/crit - Output an error message
* @f: A printf format string containing the message
* @...: Variables to substitute into @f
--
2.22.0.rc1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [PATCH 1/8] block/partitions/ldm: Convert a kernel-doc header into a non-kernel-doc header
2019-05-31 0:00 ` [PATCH 1/8] block/partitions/ldm: Convert a kernel-doc header into a non-kernel-doc header Bart Van Assche
@ 2019-05-31 2:00 ` Chaitanya Kulkarni
0 siblings, 0 replies; 19+ messages in thread
From: Chaitanya Kulkarni @ 2019-05-31 2:00 UTC (permalink / raw)
To: Bart Van Assche, Jens Axboe; +Cc: linux-block, Christoph Hellwig
Looks good.
Reviewed-by: Chaitanya Kulkarni <chiatanya.kulkarni@wdc.com>
On 5/30/19 5:01 PM, Bart Van Assche wrote:
> This patch avoids that the kernel-doc tool warns about this function
> header when building with W=1.
>
> Signed-off-by: Bart Van Assche <bvanassche@acm.org>
> ---
> block/partitions/ldm.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/block/partitions/ldm.c b/block/partitions/ldm.c
> index 6db573f33219..fe5d970e2e60 100644
> --- a/block/partitions/ldm.c
> +++ b/block/partitions/ldm.c
> @@ -19,7 +19,7 @@
> #include "check.h"
> #include "msdos.h"
>
> -/**
> +/*
> * ldm_debug/info/error/crit - Output an error message
> * @f: A printf format string containing the message
> * @...: Variables to substitute into @f
>
^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH 2/8] block: Convert blk_invalidate_devt() header into a non-kernel-doc header
2019-05-31 0:00 [PATCH 0/8] Improve block layer function documentation Bart Van Assche
2019-05-31 0:00 ` [PATCH 1/8] block/partitions/ldm: Convert a kernel-doc header into a non-kernel-doc header Bart Van Assche
@ 2019-05-31 0:00 ` Bart Van Assche
2019-05-31 2:00 ` Chaitanya Kulkarni
2019-05-31 0:00 ` [PATCH 3/8] block: Fix throtl_pending_timer_fn() kernel-doc header Bart Van Assche
` (6 subsequent siblings)
8 siblings, 1 reply; 19+ messages in thread
From: Bart Van Assche @ 2019-05-31 0:00 UTC (permalink / raw)
To: Jens Axboe; +Cc: linux-block, Christoph Hellwig, Bart Van Assche
This patch avoids that the kernel-doc tool warns about this function
header when building with W=1.
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
---
block/genhd.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/block/genhd.c b/block/genhd.c
index ad6826628e79..24654e1d83e6 100644
--- a/block/genhd.c
+++ b/block/genhd.c
@@ -532,8 +532,8 @@ void blk_free_devt(dev_t devt)
}
}
-/**
- * We invalidate devt by assigning NULL pointer for devt in idr.
+/*
+ * We invalidate devt by assigning NULL pointer for devt in idr.
*/
void blk_invalidate_devt(dev_t devt)
{
--
2.22.0.rc1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [PATCH 2/8] block: Convert blk_invalidate_devt() header into a non-kernel-doc header
2019-05-31 0:00 ` [PATCH 2/8] block: Convert blk_invalidate_devt() " Bart Van Assche
@ 2019-05-31 2:00 ` Chaitanya Kulkarni
0 siblings, 0 replies; 19+ messages in thread
From: Chaitanya Kulkarni @ 2019-05-31 2:00 UTC (permalink / raw)
To: Bart Van Assche, Jens Axboe; +Cc: linux-block, Christoph Hellwig
Looks good.
Reviewed-by: Chaitanya Kulkarni <chiatanya.kulkarni@wdc.com>
On 5/30/19 5:01 PM, Bart Van Assche wrote:
> This patch avoids that the kernel-doc tool warns about this function
> header when building with W=1.
>
> Signed-off-by: Bart Van Assche <bvanassche@acm.org>
> ---
> block/genhd.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/block/genhd.c b/block/genhd.c
> index ad6826628e79..24654e1d83e6 100644
> --- a/block/genhd.c
> +++ b/block/genhd.c
> @@ -532,8 +532,8 @@ void blk_free_devt(dev_t devt)
> }
> }
>
> -/**
> - * We invalidate devt by assigning NULL pointer for devt in idr.
> +/*
> + * We invalidate devt by assigning NULL pointer for devt in idr.
> */
> void blk_invalidate_devt(dev_t devt)
> {
>
^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH 3/8] block: Fix throtl_pending_timer_fn() kernel-doc header
2019-05-31 0:00 [PATCH 0/8] Improve block layer function documentation Bart Van Assche
2019-05-31 0:00 ` [PATCH 1/8] block/partitions/ldm: Convert a kernel-doc header into a non-kernel-doc header Bart Van Assche
2019-05-31 0:00 ` [PATCH 2/8] block: Convert blk_invalidate_devt() " Bart Van Assche
@ 2019-05-31 0:00 ` Bart Van Assche
2019-05-31 2:01 ` Chaitanya Kulkarni
2019-05-31 2:08 ` Kees Cook
2019-05-31 0:00 ` [PATCH 4/8] block: Fix blk_mq_*_map_queues() kernel-doc headers Bart Van Assche
` (5 subsequent siblings)
8 siblings, 2 replies; 19+ messages in thread
From: Bart Van Assche @ 2019-05-31 0:00 UTC (permalink / raw)
To: Jens Axboe; +Cc: linux-block, Christoph Hellwig, Bart Van Assche, Kees Cook
Commit e99e88a9d2b0 renamed a function argument without updating the
corresponding kernel-doc header. Update the kernel-doc header.
Cc: Kees Cook <keescook@chromium.org>
Fixes: e99e88a9d2b0 ("treewide: setup_timer() -> timer_setup()") # v4.15.
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
---
block/blk-throttle.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/block/blk-throttle.c b/block/blk-throttle.c
index 1b97a73d2fb1..9ea7c0ecad10 100644
--- a/block/blk-throttle.c
+++ b/block/blk-throttle.c
@@ -1220,7 +1220,7 @@ static bool throtl_can_upgrade(struct throtl_data *td,
struct throtl_grp *this_tg);
/**
* throtl_pending_timer_fn - timer function for service_queue->pending_timer
- * @arg: the throtl_service_queue being serviced
+ * @t: the pending_timer member of the throtl_service_queue being serviced
*
* This timer is armed when a child throtl_grp with active bio's become
* pending and queued on the service_queue's pending_tree and expires when
--
2.22.0.rc1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [PATCH 3/8] block: Fix throtl_pending_timer_fn() kernel-doc header
2019-05-31 0:00 ` [PATCH 3/8] block: Fix throtl_pending_timer_fn() kernel-doc header Bart Van Assche
@ 2019-05-31 2:01 ` Chaitanya Kulkarni
2019-05-31 2:08 ` Kees Cook
1 sibling, 0 replies; 19+ messages in thread
From: Chaitanya Kulkarni @ 2019-05-31 2:01 UTC (permalink / raw)
To: Bart Van Assche, Jens Axboe; +Cc: linux-block, Christoph Hellwig, Kees Cook
Looks good.
Reviewed-by: Chaitanya Kulkarni <chiatanya.kulkarni@wdc.com>
On 5/30/19 5:01 PM, Bart Van Assche wrote:
> Commit e99e88a9d2b0 renamed a function argument without updating the
> corresponding kernel-doc header. Update the kernel-doc header.
>
> Cc: Kees Cook <keescook@chromium.org>
> Fixes: e99e88a9d2b0 ("treewide: setup_timer() -> timer_setup()") # v4.15.
> Signed-off-by: Bart Van Assche <bvanassche@acm.org>
> ---
> block/blk-throttle.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/block/blk-throttle.c b/block/blk-throttle.c
> index 1b97a73d2fb1..9ea7c0ecad10 100644
> --- a/block/blk-throttle.c
> +++ b/block/blk-throttle.c
> @@ -1220,7 +1220,7 @@ static bool throtl_can_upgrade(struct throtl_data *td,
> struct throtl_grp *this_tg);
> /**
> * throtl_pending_timer_fn - timer function for service_queue->pending_timer
> - * @arg: the throtl_service_queue being serviced
> + * @t: the pending_timer member of the throtl_service_queue being serviced
> *
> * This timer is armed when a child throtl_grp with active bio's become
> * pending and queued on the service_queue's pending_tree and expires when
>
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 3/8] block: Fix throtl_pending_timer_fn() kernel-doc header
2019-05-31 0:00 ` [PATCH 3/8] block: Fix throtl_pending_timer_fn() kernel-doc header Bart Van Assche
2019-05-31 2:01 ` Chaitanya Kulkarni
@ 2019-05-31 2:08 ` Kees Cook
1 sibling, 0 replies; 19+ messages in thread
From: Kees Cook @ 2019-05-31 2:08 UTC (permalink / raw)
To: Bart Van Assche; +Cc: Jens Axboe, linux-block, Christoph Hellwig
On Thu, May 30, 2019 at 05:00:48PM -0700, Bart Van Assche wrote:
> Commit e99e88a9d2b0 renamed a function argument without updating the
> corresponding kernel-doc header. Update the kernel-doc header.
>
> Cc: Kees Cook <keescook@chromium.org>
> Fixes: e99e88a9d2b0 ("treewide: setup_timer() -> timer_setup()") # v4.15.
> Signed-off-by: Bart Van Assche <bvanassche@acm.org>
Reviewed-by: Kees Cook <keescook@chromium.org>
-Kees
> ---
> block/blk-throttle.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/block/blk-throttle.c b/block/blk-throttle.c
> index 1b97a73d2fb1..9ea7c0ecad10 100644
> --- a/block/blk-throttle.c
> +++ b/block/blk-throttle.c
> @@ -1220,7 +1220,7 @@ static bool throtl_can_upgrade(struct throtl_data *td,
> struct throtl_grp *this_tg);
> /**
> * throtl_pending_timer_fn - timer function for service_queue->pending_timer
> - * @arg: the throtl_service_queue being serviced
> + * @t: the pending_timer member of the throtl_service_queue being serviced
> *
> * This timer is armed when a child throtl_grp with active bio's become
> * pending and queued on the service_queue's pending_tree and expires when
> --
> 2.22.0.rc1
>
--
Kees Cook
^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH 4/8] block: Fix blk_mq_*_map_queues() kernel-doc headers
2019-05-31 0:00 [PATCH 0/8] Improve block layer function documentation Bart Van Assche
` (2 preceding siblings ...)
2019-05-31 0:00 ` [PATCH 3/8] block: Fix throtl_pending_timer_fn() kernel-doc header Bart Van Assche
@ 2019-05-31 0:00 ` Bart Van Assche
2019-05-31 2:01 ` Chaitanya Kulkarni
2019-05-31 0:00 ` [PATCH 5/8] block: Fix rq_qos_wait() kernel-doc header Bart Van Assche
` (4 subsequent siblings)
8 siblings, 1 reply; 19+ messages in thread
From: Bart Van Assche @ 2019-05-31 0:00 UTC (permalink / raw)
To: Jens Axboe
Cc: linux-block, Christoph Hellwig, Bart Van Assche, Hannes Reinecke,
Keith Busch
This patch avoids that the kernel-doc script complains about these
function headers when building with W=1.
Cc: Hannes Reinecke <hare@suse.com>
Cc: Keith Busch <keith.busch@intel.com>
Fixes: ed76e329d74a ("blk-mq: abstract out queue map") # v5.0.
Fixes: e42b3867de4b ("blk-mq-rdma: pass in queue map to blk_mq_rdma_map_queues") # v5.0.
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
---
block/blk-mq-pci.c | 2 +-
block/blk-mq-rdma.c | 4 ++--
block/blk-mq-virtio.c | 4 ++--
3 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/block/blk-mq-pci.c b/block/blk-mq-pci.c
index ad4545a2a98b..b595a94c4d16 100644
--- a/block/blk-mq-pci.c
+++ b/block/blk-mq-pci.c
@@ -13,7 +13,7 @@
/**
* blk_mq_pci_map_queues - provide a default queue mapping for PCI device
- * @set: tagset to provide the mapping for
+ * @qmap: CPU to hardware queue map.
* @pdev: PCI device associated with @set.
* @offset: Offset to use for the pci irq vector
*
diff --git a/block/blk-mq-rdma.c b/block/blk-mq-rdma.c
index cc921e6ba709..14f968e58b8f 100644
--- a/block/blk-mq-rdma.c
+++ b/block/blk-mq-rdma.c
@@ -8,8 +8,8 @@
/**
* blk_mq_rdma_map_queues - provide a default queue mapping for rdma device
- * @set: tagset to provide the mapping for
- * @dev: rdma device associated with @set.
+ * @map: CPU to hardware queue map.
+ * @dev: rdma device to provide a mapping for.
* @first_vec: first interrupt vectors to use for queues (usually 0)
*
* This function assumes the rdma device @dev has at least as many available
diff --git a/block/blk-mq-virtio.c b/block/blk-mq-virtio.c
index 75a52c18a8f6..488341628256 100644
--- a/block/blk-mq-virtio.c
+++ b/block/blk-mq-virtio.c
@@ -11,8 +11,8 @@
/**
* blk_mq_virtio_map_queues - provide a default queue mapping for virtio device
- * @set: tagset to provide the mapping for
- * @vdev: virtio device associated with @set.
+ * @qmap: CPU to hardware queue map.
+ * @vdev: virtio device to provide a mapping for.
* @first_vec: first interrupt vectors to use for queues (usually 0)
*
* This function assumes the virtio device @vdev has at least as many available
--
2.22.0.rc1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [PATCH 4/8] block: Fix blk_mq_*_map_queues() kernel-doc headers
2019-05-31 0:00 ` [PATCH 4/8] block: Fix blk_mq_*_map_queues() kernel-doc headers Bart Van Assche
@ 2019-05-31 2:01 ` Chaitanya Kulkarni
0 siblings, 0 replies; 19+ messages in thread
From: Chaitanya Kulkarni @ 2019-05-31 2:01 UTC (permalink / raw)
To: Bart Van Assche, Jens Axboe
Cc: linux-block, Christoph Hellwig, Hannes Reinecke, Keith Busch
Looks good.
Reviewed-by: Chaitanya Kulkarni <chiatanya.kulkarni@wdc.com>
On 5/30/19 5:01 PM, Bart Van Assche wrote:
> This patch avoids that the kernel-doc script complains about these
> function headers when building with W=1.
>
> Cc: Hannes Reinecke <hare@suse.com>
> Cc: Keith Busch <keith.busch@intel.com>
> Fixes: ed76e329d74a ("blk-mq: abstract out queue map") # v5.0.
> Fixes: e42b3867de4b ("blk-mq-rdma: pass in queue map to blk_mq_rdma_map_queues") # v5.0.
> Signed-off-by: Bart Van Assche <bvanassche@acm.org>
> ---
> block/blk-mq-pci.c | 2 +-
> block/blk-mq-rdma.c | 4 ++--
> block/blk-mq-virtio.c | 4 ++--
> 3 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/block/blk-mq-pci.c b/block/blk-mq-pci.c
> index ad4545a2a98b..b595a94c4d16 100644
> --- a/block/blk-mq-pci.c
> +++ b/block/blk-mq-pci.c
> @@ -13,7 +13,7 @@
>
> /**
> * blk_mq_pci_map_queues - provide a default queue mapping for PCI device
> - * @set: tagset to provide the mapping for
> + * @qmap: CPU to hardware queue map.
> * @pdev: PCI device associated with @set.
> * @offset: Offset to use for the pci irq vector
> *
> diff --git a/block/blk-mq-rdma.c b/block/blk-mq-rdma.c
> index cc921e6ba709..14f968e58b8f 100644
> --- a/block/blk-mq-rdma.c
> +++ b/block/blk-mq-rdma.c
> @@ -8,8 +8,8 @@
>
> /**
> * blk_mq_rdma_map_queues - provide a default queue mapping for rdma device
> - * @set: tagset to provide the mapping for
> - * @dev: rdma device associated with @set.
> + * @map: CPU to hardware queue map.
> + * @dev: rdma device to provide a mapping for.
> * @first_vec: first interrupt vectors to use for queues (usually 0)
> *
> * This function assumes the rdma device @dev has at least as many available
> diff --git a/block/blk-mq-virtio.c b/block/blk-mq-virtio.c
> index 75a52c18a8f6..488341628256 100644
> --- a/block/blk-mq-virtio.c
> +++ b/block/blk-mq-virtio.c
> @@ -11,8 +11,8 @@
>
> /**
> * blk_mq_virtio_map_queues - provide a default queue mapping for virtio device
> - * @set: tagset to provide the mapping for
> - * @vdev: virtio device associated with @set.
> + * @qmap: CPU to hardware queue map.
> + * @vdev: virtio device to provide a mapping for.
> * @first_vec: first interrupt vectors to use for queues (usually 0)
> *
> * This function assumes the virtio device @vdev has at least as many available
>
^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH 5/8] block: Fix rq_qos_wait() kernel-doc header
2019-05-31 0:00 [PATCH 0/8] Improve block layer function documentation Bart Van Assche
` (3 preceding siblings ...)
2019-05-31 0:00 ` [PATCH 4/8] block: Fix blk_mq_*_map_queues() kernel-doc headers Bart Van Assche
@ 2019-05-31 0:00 ` Bart Van Assche
2019-05-31 2:02 ` Chaitanya Kulkarni
2019-05-31 0:00 ` [PATCH 6/8] block: Fix bsg_setup_queue() " Bart Van Assche
` (3 subsequent siblings)
8 siblings, 1 reply; 19+ messages in thread
From: Bart Van Assche @ 2019-05-31 0:00 UTC (permalink / raw)
To: Jens Axboe; +Cc: linux-block, Christoph Hellwig, Bart Van Assche
Add documentation for the @rqw argument and change " - " into ": ".
Fixes: 84f603246db9 ("block: add rq_qos_wait to rq_qos") # v5.0-rc1~52^2~140.
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
---
block/blk-rq-qos.c | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/block/blk-rq-qos.c b/block/blk-rq-qos.c
index 3f55b56f24bc..659ccb8b693f 100644
--- a/block/blk-rq-qos.c
+++ b/block/blk-rq-qos.c
@@ -209,9 +209,10 @@ static int rq_qos_wake_function(struct wait_queue_entry *curr,
/**
* rq_qos_wait - throttle on a rqw if we need to
- * @private_data - caller provided specific data
- * @acquire_inflight_cb - inc the rqw->inflight counter if we can
- * @cleanup_cb - the callback to cleanup in case we race with a waker
+ * @rqw: rqw to throttle on
+ * @private_data: caller provided specific data
+ * @acquire_inflight_cb: inc the rqw->inflight counter if we can
+ * @cleanup_cb: the callback to cleanup in case we race with a waker
*
* This provides a uniform place for the rq_qos users to do their throttling.
* Since you can end up with a lot of things sleeping at once, this manages the
--
2.22.0.rc1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [PATCH 5/8] block: Fix rq_qos_wait() kernel-doc header
2019-05-31 0:00 ` [PATCH 5/8] block: Fix rq_qos_wait() kernel-doc header Bart Van Assche
@ 2019-05-31 2:02 ` Chaitanya Kulkarni
0 siblings, 0 replies; 19+ messages in thread
From: Chaitanya Kulkarni @ 2019-05-31 2:02 UTC (permalink / raw)
To: Bart Van Assche, Jens Axboe; +Cc: linux-block, Christoph Hellwig
Looks good.
Reviewed-by: Chaitanya Kulkarni <chiatanya.kulkarni@wdc.com>
On 5/30/19 5:01 PM, Bart Van Assche wrote:
> Add documentation for the @rqw argument and change " - " into ": ".
>
> Fixes: 84f603246db9 ("block: add rq_qos_wait to rq_qos") # v5.0-rc1~52^2~140.
> Signed-off-by: Bart Van Assche <bvanassche@acm.org>
> ---
> block/blk-rq-qos.c | 7 ++++---
> 1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/block/blk-rq-qos.c b/block/blk-rq-qos.c
> index 3f55b56f24bc..659ccb8b693f 100644
> --- a/block/blk-rq-qos.c
> +++ b/block/blk-rq-qos.c
> @@ -209,9 +209,10 @@ static int rq_qos_wake_function(struct wait_queue_entry *curr,
>
> /**
> * rq_qos_wait - throttle on a rqw if we need to
> - * @private_data - caller provided specific data
> - * @acquire_inflight_cb - inc the rqw->inflight counter if we can
> - * @cleanup_cb - the callback to cleanup in case we race with a waker
> + * @rqw: rqw to throttle on
> + * @private_data: caller provided specific data
> + * @acquire_inflight_cb: inc the rqw->inflight counter if we can
> + * @cleanup_cb: the callback to cleanup in case we race with a waker
> *
> * This provides a uniform place for the rq_qos users to do their throttling.
> * Since you can end up with a lot of things sleeping at once, this manages the
>
^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH 6/8] block: Fix bsg_setup_queue() kernel-doc header
2019-05-31 0:00 [PATCH 0/8] Improve block layer function documentation Bart Van Assche
` (4 preceding siblings ...)
2019-05-31 0:00 ` [PATCH 5/8] block: Fix rq_qos_wait() kernel-doc header Bart Van Assche
@ 2019-05-31 0:00 ` Bart Van Assche
2019-05-31 2:02 ` Chaitanya Kulkarni
2019-05-31 0:00 ` [PATCH 7/8] blk-mq: Fix spelling in a source code comment Bart Van Assche
` (2 subsequent siblings)
8 siblings, 1 reply; 19+ messages in thread
From: Bart Van Assche @ 2019-05-31 0:00 UTC (permalink / raw)
To: Jens Axboe; +Cc: linux-block, Christoph Hellwig, Bart Van Assche
Document all bsg_setup_queue() arguments as required.
Fixes: aae3b069d5ce ("bsg: pass in desired timeout handler") # v5.0.
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
---
block/bsg-lib.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/block/bsg-lib.c b/block/bsg-lib.c
index b898a1cdf872..785dd58947f1 100644
--- a/block/bsg-lib.c
+++ b/block/bsg-lib.c
@@ -354,6 +354,7 @@ static const struct blk_mq_ops bsg_mq_ops = {
* @dev: device to attach bsg device to
* @name: device to give bsg device
* @job_fn: bsg job handler
+ * @timeout: timeout handler function pointer
* @dd_job_size: size of LLD data needed for each job
*/
struct request_queue *bsg_setup_queue(struct device *dev, const char *name,
--
2.22.0.rc1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [PATCH 6/8] block: Fix bsg_setup_queue() kernel-doc header
2019-05-31 0:00 ` [PATCH 6/8] block: Fix bsg_setup_queue() " Bart Van Assche
@ 2019-05-31 2:02 ` Chaitanya Kulkarni
0 siblings, 0 replies; 19+ messages in thread
From: Chaitanya Kulkarni @ 2019-05-31 2:02 UTC (permalink / raw)
To: Bart Van Assche, Jens Axboe; +Cc: linux-block, Christoph Hellwig
Looks good.
Reviewed-by: Chaitanya Kulkarni <chiatanya.kulkarni@wdc.com>
On 5/30/19 5:01 PM, Bart Van Assche wrote:
> Document all bsg_setup_queue() arguments as required.
>
> Fixes: aae3b069d5ce ("bsg: pass in desired timeout handler") # v5.0.
> Signed-off-by: Bart Van Assche <bvanassche@acm.org>
> ---
> block/bsg-lib.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/block/bsg-lib.c b/block/bsg-lib.c
> index b898a1cdf872..785dd58947f1 100644
> --- a/block/bsg-lib.c
> +++ b/block/bsg-lib.c
> @@ -354,6 +354,7 @@ static const struct blk_mq_ops bsg_mq_ops = {
> * @dev: device to attach bsg device to
> * @name: device to give bsg device
> * @job_fn: bsg job handler
> + * @timeout: timeout handler function pointer
> * @dd_job_size: size of LLD data needed for each job
> */
> struct request_queue *bsg_setup_queue(struct device *dev, const char *name,
>
^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH 7/8] blk-mq: Fix spelling in a source code comment
2019-05-31 0:00 [PATCH 0/8] Improve block layer function documentation Bart Van Assche
` (5 preceding siblings ...)
2019-05-31 0:00 ` [PATCH 6/8] block: Fix bsg_setup_queue() " Bart Van Assche
@ 2019-05-31 0:00 ` Bart Van Assche
2019-05-31 2:03 ` Chaitanya Kulkarni
2019-05-31 0:00 ` [PATCH 8/8] blk-mq: Document the blk_mq_hw_queue_to_node() arguments Bart Van Assche
2019-05-31 16:58 ` [PATCH 0/8] Improve block layer function documentation Jens Axboe
8 siblings, 1 reply; 19+ messages in thread
From: Bart Van Assche @ 2019-05-31 0:00 UTC (permalink / raw)
To: Jens Axboe; +Cc: linux-block, Christoph Hellwig, Bart Van Assche, Max Gurtovoy
Change one occurrence of 'performace' into 'performance'.
Cc: Max Gurtovoy <maxg@mellanox.com>
Fixes: fe631457ff3e ("blk-mq: map all HWQ also in hyperthreaded system") # v4.13.
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
---
block/blk-mq-cpumap.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/block/blk-mq-cpumap.c b/block/blk-mq-cpumap.c
index 48bebf00a5f3..0afa4dc48365 100644
--- a/block/blk-mq-cpumap.c
+++ b/block/blk-mq-cpumap.c
@@ -42,8 +42,8 @@ int blk_mq_map_queues(struct blk_mq_queue_map *qmap)
/*
* First do sequential mapping between CPUs and queues.
* In case we still have CPUs to map, and we have some number of
- * threads per cores then map sibling threads to the same queue for
- * performace optimizations.
+ * threads per cores then map sibling threads to the same queue
+ * for performance optimizations.
*/
if (cpu < nr_queues) {
map[cpu] = cpu_to_queue_index(qmap, nr_queues, cpu);
--
2.22.0.rc1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [PATCH 7/8] blk-mq: Fix spelling in a source code comment
2019-05-31 0:00 ` [PATCH 7/8] blk-mq: Fix spelling in a source code comment Bart Van Assche
@ 2019-05-31 2:03 ` Chaitanya Kulkarni
0 siblings, 0 replies; 19+ messages in thread
From: Chaitanya Kulkarni @ 2019-05-31 2:03 UTC (permalink / raw)
To: Bart Van Assche, Jens Axboe; +Cc: linux-block, Christoph Hellwig, Max Gurtovoy
Looks good.
Reviewed-by: Chaitanya Kulkarni <chiatanya.kulkarni@wdc.com>
On 5/30/19 5:01 PM, Bart Van Assche wrote:
> Change one occurrence of 'performace' into 'performance'.
>
> Cc: Max Gurtovoy <maxg@mellanox.com>
> Fixes: fe631457ff3e ("blk-mq: map all HWQ also in hyperthreaded system") # v4.13.
> Signed-off-by: Bart Van Assche <bvanassche@acm.org>
> ---
> block/blk-mq-cpumap.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/block/blk-mq-cpumap.c b/block/blk-mq-cpumap.c
> index 48bebf00a5f3..0afa4dc48365 100644
> --- a/block/blk-mq-cpumap.c
> +++ b/block/blk-mq-cpumap.c
> @@ -42,8 +42,8 @@ int blk_mq_map_queues(struct blk_mq_queue_map *qmap)
> /*
> * First do sequential mapping between CPUs and queues.
> * In case we still have CPUs to map, and we have some number of
> - * threads per cores then map sibling threads to the same queue for
> - * performace optimizations.
> + * threads per cores then map sibling threads to the same queue
> + * for performance optimizations.
> */
> if (cpu < nr_queues) {
> map[cpu] = cpu_to_queue_index(qmap, nr_queues, cpu);
>
^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH 8/8] blk-mq: Document the blk_mq_hw_queue_to_node() arguments
2019-05-31 0:00 [PATCH 0/8] Improve block layer function documentation Bart Van Assche
` (6 preceding siblings ...)
2019-05-31 0:00 ` [PATCH 7/8] blk-mq: Fix spelling in a source code comment Bart Van Assche
@ 2019-05-31 0:00 ` Bart Van Assche
2019-05-31 2:04 ` Chaitanya Kulkarni
2019-05-31 16:58 ` [PATCH 0/8] Improve block layer function documentation Jens Axboe
8 siblings, 1 reply; 19+ messages in thread
From: Bart Van Assche @ 2019-05-31 0:00 UTC (permalink / raw)
To: Jens Axboe; +Cc: linux-block, Christoph Hellwig, Bart Van Assche
Document the meaning of the blk_mq_hw_queue_to_node() arguments.
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
---
block/blk-mq-cpumap.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/block/blk-mq-cpumap.c b/block/blk-mq-cpumap.c
index 0afa4dc48365..f945621a0e8f 100644
--- a/block/blk-mq-cpumap.c
+++ b/block/blk-mq-cpumap.c
@@ -60,7 +60,11 @@ int blk_mq_map_queues(struct blk_mq_queue_map *qmap)
}
EXPORT_SYMBOL_GPL(blk_mq_map_queues);
-/*
+/**
+ * blk_mq_hw_queue_to_node - Look up the memory node for a hardware queue index
+ * @qmap: CPU to hardware queue map.
+ * @index: hardware queue index.
+ *
* We have no quick way of doing reverse lookups. This is only used at
* queue init time, so runtime isn't important.
*/
--
2.22.0.rc1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [PATCH 8/8] blk-mq: Document the blk_mq_hw_queue_to_node() arguments
2019-05-31 0:00 ` [PATCH 8/8] blk-mq: Document the blk_mq_hw_queue_to_node() arguments Bart Van Assche
@ 2019-05-31 2:04 ` Chaitanya Kulkarni
0 siblings, 0 replies; 19+ messages in thread
From: Chaitanya Kulkarni @ 2019-05-31 2:04 UTC (permalink / raw)
To: Bart Van Assche, Jens Axboe; +Cc: linux-block, Christoph Hellwig
Looks good.
Reviewed-by: Chaitanya Kulkarni <chiatanya.kulkarni@wdc.com>
On 5/30/19 5:01 PM, Bart Van Assche wrote:
> Document the meaning of the blk_mq_hw_queue_to_node() arguments.
>
> Signed-off-by: Bart Van Assche <bvanassche@acm.org>
> ---
> block/blk-mq-cpumap.c | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/block/blk-mq-cpumap.c b/block/blk-mq-cpumap.c
> index 0afa4dc48365..f945621a0e8f 100644
> --- a/block/blk-mq-cpumap.c
> +++ b/block/blk-mq-cpumap.c
> @@ -60,7 +60,11 @@ int blk_mq_map_queues(struct blk_mq_queue_map *qmap)
> }
> EXPORT_SYMBOL_GPL(blk_mq_map_queues);
>
> -/*
> +/**
> + * blk_mq_hw_queue_to_node - Look up the memory node for a hardware queue index
> + * @qmap: CPU to hardware queue map.
> + * @index: hardware queue index.
> + *
> * We have no quick way of doing reverse lookups. This is only used at
> * queue init time, so runtime isn't important.
> */
>
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 0/8] Improve block layer function documentation
2019-05-31 0:00 [PATCH 0/8] Improve block layer function documentation Bart Van Assche
` (7 preceding siblings ...)
2019-05-31 0:00 ` [PATCH 8/8] blk-mq: Document the blk_mq_hw_queue_to_node() arguments Bart Van Assche
@ 2019-05-31 16:58 ` Jens Axboe
8 siblings, 0 replies; 19+ messages in thread
From: Jens Axboe @ 2019-05-31 16:58 UTC (permalink / raw)
To: Bart Van Assche; +Cc: linux-block, Christoph Hellwig
On 5/30/19 6:00 PM, Bart Van Assche wrote:
> Hi Jens,
>
> This is a series with eight patches that address source code comments. Most
> patches suppress kernel-doc warnings that appear when building with W=1. Please
> consider these patches.
Thanks Bart, applied.
--
Jens Axboe
^ permalink raw reply [flat|nested] 19+ messages in thread