All of lore.kernel.org
 help / color / mirror / Atom feed
From: Suravee Suthikulanit <suravee.suthikulpanit-5C7GfCeVMHo@public.gmane.org>
To: Don Dutile <ddutile-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
Cc: "iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org"
	<iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org>,
	"linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
Subject: Re: RFC: IOMMU/AMD: Error Handling
Date: Tue, 30 Apr 2013 09:56:22 -0500	[thread overview]
Message-ID: <517FDB96.7060602@amd.com> (raw)
In-Reply-To: <517EE940.8010005-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>

On 4/29/2013 4:42 PM, Don Dutile wrote:
> On 04/29/2013 04:34 PM, Duran, Leo wrote:
>> I'm wondering if resetting the IOMMU at init-time (once) would clear 
>> any BIOS induced noise.
>> Leo
>>
> Well, depends what you mean by 'reset'....
> (a) setting it up for OS use is effectively a reset, but doesn't 
> quiesce a device
>      doing dma reads of a (bios-setup) queue.  then the noisy messages 
> begin
> (b) disable the iommu, and then the dma just occurs... and bad for 
> writes, potentially.
>
> Similar issue is being reported & worked for kdump, where device are 
> still
> doing DMA while the system is trying to 'reset' to the kexec'd kernel, 
> and
> take a crash dump.
>
> Solution: stop devices from doing dma... but some you _want_ enabled 
> throughout...
>           like keyboard & mouse via usb controller, so you get to pick 
> os from
>           grub...  not so for kexec...
>
> so, again, for isolation faults.... let the hw do its job -- isolate
> and throttle/silence the fault messages on a per-device, time-duration 
> heuristic
> so the system can get through boot-up where enough OS is init'd 
> (drivers started)
> to stop the temporary noise.
This sounds more like issue with the order of how things are initialized 
in the system.
If so, could we separate the code which enabling of IOMMU error 
logging/handling and
delay it until we are certain that systems are stable?

Suravee

WARNING: multiple messages have this Message-ID (diff)
From: Suravee Suthikulanit <suravee.suthikulpanit@amd.com>
To: Don Dutile <ddutile@redhat.com>
Cc: "Duran, Leo" <leo.duran@amd.com>,
	"iommu@lists.linux-foundation.org"
	<iommu@lists.linux-foundation.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: RFC: IOMMU/AMD: Error Handling
Date: Tue, 30 Apr 2013 09:56:22 -0500	[thread overview]
Message-ID: <517FDB96.7060602@amd.com> (raw)
In-Reply-To: <517EE940.8010005@redhat.com>

On 4/29/2013 4:42 PM, Don Dutile wrote:
> On 04/29/2013 04:34 PM, Duran, Leo wrote:
>> I'm wondering if resetting the IOMMU at init-time (once) would clear 
>> any BIOS induced noise.
>> Leo
>>
> Well, depends what you mean by 'reset'....
> (a) setting it up for OS use is effectively a reset, but doesn't 
> quiesce a device
>      doing dma reads of a (bios-setup) queue.  then the noisy messages 
> begin
> (b) disable the iommu, and then the dma just occurs... and bad for 
> writes, potentially.
>
> Similar issue is being reported & worked for kdump, where device are 
> still
> doing DMA while the system is trying to 'reset' to the kexec'd kernel, 
> and
> take a crash dump.
>
> Solution: stop devices from doing dma... but some you _want_ enabled 
> throughout...
>           like keyboard & mouse via usb controller, so you get to pick 
> os from
>           grub...  not so for kexec...
>
> so, again, for isolation faults.... let the hw do its job -- isolate
> and throttle/silence the fault messages on a per-device, time-duration 
> heuristic
> so the system can get through boot-up where enough OS is init'd 
> (drivers started)
> to stop the temporary noise.
This sounds more like issue with the order of how things are initialized 
in the system.
If so, could we separate the code which enabling of IOMMU error 
logging/handling and
delay it until we are certain that systems are stable?

Suravee


  parent reply	other threads:[~2013-04-30 14:56 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-29 19:45 RFC: IOMMU/AMD: Error Handling Suravee Suthikulanit
2013-04-29 19:45 ` Suravee Suthikulanit
     [not found] ` <517ECDDA.3000606-5C7GfCeVMHo@public.gmane.org>
2013-04-29 20:10   ` Don Dutile
2013-04-29 20:10     ` Don Dutile
2013-04-29 20:34     ` Duran, Leo
     [not found]       ` <BA42942F2D0DED45AFB0A6216D1E951D44CBE1F9-Vo+W8YXarrgxlywnonMhLEEOCMrvLtNR@public.gmane.org>
2013-04-29 21:42         ` Don Dutile
2013-04-29 21:42           ` Don Dutile
2013-04-29 22:31           ` Duran, Leo
     [not found]           ` <517EE940.8010005-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2013-04-30 14:56             ` Suravee Suthikulanit [this message]
2013-04-30 14:56               ` Suravee Suthikulanit
     [not found]               ` <517FDB96.7060602-5C7GfCeVMHo@public.gmane.org>
2013-04-30 15:09                 ` Don Dutile
2013-04-30 15:09                   ` Don Dutile
2013-04-30 15:21                 ` Joerg Roedel
2013-04-30 15:21                   ` Joerg Roedel
     [not found]     ` <517ED3A9.2050508-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2013-04-30 14:49       ` Suravee Suthikulanit
2013-04-30 14:49         ` Suravee Suthikulanit
     [not found]         ` <517FD9E8.8070802-5C7GfCeVMHo@public.gmane.org>
2013-04-30 15:06           ` Don Dutile
2013-04-30 15:06             ` Don Dutile
     [not found]             ` <517FDDF6.8090707-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2013-04-30 16:02               ` Alex Williamson
2013-04-30 16:02                 ` Alex Williamson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=517FDB96.7060602@amd.com \
    --to=suravee.suthikulpanit-5c7gfcevmho@public.gmane.org \
    --cc=ddutile-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.