Re: [PATCH v4 38/52] docs: block: typec_bus.rst: get rid of :c:type
From: André Almeida
Date: Wed Sep 30 2020 - 12:48:36 EST
On 9/30/20 10:25 AM, Mauro Carvalho Chehab wrote:
> The :c:type macros are not used properly there, as reported
> by Sphinx 3:
>
> ./Documentation/block/blk-mq.rst:112: WARNING: Unparseable C cross-reference: 'hctx->dispatch'
> Invalid C declaration: Expected end of definition. [error at 4]
> hctx->dispatch
> ----^
>
> Also, they won't be generating any cross references.
>
> So, replace them by a literal markup.
Your commit message says "docs: block: typec_bus.rst: ...", but the name
of this file is "blk-mq.rst".
>
> Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@xxxxxxxxxx>
After solving the issue above:
Reviewed-by: André Almeida <andrealmeid@xxxxxxxxxxxxx>
> ---
> Documentation/block/blk-mq.rst | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/block/blk-mq.rst b/Documentation/block/blk-mq.rst
> index 86a632af02b0..a980d23af48c 100644
> --- a/Documentation/block/blk-mq.rst
> +++ b/Documentation/block/blk-mq.rst
> @@ -110,9 +110,9 @@ block layer removes requests from the associated software queues and tries to
> dispatch to the hardware.
>
> If it's not possible to send the requests directly to hardware, they will be
> -added to a linked list (:c:type:`hctx->dispatch`) of requests. Then,
> +added to a linked list (``hctx->dispatch``) of requests. Then,
> next time the block layer runs a queue, it will send the requests laying at the
> -:c:type:`dispatch` list first, to ensure a fairness dispatch with those
> +``dispatch`` list first, to ensure a fairness dispatch with those
> requests that were ready to be sent first. The number of hardware queues
> depends on the number of hardware contexts supported by the hardware and its
> device driver, but it will not be more than the number of cores of the system.
>