All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stephen Hemminger <stephen@networkplumber.org>
To: Bruce Richardson <bruce.richardson@intel.com>
Cc: Thomas Monjalon <thomas@monjalon.net>,
	David Marchand <david.marchand@redhat.com>,
	Robin Jarry <rjarry@redhat.com>, <dev@dpdk.org>,
	Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>
Subject: Re: [RFC v2 5/5] eal: configure initial device probing
Date: Fri, 27 Feb 2026 14:10:45 -0800	[thread overview]
Message-ID: <20260227141045.236af55b@phoenix.local> (raw)
In-Reply-To: <aaGhdEEgxiA4Z33A@bricha3-mobl1.ger.corp.intel.com>

On Fri, 27 Feb 2026 13:51:48 +0000
Bruce Richardson <bruce.richardson@intel.com> wrote:

> On Fri, Feb 27, 2026 at 02:43:46PM +0100, Thomas Monjalon wrote:
> > 26/02/2026 17:50, Robin Jarry:  
> > > David Marchand, Feb 26, 2026 at 17:20:  
> > > > Some applications use port hotplug as their primary way for using DPDK
> > > > resources.
> > > > Having a systematic device probing is a problem when not all available
> > > > resources will be used by the application, as such applications won't set
> > > > an explicit allow list at startup.
> > > >
> > > > This is the case for OVS on systems with multiple mlx5 devices:
> > > > one device can be used by the kernel while the other(s) are used by DPDK.
> > > > In such a setup, the kernel used device may get reconfigured in
> > > > unexpected ways and trigger issues like the one described by Kevin
> > > > not so long ago in bugzilla 1873.
> > > >
> > > > Add an EAL option so that we can change the default behavior from
> > > > block-listing to allow-listing.  
> > [...]  
> > > > +	const char * const argv29[] = {prgname, prefix, mp_flag, eal_debug_logs,
> > > > +				       "--allow-explicitly" };  
> > > 
> > > I am not convinced by the option name. What do you think of:
> > > 
> > > 	--no-autoprobe
> > > 
> > > That would match the Linux sriov_drivers_autoprobe sysfs.  
> > 
> > The name --no-autoprobe is better indeed.
> > 
> > The exact effect of this option is to disable initial probing
> > of devices on all buses (except vdev).
> > Another name could be --no-initial-probing
> > 
> > I think we should add the opposite option as well
> > to allow changing the default mode later.
> > For such an option, --autoprobe looks better than --initial-probing.
> > 
> > Other opinions?
> > 
> > 
> > [...]  
> > > Depending on what option name we settle on, could you add a short flag
> > > too? E.g.:
> > > 
> > > BOOL_ARG("--no-autoprobe", "-N", "Disable automatic probing of non-blocked devices", no_autoprobe)
> > > 
> > > Or:
> > > 
> > > BOOL_ARG("--no-autoprobe", "-P", "Disable automatic probing of non-blocked devices", no_autoprobe)  
> > 
> > I don't see the benefit of a short flag.
> > It makes reading commands less obvious.
> >   
> I actually would prefer to have a short option available, and I'd really
> like that short option to be "-A" since it serves as the perfect addition
> to the "-a" flag to specify devices to probe.
> 
> Based on that, I would look for long options which allow "-A" as the short
> version for example:
> 
> --allowlisted-devs-only
> 
> /Bruce

Also if -b or --block-list become a no op with --no-autoprobe. So it should be a warning?

  reply	other threads:[~2026-02-27 22:10 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-02-25 11:29 [RFC 1/2] devtools: check packet forwarding in null test David Marchand
2026-02-25 11:29 ` [RFC 2/2] eal: configure initial device probing David Marchand
2026-02-25 12:09   ` Bruce Richardson
2026-02-25 17:53     ` David Marchand
2026-02-26 16:20 ` [RFC v2 0/5] Rework " David Marchand
2026-02-26 16:20   ` [RFC v2 1/5] devtools: check packet forwarding in null test David Marchand
2026-02-26 16:35     ` Bruce Richardson
2026-02-26 16:20   ` [RFC v2 2/5] bus/fslmc: fix bus cleanup David Marchand
2026-02-26 16:20   ` [RFC v2 3/5] drivers/bus: require probe function for NXP drivers David Marchand
2026-02-26 16:24     ` Bruce Richardson
2026-02-26 16:20   ` [RFC v2 4/5] bus: factorize device selection David Marchand
2026-02-26 16:31     ` Bruce Richardson
2026-02-27 14:17       ` David Marchand
2026-02-27 14:33         ` Bruce Richardson
2026-02-26 16:20   ` [RFC v2 5/5] eal: configure initial device probing David Marchand
2026-02-26 16:34     ` Bruce Richardson
2026-02-26 16:50     ` Robin Jarry
2026-02-27 13:43       ` Thomas Monjalon
2026-02-27 13:51         ` Bruce Richardson
2026-02-27 22:10           ` Stephen Hemminger [this message]
2026-03-02  9:02             ` Bruce Richardson
2026-03-02 11:08               ` Morten Brørup
2026-03-02 11:13               ` David Marchand
2026-03-05 16:45 ` [RFC v3 0/7] Rework " David Marchand
2026-03-05 16:45   ` [RFC v3 1/7] devtools: check packet forwarding in null test David Marchand
2026-03-05 16:45   ` [RFC v3 2/7] bus/fslmc: fix bus cleanup David Marchand
2026-03-05 16:45   ` [RFC v3 3/7] drivers/bus: require probe function for NXP drivers David Marchand
2026-03-05 16:45   ` [RFC v3 4/7] drivers: cleanup devargs lookup in bus scan David Marchand
2026-03-05 16:56     ` Bruce Richardson
2026-03-05 16:45   ` [RFC v3 5/7] bus: factorize devargs lookup David Marchand
2026-03-05 17:06     ` Bruce Richardson
2026-03-05 17:10       ` David Marchand
2026-03-05 16:45   ` [RFC v3 6/7] bus: factorize device selection David Marchand
2026-03-05 17:30     ` Bruce Richardson
2026-03-09  9:50       ` David Marchand
2026-03-07 20:59     ` Robin Jarry
2026-03-05 16:45   ` [RFC v3 7/7] eal: configure initial device probing David Marchand
2026-03-05 17:33     ` Bruce Richardson
2026-03-07 21:05     ` Robin Jarry
2026-03-06  8:26   ` [RFC v3 0/7] Rework " David Marchand

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=20260227141045.236af55b@phoenix.local \
    --to=stephen@networkplumber.org \
    --cc=andrew.rybchenko@oktetlabs.ru \
    --cc=bruce.richardson@intel.com \
    --cc=david.marchand@redhat.com \
    --cc=dev@dpdk.org \
    --cc=rjarry@redhat.com \
    --cc=thomas@monjalon.net \
    /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.