From: Jakub Kicinski <kuba@kernel.org>
To: Saeed Mahameed <saeedm@mellanox.com>
Cc: "davem@davemloft.net" <davem@davemloft.net>,
"jacob.e.keller@intel.com" <jacob.e.keller@intel.com>,
"linux-doc@vger.kernel.org" <linux-doc@vger.kernel.org>,
"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
"rdunlap@infradead.org" <rdunlap@infradead.org>,
Tal Gilboa <talgi@mellanox.com>,
"leon@kernel.org" <leon@kernel.org>
Subject: Re: [PATCH net v2 1/2] docs: networking: convert DIM to RST
Date: Thu, 9 Apr 2020 16:06:58 -0700 [thread overview]
Message-ID: <20200409160658.1b940fcf@kicinski-fedora-PC1C0HJN> (raw)
In-Reply-To: <1210a28bfe1a67818f3f814e38f52923cbd201c0.camel@mellanox.com>
On Thu, 9 Apr 2020 22:46:55 +0000 Saeed Mahameed wrote:
> On Thu, 2020-04-09 at 14:21 -0700, Jakub Kicinski wrote:
> > Convert the Dynamic Interrupt Moderation doc to RST and
> > use the RST features like syntax highlight, function and
> > structure documentation, enumerations, table of contents.
> >
> > Signed-off-by: Jakub Kicinski <kuba@kernel.org>
> > Reviewed-by: Randy Dunlap <rdunlap@infradead.org>
> > ---
> > v2:
> > - remove the functions/type definition markup
> > - change the contents definition (the :local: seem to
> > not work too well with kdoc)
> > ---
> > Documentation/networking/index.rst | 1 +
> > .../networking/{net_dim.txt => net_dim.rst} | 90 +++++++++------
> > ----
> > MAINTAINERS | 1 +
> > 3 files changed, 45 insertions(+), 47 deletions(-)
> > rename Documentation/networking/{net_dim.txt => net_dim.rst} (79%)
> >
> > diff --git a/Documentation/networking/index.rst
> > b/Documentation/networking/index.rst
> > index 50133d9761c9..6538ede29661 100644
> > --- a/Documentation/networking/index.rst
> > +++ b/Documentation/networking/index.rst
> > @@ -22,6 +22,7 @@ Linux Networking Documentation
> > z8530book
> > msg_zerocopy
> > failover
> > + net_dim
>
> net_dim is a performance feature, i would move further down the list
> where the perf features such as scaling and offloads are ..
I mean.. so is msg_zerocopy just above ;-) I spotted slight
alphabetical ordering there, which may have not been intentional,
that's why I put it here. Marking with # things out of order, but
based on just the first letter:
# netdev-FAQ
af_xdp
bareudp
batman-adv
can
can_ucan_protocol
device_drivers/index
dsa/index
devlink/index
ethtool-netlink
ieee802154
j1939
kapi
# z8530book
msg_zerocopy
# failover
net_dim
net_failover
phy
sfp-phylink
# alias
# bridge
snmp_counter
# checksum-offloads
segmentation-offloads
scaling
tls
tls-offload
# nfc
6lowpan
My feeling is that we should start considering splitting kernel-only
docs and admin-only docs for networking, which I believe is the
direction Jon and folks want Documentation/ to go. But I wasn't brave
enough to be the first one. Then we can impose some more structure,
like putting all "performance" docs in one subdir..?
WDYT?
next prev parent reply other threads:[~2020-04-09 23:07 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-04-09 21:21 [PATCH net v2 1/2] docs: networking: convert DIM to RST Jakub Kicinski
2020-04-09 21:21 ` [PATCH net v2 2/2] docs: networking: add full DIM API Jakub Kicinski
2020-04-09 22:46 ` [PATCH net v2 1/2] docs: networking: convert DIM to RST Saeed Mahameed
2020-04-09 23:06 ` Jakub Kicinski [this message]
2020-04-10 21:59 ` Saeed Mahameed
2020-04-11 1:25 ` Jakub Kicinski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200409160658.1b940fcf@kicinski-fedora-PC1C0HJN \
--to=kuba@kernel.org \
--cc=davem@davemloft.net \
--cc=jacob.e.keller@intel.com \
--cc=leon@kernel.org \
--cc=linux-doc@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=rdunlap@infradead.org \
--cc=saeedm@mellanox.com \
--cc=talgi@mellanox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox