All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jakub Kicinski <kuba@kernel.org>
To: Stephen Hemminger <stephen@networkplumber.org>
Cc: Jamal Hadi Salim <jhs@mojatatu.com>,
	dsahern@gmail.com, netdev@vger.kernel.org
Subject: Re: [PATCH iproute2] genl: print caps for all families
Date: Fri, 24 Feb 2023 16:56:18 -0800	[thread overview]
Message-ID: <20230224165618.5e4cbbf4@kernel.org> (raw)
In-Reply-To: <20230224143327.4221f8a5@hermes.local>

On Fri, 24 Feb 2023 14:33:27 -0800 Stephen Hemminger wrote:
> > I'm biased but at this point the time is probably better spent trying
> > to filling the gaps in ynl than add JSON to a CLI tool nobody knows
> > about... too harsh? :  
> 
> So I can drop it (insert sarcasm here)

It may be useful to experts as a for-human-consumption CLI.

JSON to me implies use in scripts or higher level code, I can't
think of a reason why scripts would poke into genl internals.
E.g. if a script wants something from devlink it will call devlink,
and the devlink tool internally may interrogate genl internals.

IOW we're tapping into a layer in the middle of the tech stack, 
while user wants JSON out of the end of the stack.

[We can replace ynl with a iproute2 internal lib or libml to avoid bias]
Now, what I'm saying is - for devlink - we don't have a easy to use
library which a programmer can interact with to query the family info.
Parsing thru the policy dumps is _hard_. So building a C library for
this seems more fruitful than adding JSON to the CLI tool.

IDK if I'm making sense.  I could well be wrong.

  reply	other threads:[~2023-02-25  0:56 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-02-24  1:52 [PATCH iproute2] genl: print caps for all families Jakub Kicinski
2023-02-24  1:57 ` Jakub Kicinski
2023-02-24  8:33   ` Johannes Berg
2023-02-24 15:15     ` Jamal Hadi Salim
2023-02-24 15:22       ` Jamal Hadi Salim
2023-02-24 17:10         ` Jakub Kicinski
2023-02-24 17:46           ` Jamal Hadi Salim
2023-02-24 18:30             ` Jakub Kicinski
2023-02-24  3:27 ` Stephen Hemminger
2023-02-24 17:11   ` Jakub Kicinski
2023-02-24 17:47     ` Jamal Hadi Salim
2023-02-24 18:29       ` Jakub Kicinski
2023-02-24 22:33         ` Stephen Hemminger
2023-02-25  0:56           ` Jakub Kicinski [this message]
2023-02-25 17:21             ` Jamal Hadi Salim
  -- strict thread matches above, loose matches on Subject: below --
2023-02-25  0:37 Jakub Kicinski
2023-02-25 16:41 ` Jamal Hadi Salim
2023-03-04  2:20 ` patchwork-bot+netdevbpf

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=20230224165618.5e4cbbf4@kernel.org \
    --to=kuba@kernel.org \
    --cc=dsahern@gmail.com \
    --cc=jhs@mojatatu.com \
    --cc=netdev@vger.kernel.org \
    --cc=stephen@networkplumber.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.