[Buildroot] [PATCH] libpciaccess: add dependency on hwdata

Carlos Santos casantos at datacom.ind.br
Tue Apr 3 01:38:39 UTC 2018


> From: "Peter Korsgaard" <peter at korsgaard.com>
> To: "Yann E. MORIN" <yann.morin.1998 at free.fr>
> Cc: "Carlos Santos" <casantos at datacom.ind.br>, "Bernd Kuhls" <bernd.kuhls at t-online.de>, "buildroot"
> <buildroot at buildroot.org>
> Sent: Thursday, March 29, 2018 4:05:28 PM
> Subject: Re: [PATCH] libpciaccess: add dependency on hwdata

>>>>>> "Yann" == Yann E MORIN <yann.morin.1998 at free.fr> writes:
> 
> Hi,
> 
> >> It is required by libvirt[1] to show device names when we use virtual
> >> machine manager to add hardware via PCI passthrough, otherwise only
> >> the PCI bus information (slot, port, function) is shown.
> 
> > Then I would say that it is not _required_. It may be _needed_, but it
> > is not mandatory; it just makes it a little bit more user-friendly.
> 
> > Thus, I would say that we do not want to enforce this dependency,
> > because it still works without it.
> 
> And if it IS required when using libvirt with pci passthrough, but not
> necessarily for other users of libpciaccess, then we can make libvirt
> select it.
> 
> In the general case for nice-to-have-but-not-absolutely-required
> dependencies we don't have a solution in Buildroot today except for a
> note in the help text. Upstream kconfig has recently added the imply
> keyword which could be used to implement this, but we would need to sync
> our kconfig with upstream for that.

OK, I will move the selection o hwdata to libvirt in the next version
of the path series.

-- 
Carlos Santos (Casantos) - DATACOM, P&D
“The greatest triumph that modern PR can offer is the transcendent 
success of having your words and actions judged by your reputation, 
rather than the other way about.” — Christopher Hitchens


More information about the buildroot mailing list