Netdev List
 help / color / mirror / Atom feed
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?

  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