Re: [Xen-devel] [PATCH V2] x86/AMD-Vi: Add additional check for invalid special->handle

[Suravee Suthikulpanit <suravee.suthikulpanit@xxxxxxx>]

On 9/5/2013 2:14 AM, Jan Beulich wrote:
> > > > On 05.09.13 at 00:48, Suravee Suthikulpanit <suravee.suthikulpanit@xxxxxxx> 
> > > > wrote:
> > On 9/4/2013 4:57 AM, Jan Beulich wrote:
> > > It would, btw, be possible to get along with a single command line
> > > option, just having two alternating forms:
> > >
> > > ivrs_ioapic[id]=<sbdf>
> > > ivrs_ioapic[<sbdf>]=<id>
> > After looking through the examples above, I still think we only need the
> > "ivrs_ioapic[<sbdf>]=<id>" version.
> Perhaps, but then could you talk to your Linux side colleagues to
> find out why they picked (only) the other alternative? I'd really
> like to keep such workarounds largely in sync (a superset is fine,
> but a subset, not to speak of a disjoint set, are not) with Linux...
>
> Jan
At this point, I think it's mainly syntax.  Since the implementation is 
different.  If you want to keep the syntax the same (i.e. 
ivrs_ioapic[id]=<sdbf>), we can do that.  I don't think Linux tries to 
interpret as "pin id" or "pin sdbf".  All it does is just ties both 
values together.
Suravee


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