Ok, that make sense, I will re-correct it then. Thanks.

Best regards,
Jeff Guo

-----Original Message-----
From: Mcnamara, John 
Sent: Thursday, January 19, 2017 10:54 PM
To: Guo, Jia <[email protected]>; Zhang, Helin <[email protected]>; Wu, 
Jingjing <[email protected]>
Cc: [email protected]; [email protected]
Subject: RE: [dpdk-dev] [PATCH v3] doc: add known uio_pci_generic issue for i40e

> -----Original Message-----
> From: Guo, Jia
> Sent: Thursday, January 19, 2017 1:50 PM
> To: Mcnamara, John <[email protected]>; Zhang, Helin 
> <[email protected]>; Wu, Jingjing <[email protected]>
> Cc: [email protected]; [email protected]
> Subject: RE: [dpdk-dev] [PATCH v3] doc: add known uio_pci_generic 
> issue for i40e
> 
> Hi,john
>       The "bind uio_pci_generic`` is not the definitely command, but 
> very appreciate for your correcting the other syntax issue. I will 
> involve them in v4, Thanks very much.

Hi Jeff, 

Variables, functions, program names, file names and any other literal should be 
in backtick quotes, as outlined in the DPDK Documentation: 

http://dpdk.org/doc/guides/contributing/documentation.html#code-and-literal-block-sections

So uio_pci_generic should be quoted with backticks. If you want to add bind 
then do:
``--bind= uio_pci_generic`` or similar.

Either way it shouldn't be quoted using the "smart" Unicode or non-Unicode 
quotes as in the patch. All quotes should be straight ASCII double quotes (""). 
The Html/PDF renderers will convert to the correct quotes as required.



> > related `linux kernel commit
> > +   < https://git.kernel.org/cgit/linux/kernel/git/stable/linux-
> > stable.git/commit/drivers/pci/quirks.c?id=8bcf4525c5d43306c5fd07e132
> > bc
> > 8650
> > e3491aec>`_.

Also, this link generates a doc build error. I think because of the space 
between the < and the https:.

John

Reply via email to