Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2020-10-19 Thread boris . ostrovsky
On 10/19/20 6:43 PM, Rich Persaud wrote: > On Oct 19, 2020, at 11:52, Håkon Alstadheim wrote: >>  >> Den 19.10.2020 17:16, skrev Håkon Alstadheim: >>> Den 19.10.2020 13:00, skrev George Dunlap: > On Jan 31, 2020, at 3:33 PM, Wei Liu wrote: > > On Fri, Jan 17, 2020 at 02:13:04PM

Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2020-10-19 Thread Rich Persaud
On Oct 19, 2020, at 11:52, Håkon Alstadheim wrote: > >  > Den 19.10.2020 17:16, skrev Håkon Alstadheim: >> Den 19.10.2020 13:00, skrev George Dunlap: >>> On Jan 31, 2020, at 3:33 PM, Wei Liu wrote: On Fri, Jan 17, 2020 at 02:13:04PM -0500, Rich Persaud wrote: > On Aug 26,

Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2020-10-19 Thread Håkon Alstadheim
Den 19.10.2020 17:16, skrev Håkon Alstadheim: Den 19.10.2020 13:00, skrev George Dunlap: On Jan 31, 2020, at 3:33 PM, Wei Liu wrote: On Fri, Jan 17, 2020 at 02:13:04PM -0500, Rich Persaud wrote: On Aug 26, 2019, at 17:08, Pasi Kärkkäinen wrote: Hi, On Mon, Oct 08, 2018 at 10:32:45AM

Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2020-10-19 Thread Håkon Alstadheim
Den 19.10.2020 13:00, skrev George Dunlap: On Jan 31, 2020, at 3:33 PM, Wei Liu wrote: On Fri, Jan 17, 2020 at 02:13:04PM -0500, Rich Persaud wrote: On Aug 26, 2019, at 17:08, Pasi Kärkkäinen wrote: Hi, On Mon, Oct 08, 2018 at 10:32:45AM -0400, Boris Ostrovsky wrote: On 10/3/18 11:51

Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2020-10-19 Thread George Dunlap
> On Jan 31, 2020, at 3:33 PM, Wei Liu wrote: > > On Fri, Jan 17, 2020 at 02:13:04PM -0500, Rich Persaud wrote: >> On Aug 26, 2019, at 17:08, Pasi Kärkkäinen wrote: >>> Hi, >>> On Mon, Oct 08, 2018 at 10:32:45AM -0400, Boris Ostrovsky wrote: > On 10/3/18 11:51 AM, Pasi Kärkkäinen

Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2020-01-31 Thread Wei Liu
On Fri, Jan 17, 2020 at 02:13:04PM -0500, Rich Persaud wrote: > On Aug 26, 2019, at 17:08, Pasi Kärkkäinen wrote: > > Hi, > > > >> On Mon, Oct 08, 2018 at 10:32:45AM -0400, Boris Ostrovsky wrote: > >>> On 10/3/18 11:51 AM, Pasi Kärkkäinen wrote: > >>> On Wed, Sep 19, 2018 at 11:05:26AM +0200,

Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2020-01-23 Thread Pasi Kärkkäinen
Hi, On Fri, Jan 17, 2020 at 02:13:04PM -0500, Rich Persaud wrote: >On Aug 26, 2019, at 17:08, Pasi Kärkkäinen wrote: > > Hi, > On Mon, Oct 08, 2018 at 10:32:45AM -0400, Boris Ostrovsky wrote: > >On 10/3/18 11:51 AM, Pasi Kärkkäinen wrote: > > On Wed, Sep 19,

Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2020-01-17 Thread Rich Persaud
On Aug 26, 2019, at 17:08, Pasi Kärkkäinen wrote: > Hi, > >> On Mon, Oct 08, 2018 at 10:32:45AM -0400, Boris Ostrovsky wrote: >>> On 10/3/18 11:51 AM, Pasi Kärkkäinen wrote: >>> On Wed, Sep 19, 2018 at 11:05:26AM +0200, Roger Pau Monné wrote: On Tue, Sep 18, 2018 at 02:09:53PM -0400, Boris

Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2019-08-26 Thread Pasi Kärkkäinen
Hi, On Mon, Oct 08, 2018 at 10:32:45AM -0400, Boris Ostrovsky wrote: > On 10/3/18 11:51 AM, Pasi Kärkkäinen wrote: > > On Wed, Sep 19, 2018 at 11:05:26AM +0200, Roger Pau Monné wrote: > >> On Tue, Sep 18, 2018 at 02:09:53PM -0400, Boris Ostrovsky wrote: > >>> On 9/18/18 5:32 AM, George Dunlap

Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2018-10-29 Thread Pasi Kärkkäinen
Hi, On Tue, Oct 23, 2018 at 08:40:29PM +0200, Håkon Alstadheim wrote: > > > Den 08. okt. 2018 16:32, skrev Boris Ostrovsky: > > > > Are these two patches still needed? ISTR they were  written originally > > to deal with guest trying to use device that was previously assigned to > > another

Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2018-10-08 Thread Boris Ostrovsky
On 10/3/18 11:51 AM, Pasi Kärkkäinen wrote: > On Wed, Sep 19, 2018 at 11:05:26AM +0200, Roger Pau Monné wrote: >> On Tue, Sep 18, 2018 at 02:09:53PM -0400, Boris Ostrovsky wrote: >>> On 9/18/18 5:32 AM, George Dunlap wrote: > On Sep 18, 2018, at 8:15 AM, Pasi Kärkkäinen wrote: > > Hi,

Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2018-10-03 Thread Pasi Kärkkäinen
On Wed, Sep 19, 2018 at 11:05:26AM +0200, Roger Pau Monné wrote: > On Tue, Sep 18, 2018 at 02:09:53PM -0400, Boris Ostrovsky wrote: > > On 9/18/18 5:32 AM, George Dunlap wrote: > > > > > >> On Sep 18, 2018, at 8:15 AM, Pasi Kärkkäinen wrote: > > >> > > >> Hi, > > >> > > >> On Mon, Sep 17, 2018 at

Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2018-09-19 Thread Roger Pau Monné
On Tue, Sep 18, 2018 at 02:09:53PM -0400, Boris Ostrovsky wrote: > On 9/18/18 5:32 AM, George Dunlap wrote: > > > >> On Sep 18, 2018, at 8:15 AM, Pasi Kärkkäinen wrote: > >> > >> Hi, > >> > >> On Mon, Sep 17, 2018 at 02:06:02PM -0400, Boris Ostrovsky wrote: > >>> What about the toolstack changes?

Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2018-09-18 Thread Boris Ostrovsky
On 9/18/18 5:32 AM, George Dunlap wrote: > >> On Sep 18, 2018, at 8:15 AM, Pasi Kärkkäinen wrote: >> >> Hi, >> >> On Mon, Sep 17, 2018 at 02:06:02PM -0400, Boris Ostrovsky wrote: >>> What about the toolstack changes? Have they been accepted? I vaguely >>> recall there was a discussion about those

Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2018-09-18 Thread George Dunlap
> On Sep 18, 2018, at 8:15 AM, Pasi Kärkkäinen wrote: > > Hi, > > On Mon, Sep 17, 2018 at 02:06:02PM -0400, Boris Ostrovsky wrote: >> On 9/16/18 7:43 AM, Pasi Kärkkäinen wrote: >>> Hi, >>> >>> On Mon, Dec 18, 2017 at 12:32:11PM -0500, Boris Ostrovsky wrote: On 12/18/2017 02:36 AM, Jan

Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2018-09-18 Thread Pasi Kärkkäinen
Hi, On Mon, Sep 17, 2018 at 02:06:02PM -0400, Boris Ostrovsky wrote: > On 9/16/18 7:43 AM, Pasi Kärkkäinen wrote: > > Hi, > > > > On Mon, Dec 18, 2017 at 12:32:11PM -0500, Boris Ostrovsky wrote: > >> On 12/18/2017 02:36 AM, Jan Beulich wrote: > >> On 15.12.17 at 20:52, wrote: > >>>

Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2018-09-17 Thread Boris Ostrovsky
On 9/16/18 7:43 AM, Pasi Kärkkäinen wrote: > Hi, > > On Mon, Dec 18, 2017 at 12:32:11PM -0500, Boris Ostrovsky wrote: >> On 12/18/2017 02:36 AM, Jan Beulich wrote: >> On 15.12.17 at 20:52, wrote: >>> +static int pcistub_device_reset(struct pci_dev *dev) >>> +{ >>> + struct

Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2018-09-16 Thread Pasi Kärkkäinen
Hi, On Mon, Dec 18, 2017 at 12:32:11PM -0500, Boris Ostrovsky wrote: > On 12/18/2017 02:36 AM, Jan Beulich wrote: > On 15.12.17 at 20:52, wrote: > > +static int pcistub_device_reset(struct pci_dev *dev) > > +{ > > + struct xen_pcibk_dev_data *dev_data; > > + bool

Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2017-12-18 Thread Boris Ostrovsky
On 12/18/2017 02:36 AM, Jan Beulich wrote: On 15.12.17 at 20:52, wrote: > +static int pcistub_device_reset(struct pci_dev *dev) > +{ > + struct xen_pcibk_dev_data *dev_data; > + bool slot = false, bus = false; > + struct pcistub_args arg = {};

Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2017-12-17 Thread Jan Beulich
>>> On 15.12.17 at 20:52, wrote: +static int pcistub_device_reset(struct pci_dev *dev) +{ + struct xen_pcibk_dev_data *dev_data; + bool slot = false, bus = false; + struct pcistub_args arg = {}; + + if (!dev) + return

Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2017-12-15 Thread Govinda Tatti
Jan, One quick update on pcie_flr() specific implementation. Please see below. +static int pcistub_device_reset(struct pci_dev *dev) +{ + struct xen_pcibk_dev_data *dev_data; + bool slot = false, bus = false; + struct pcistub_args arg = {}; + + if (!dev) +

Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2017-12-12 Thread Govinda Tatti
On 12/12/2017 9:01 AM, Jan Beulich wrote: On 12.12.17 at 15:48, wrote: Thanks Jan for your review comments. Please see below for my comments. First of all - can you please do something about your reply style? HTML mail should be avoided. You'll see that the (plain

Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2017-12-12 Thread Jan Beulich
>>> On 12.12.17 at 15:48, wrote: > Thanks Jan for your review comments. Please see below for my comments. First of all - can you please do something about your reply style? HTML mail should be avoided. You'll see that the (plain text) reply as a result is rather hard to

Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2017-12-12 Thread Govinda Tatti
Thanks Jan for your review comments. Please see below for my comments. On 12/8/2017 3:34 AM, Jan Beulich wrote: On 07.12.17 at 23:21, wrote: Due to the

Re: [Xen-devel] [PATCH V3 2/2] Xen/PCIback: Implement PCI flr/slot/bus reset with 'reset' SysFS attribute

2017-12-08 Thread Jan Beulich
>>> On 07.12.17 at 23:21, wrote: > Due to the complexity with the PCI lock we cannot do the reset when a > device is bound ('echo $BDF > bind') or when unbound ('echo $BDF > unbind') > as the pci_[slot|bus]_reset also takes the same lock resulting in a > dead-lock. It