Re: [PATCH V3] iommu/amd: Add logic to decode AMD IOMMU event flag

2013-04-09 Thread Joerg Roedel
On Tue, Apr 09, 2013 at 10:12:13AM -0500, Suthikulpanit, Suravee wrote: > These messages are not "high volumn". What I am trying to achieve > is for users to be able to just send the errors to developers once > they are encountered without having to tell them to go back and > reboot with the

Re: [PATCH V3] iommu/amd: Add logic to decode AMD IOMMU event flag

2013-04-09 Thread Suravee Suthikulanit
On 4/9/2013 4:41 AM, Joerg Roedel wrote: On Tue, Apr 02, 2013 at 07:06:50PM -0500, Suthikulpanit, Suravee wrote: From: Suravee Suthikulpanit Add logic to decode AMD IOMMU event flag based on information from AMD IOMMU specification. This should simplify debugging IOMMU errors. Also, dump

Re: [PATCH V3] iommu/amd: Add logic to decode AMD IOMMU event flag

2013-04-09 Thread Joerg Roedel
On Tue, Apr 02, 2013 at 07:06:50PM -0500, Suthikulpanit, Suravee wrote: > From: Suravee Suthikulpanit > > Add logic to decode AMD IOMMU event flag based on information from AMD IOMMU > specification. > This should simplify debugging IOMMU errors. Also, dump DTE information in > additional

Re: [PATCH V3] iommu/amd: Add logic to decode AMD IOMMU event flag

2013-04-09 Thread Joerg Roedel
On Tue, Apr 02, 2013 at 07:06:50PM -0500, Suthikulpanit, Suravee wrote: From: Suravee Suthikulpanit suravee.suthikulpa...@amd.com Add logic to decode AMD IOMMU event flag based on information from AMD IOMMU specification. This should simplify debugging IOMMU errors. Also, dump DTE

Re: [PATCH V3] iommu/amd: Add logic to decode AMD IOMMU event flag

2013-04-09 Thread Suravee Suthikulanit
On 4/9/2013 4:41 AM, Joerg Roedel wrote: On Tue, Apr 02, 2013 at 07:06:50PM -0500, Suthikulpanit, Suravee wrote: From: Suravee Suthikulpanit suravee.suthikulpa...@amd.com Add logic to decode AMD IOMMU event flag based on information from AMD IOMMU specification. This should simplify debugging

Re: [PATCH V3] iommu/amd: Add logic to decode AMD IOMMU event flag

2013-04-09 Thread Joerg Roedel
On Tue, Apr 09, 2013 at 10:12:13AM -0500, Suthikulpanit, Suravee wrote: These messages are not high volumn. What I am trying to achieve is for users to be able to just send the errors to developers once they are encountered without having to tell them to go back and reboot with the

Re: [PATCH V3] iommu/amd: Add logic to decode AMD IOMMU event flag

2013-04-08 Thread Borislav Petkov
On Mon, Apr 08, 2013 at 10:43:45AM -0500, Suravee Suthikulanit wrote: > On 4/8/2013 9:50 AM, Borislav Petkov wrote: > >Independent from Joerg's feedback on this, I have only one question: > >you're not seriously considering on dumping this "Note:..." line above > >on*every* IO-PF, right? > If you

Re: [PATCH V3] iommu/amd: Add logic to decode AMD IOMMU event flag

2013-04-08 Thread Borislav Petkov
On Mon, Apr 08, 2013 at 02:33:32PM +, Suthikulpanit, Suravee wrote: > Joerg, > > Do you have any more feedback about this patch? > > Thanks, > > Suravee > > From: suravee.suthikulpa...@amd.com [suravee.suthikulpa...@amd.com] > Sent: Tuesday, April

RE: [PATCH V3] iommu/amd: Add logic to decode AMD IOMMU event flag

2013-04-08 Thread Suthikulpanit, Suravee
Joerg, Do you have any more feedback about this patch? Thanks, Suravee From: suravee.suthikulpa...@amd.com [suravee.suthikulpa...@amd.com] Sent: Tuesday, April 02, 2013 7:06 PM To: io...@lists.linux-foundation.org; j...@8bytes.org Cc:

RE: [PATCH V3] iommu/amd: Add logic to decode AMD IOMMU event flag

2013-04-08 Thread Suthikulpanit, Suravee
Joerg, Do you have any more feedback about this patch? Thanks, Suravee From: suravee.suthikulpa...@amd.com [suravee.suthikulpa...@amd.com] Sent: Tuesday, April 02, 2013 7:06 PM To: io...@lists.linux-foundation.org; j...@8bytes.org Cc:

Re: [PATCH V3] iommu/amd: Add logic to decode AMD IOMMU event flag

2013-04-08 Thread Borislav Petkov
On Mon, Apr 08, 2013 at 02:33:32PM +, Suthikulpanit, Suravee wrote: Joerg, Do you have any more feedback about this patch? Thanks, Suravee From: suravee.suthikulpa...@amd.com [suravee.suthikulpa...@amd.com] Sent: Tuesday, April 02, 2013

Re: [PATCH V3] iommu/amd: Add logic to decode AMD IOMMU event flag

2013-04-08 Thread Borislav Petkov
On Mon, Apr 08, 2013 at 10:43:45AM -0500, Suravee Suthikulanit wrote: On 4/8/2013 9:50 AM, Borislav Petkov wrote: Independent from Joerg's feedback on this, I have only one question: you're not seriously considering on dumping this Note:... line above on*every* IO-PF, right? If you think