Re: [Xen-devel] [PATCH v5 01/11] x86/pci: introduce hvm_pci_decode_addr

[Roger Pau Monne <roger.pau@xxxxxxxxxx>]

On Thu, Aug 24, 2017 at 09:46:16AM -0600, Jan Beulich wrote:
> >>> On 14.08.17 at 16:28, <roger.pau@xxxxxxxxxx> wrote:
> > --- a/xen/arch/x86/hvm/io.c
> > +++ b/xen/arch/x86/hvm/io.c
> > @@ -256,6 +256,25 @@ void register_g2m_portio_handler(struct domain *d)
> >      handler->ops = &g2m_portio_ops;
> >  }
> >  
> > +unsigned int hvm_pci_decode_addr(unsigned int cf8, unsigned int addr,
> > +                                 unsigned int *bus, unsigned int *slot,
> > +                                 unsigned int *func)
> > +{
> > +    unsigned long bdf;
> 
> Is there a need for this being unsigned long instead of unsigned int?
> If not, I'd be fine changing this while committing.

It's my mistake it certainly needs to be unsigned int. I've wrongly
fixed this in patch 2.

Thanks, Roger.

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