From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joerg Roedel Subject: Re: [PATCH 1/3] iommu/amd: Add logic to decode AMD IOMMU event flag Date: Tue, 2 Apr 2013 22:59:04 +0200 Message-ID: <20130402205904.GT30540@8bytes.org> References: <1364428283-2548-1-git-send-email-suravee.suthikulpanit@amd.com> <20130402143335.GB15687@8bytes.org> <20130402144037.GE5488@pd.tnic> <20130402150302.GF15687@8bytes.org> <20130402152956.GE4391@pd.tnic> <20130402160400.GP30540@8bytes.org> <20130402161757.GC17519@pd.tnic> <20130402163318.GS30540@8bytes.org> <20130402193240.GF17675@pd.tnic> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <20130402193240.GF17675-fF5Pk5pvG8Y@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: iommu-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: iommu-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: Borislav Petkov , suravee.suthikulpanit-5C7GfCeVMHo@public.gmane.org, iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: iommu@lists.linux-foundation.org On Tue, Apr 02, 2013 at 09:32:40PM +0200, Borislav Petkov wrote: > On Tue, Apr 02, 2013 at 06:33:18PM +0200, Joerg Roedel wrote: > > Okay, in theory I could implement a feedback loop between timer-setup > > and intremap code and try fixups until it works. But that seems not to > > be worth it to work around a buggy BIOS. > > Yeah, same here. It's not like we really need intremap to work - we're > only trying to fix the annoying error message currently. :-) Right :) > > What I actually thought about was providing an IVRS-override on the > > kernel command line. So that you can specify the IOAPIC_ID->DEVID > > mapping there and make it work this way. What do you think? > > I guess that is workable. I can imagine people wanting this if they want > to do the intremap thing on such b0rked BIOSen. So how do I specify this > IOAPIC_ID->DEVID mapping on the cmdline exactly? Don't know yet, probably something like ivrs_ioapic[]=0:14.2 and ivrs_hpet[]=0:14.2. But not entierly sure yet, at least parsing shouldn't require lex and yacc ;) > > Yeah, that's my fear too. So we leave it better as it is... > > Hohumm. Thus shall it be! Joerg