[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 11/18 V2]: PVH xen: some misc changes like mtrr, intr, msi.



On Mon, 18 Mar 2013 12:21:11 +0000
"Jan Beulich" <JBeulich@xxxxxxxx> wrote:

>  >>> On 16.03.13 at 01:46, Mukesh Rathor <mukesh.rathor@xxxxxxxxxx>
>  >>> wrote:
> > +++ b/xen/arch/x86/msi.c
> > @@ -787,10 +787,15 @@ static int msix_capability_init(struct
> > pci_dev *dev, 
> >      if ( !dev->msix_used_entries )
> >      {
> > -        if ( rangeset_add_range(mmio_ro_ranges,
> > dev->msix_table.first,
> > +        /* PVH: this is temporary only until linux msi.c is fixed.
> > See xen-devel
> > +         * thread: "[PVH]: Help: msi.c".
> > +         */
> > +        if ( !is_pvh_domain(dev->domain) &&
> > +             rangeset_add_range(mmio_ro_ranges,
> > dev->msix_table.first, dev->msix_table.last) )
> >              WARN();
> > -        if ( rangeset_add_range(mmio_ro_ranges,
> > dev->msix_pba.first,
> > +        if ( !is_pvh_domain(dev->domain) &&
> > +             rangeset_add_range(mmio_ro_ranges,
> > dev->msix_pba.first, dev->msix_pba.last) )
> >              WARN();
> >  
> 
> As already said before, the whole change above has my explicit
> NAK.
> Jan

Do you have any suggestions? Do you want to hold off on entire xen patch
until we go fix linux for this? Or can we just omit this change in the next
V 3 and come back to this later.  Would you be OK with that?

thanks
MR

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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.