[dpdk-dev] [PATCH] pci: fix unneeded includes in public header file

Jerin Jacob jerinjacobk at gmail.com
Fri Mar 20 14:25:47 CET 2020


On Thu, Mar 19, 2020 at 1:41 PM David Marchand
<david.marchand at redhat.com> wrote:
>
> On Mon, Mar 16, 2020 at 8:08 PM David Christensen
> <drc at linux.vnet.ibm.com> wrote:
> > On 3/16/20 10:49 AM, David Christensen wrote:
> > >
> > >> The rte_pci.h file includes more header files than are actually needed,
> > >> which means that all users of it also include those headers. This patch
> > >> removes the unneeded headers - adding them elsewhere where other
> > >> components
> > >> were requiring them but not including them directly.
> > >>
> > >> Fixes: c752998b5e2e ("pci: introduce library and driver")
> > >> Cc: gaetan.rivet at 6wind.com
> > >> Cc: stable at dpdk.org
> > >>
> > >> Signed-off-by: Bruce Richardson <bruce.richardson at intel.com>
> > >
> > > Patch is generating build errors with meson on my POWER system, though I
> > > don't see how they're POWER specific:
> >
> > There's a pending patch to enable Travis for POWER systems:
> >
> > http://patches.dpdk.org/patch/65986/
> >
> > Any reason it hasn't been accepted yet?  Looks like it would have caught
> > this problem.
>
> The reason is that this series has a change in the IOVA detection code.
> I have a lot of things to look at, so any help in reviews is appreciated.

Reviewed...

>
>
> --
> David Marchand
>


More information about the dev mailing list