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

[Suravee Suthikulpanit <suravee.suthikulpanit@amd.com>]

On 9/5/2013 2:14 AM, Jan Beulich wrote:
>>>> On 05.09.13 at 00:48, Suravee Suthikulpanit <suravee.suthikulpanit@amd.com> 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