Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1617415

Re: [PATCH v6] vfio error recovery: kernel support

Path csiph.com!weretis.net!feeder4.news.weretis.net!news.mixmin.net!aioe.org!bofh.it!news.nic.it!robomod
From Alex Williamson <alex.williamson@redhat.com>
Newsgroups linux.kernel
Subject Re: [PATCH v6] vfio error recovery: kernel support
Date Thu, 06 Apr 2017 01:00:02 +0200
Message-ID <tt24a-dB-13@gated-at.bofh.it> (permalink)
References <tpZFw-8kZ-25@gated-at.bofh.it> <tq20G-1Mi-5@gated-at.bofh.it> <tq9lv-78Q-17@gated-at.bofh.it> <tqc01-sx-3@gated-at.bofh.it> <tqMGe-1NY-11@gated-at.bofh.it> <tqMPU-1Sb-19@gated-at.bofh.it> <tsONA-kg-17@gated-at.bofh.it> <tsYWB-6NO-5@gated-at.bofh.it> <tt186-81R-15@gated-at.bofh.it> <tt1rs-8o8-17@gated-at.bofh.it> <tt1KN-8w3-7@gated-at.bofh.it>
Dmarc-Filter OpenDMARC Filter v1.3.2 mx1.redhat.com 43049C054C58
Authentication-Results ext-mx08.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com
Authentication-Results ext-mx08.extmail.prod.ext.phx2.redhat.com; spf=pass smtp.mailfrom=alex.williamson@redhat.com
Dkim-Filter OpenDKIM Filter v2.11.0 mx1.redhat.com 43049C054C58
MIME-Version 1.0
Content-Type text/plain; charset=US-ASCII
Content-Transfer-Encoding 7bit
X-Scanned-By MIMEDefang 2.79 on 10.5.11.14
X-Greylist Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.32]); Wed, 05 Apr 2017 22:56:18 +0000 (UTC)
Sender robomod@news.nic.it
List-ID <linux-kernel.vger.kernel.org>
X-Mailing-List linux-kernel@vger.kernel.org
Approved robomod@news.nic.it
Lines 42
Organization linux.* mail to news gateway
X-Original-Cc Cao jin <caoj.fnst@cn.fujitsu.com>, linux-kernel@vger.kernel.org, kvm@vger.kernel.org, qemu-devel@nongnu.org, izumi.taku@jp.fujitsu.com
X-Original-Date Wed, 5 Apr 2017 16:56:15 -0600
X-Original-Message-ID <20170405165615.448bcead@t450s.home>
X-Original-References <58DA6954.2000601@cn.fujitsu.com> <20170328101233.74f50a92@t450s.home> <20170329000148.GA18849@redhat.com> <20170328205513.21b97381@t450s.home> <20170330205823-mutt-send-email-mst@kernel.org> <20170330121652.2ac8fa62@t450s.home> <58E4B0C9.50109@cn.fujitsu.com> <20170405133822.76cda620@t450s.home> <20170406004845-mutt-send-email-mst@kernel.org> <20170405161910.26fee7d1@t450s.home> <20170406013534-mutt-send-email-mst@kernel.org>
X-Original-Sender linux-kernel-owner@vger.kernel.org
Xref csiph.com linux.kernel:1617415

Show key headers only | View raw


On Thu, 6 Apr 2017 01:36:31 +0300
"Michael S. Tsirkin" <mst@redhat.com> wrote:

> On Wed, Apr 05, 2017 at 04:19:10PM -0600, Alex Williamson wrote:
> > On Thu, 6 Apr 2017 00:50:22 +0300
> > "Michael S. Tsirkin" <mst@redhat.com> wrote:
> >   
> > > On Wed, Apr 05, 2017 at 01:38:22PM -0600, Alex Williamson wrote:  
> > > > The previous intention of trying to handle all sorts of AER faults
> > > > clearly had more value, though even there the implementation and
> > > > configuration requirements restricted the practicality.  For instance
> > > > is AER support actually useful to a customer if it requires all ports
> > > > of a multifunction device assigned to the VM?  This seems more like a
> > > > feature targeting whole system partitioning rather than general VM
> > > > device assignment use cases.  Maybe that's ok, but it should be a clear
> > > > design decision.    
> > > 
> > > Alex, what kind of testing do you expect to be necessary?
> > > Would you say testing on real hardware and making it trigger
> > > AER errors is a requirement?  
> > 
> > Testing various fatal, non-fatal, and corrected errors with aer-inject,
> > especially in multfunction configurations (where more than one port
> > is actually usable) would certainly be required.  If we have cases where
> > the driver for a companion function can escalate a non-fatal error to a
> > bus reset, that should be tested, even if it requires temporary hacks to
> > the host driver for the companion function to trigger that case.  AER
> > handling is not something that the typical user is going to experience,
> > so it should to be thoroughly tested to make sure it works when needed
> > or there's little point to doing it at all.  Thanks,
> > 
> > Alex  
> 
> Some things can be tested within a VM. What would you
> say would be sufficient on a VM and what has to be
> tested on bare metal?

Testing on a VM could be interesting for development, but I'd expect
bare metal for validation, no offense.  Bus reset timing can be
different, error propagation can be different, etc.  Thanks,

Alex

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

Re: [PATCH v6] vfio error recovery: kernel support Cao jin <caoj.fnst@cn.fujitsu.com> - 2017-03-28 15:50 +0200
  Re: [PATCH v6] vfio error recovery: kernel support Alex Williamson <alex.williamson@redhat.com> - 2017-03-28 18:20 +0200
    Re: [PATCH v6] vfio error recovery: kernel support "Michael S. Tsirkin" <mst@redhat.com> - 2017-03-29 02:10 +0200
      Re: [PATCH v6] vfio error recovery: kernel support Alex Williamson <alex.williamson@redhat.com> - 2017-03-29 05:00 +0200
        Re: [PATCH v6] vfio error recovery: kernel support "Michael S. Tsirkin" <mst@redhat.com> - 2017-03-30 20:10 +0200
          Re: [PATCH v6] vfio error recovery: kernel support Alex Williamson <alex.williamson@redhat.com> - 2017-03-30 20:20 +0200
            Re: [PATCH v6] vfio error recovery: kernel support Cao jin <caoj.fnst@cn.fujitsu.com> - 2017-04-05 10:50 +0200
              Re: [PATCH v6] vfio error recovery: kernel support Alex Williamson <alex.williamson@redhat.com> - 2017-04-05 21:40 +0200
                Re: [PATCH v6] vfio error recovery: kernel support "Michael S. Tsirkin" <mst@redhat.com> - 2017-04-06 00:00 +0200
                Re: [PATCH v6] vfio error recovery: kernel support Alex Williamson <alex.williamson@redhat.com> - 2017-04-06 00:20 +0200
                Re: [PATCH v6] vfio error recovery: kernel support "Michael S. Tsirkin" <mst@redhat.com> - 2017-04-06 00:40 +0200
                Re: [PATCH v6] vfio error recovery: kernel support Alex Williamson <alex.williamson@redhat.com> - 2017-04-06 01:00 +0200
                Re: [PATCH v6] vfio error recovery: kernel support Cao jin <caoj.fnst@cn.fujitsu.com> - 2017-04-06 11:10 +0200
                Re: [PATCH v6] vfio error recovery: kernel support Alex Williamson <alex.williamson@redhat.com> - 2017-04-06 17:40 +0200
                Re: [PATCH v6] vfio error recovery: kernel support Cao jin <caoj.fnst@cn.fujitsu.com> - 2017-04-06 11:40 +0200
              Re: [PATCH v6] vfio error recovery: kernel support "Michael S. Tsirkin" <mst@redhat.com> - 2017-04-06 00:00 +0200
                Re: [PATCH v6] vfio error recovery: kernel support Cao jin <caoj.fnst@cn.fujitsu.com> - 2017-04-06 11:40 +0200
                Re: [PATCH v6] vfio error recovery: kernel support Alex Williamson <alex.williamson@redhat.com> - 2017-04-06 17:40 +0200

csiph-web