Re: [Xen-devel] Fw: drivers/xen/pci.c:31:25: fatal error: asm/pci_x86.h: No such file or directory

[Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>]

On Thu, 6 Aug 2015, Boris Ostrovsky wrote:
> On 08/06/2015 01:04 PM, Robert Richter wrote:
> > Boris,
> > 
> > we are working on acpi pci support for arm64. For this we are enabling
> > PCI_MMCONFIG on arm64 which breaks compiling drivers/xen/pci.c.
> > 
> > Looking into it there is x86 code in generic driver code introduced
> > with:
> > 
> >   8deb3eb1461e xen/mcfg: Call PHYSDEVOP_pci_mmcfg_reserved for MCFG areas.
> > 
> > This implements:
> > 
> > diff --git a/drivers/xen/pci.c b/drivers/xen/pci.c
> > index 18fff88254eb..d15f6e80479f 100644
> > --- a/drivers/xen/pci.c
> > +++ b/drivers/xen/pci.c
> > @@ -26,6 +26,7 @@
> >   #include <asm/xen/hypervisor.h>
> >   #include <asm/xen/hypercall.h>
> >   #include "../pci/pci.h"
> > +#include <asm/pci_x86.h>
> >     static bool __read_mostly pci_seg_supported = true;
> >   @@ -192,3 +193,49 @@ static int __init register_xen_pci_notifier(void)
> >   }
> >     arch_initcall(register_xen_pci_notifier);
> > +
> > +#ifdef CONFIG_PCI_MMCONFIG
> > +static int __init xen_mcfg_late(void)
> > +{
> > +       struct pci_mmcfg_region *cfg;
> > +       int rc;
> > +
> > +       if (!xen_initial_domain())
> > +               return 0;
> > +
> > +       if ((pci_probe & PCI_PROBE_MMCONF) == 0)
> > +               return 0;
> > [...]
> > 
> > There are no defines for pci_probe and PCI_PROBE_MMCONF other than for
> > the x86 architecture.
> > 
> > I see several ways to fix that:
> > 
> >   * moving code to arch/x86/pci/xen.c,
> 
> If this routine is not going to be used by ARM I think this would be the way
> to go. Stefano --- will this be needed by ARM?

Yes, I think it is most probable that this code will be needed on ARM,
even though we are still at the design doc stage for Xen and ACPI on ARM
at the moment.

Besides I think you should be able to compile this function on arm64
just fine. The hypercall should build without issues. If the problem is
just this one check, we should be able to rework it somehow.

Is there something equivalent to pci_probe on arm? If not, we could just
ifdef the check for x86.


> > 
> >   * make code dependent on arch X86,
> > 
> >   * make 'if (pci_probe & PCI_PROBE_MMCONF) ...' an arch function to be
> >     implemented by archs,
> 
> Otherwise I'd go with this.

Or this of course


> -boris
> 
> > 
> >   * reworking the code by removing the check (not sure if that could be
> >     done).
> > 
> > I don't think PCI_XEN is enabled yet for arm64, so we just disalbe it
> > for !X86 or move it to arch/x86. I suggest the latter. Is there any
> > preference you have?
> 

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel