Re: [PATCH 1/1] virtio/s390: fix vritio-ccw device teardown

2021-09-21 Thread Halil Pasic
On Tue, 21 Sep 2021 15:31:03 +0200 Vineeth Vijayan wrote: > On Tue, 2021-09-21 at 05:25 +0200, Halil Pasic wrote: > > On Mon, 20 Sep 2021 12:07:23 +0200 > > Cornelia Huck wrote: > > > > > On Mon, Sep 20 2021, Vineeth Vijayan wrote: > > > > > > > On Mon, 2021-09-20 at 00:39 +0200, Halil

Re: [PATCH 1/1] virtio/s390: fix vritio-ccw device teardown

2021-09-21 Thread Cornelia Huck
On Tue, Sep 21 2021, Halil Pasic wrote: > On Mon, 20 Sep 2021 12:07:23 +0200 > Cornelia Huck wrote: > >> On Mon, Sep 20 2021, Vineeth Vijayan wrote: >> >> > On Mon, 2021-09-20 at 00:39 +0200, Halil Pasic wrote: >> >> On Fri, 17 Sep 2021 10:40:20 +0200 >> >> Cornelia Huck wrote: >> >> >>

Re: [PATCH 1/1] virtio/s390: fix vritio-ccw device teardown

2021-09-20 Thread Halil Pasic
On Mon, 20 Sep 2021 12:07:23 +0200 Cornelia Huck wrote: > On Mon, Sep 20 2021, Vineeth Vijayan wrote: > > > On Mon, 2021-09-20 at 00:39 +0200, Halil Pasic wrote: > >> On Fri, 17 Sep 2021 10:40:20 +0200 > >> Cornelia Huck wrote: > >> > > ...snip... > >> > > > >> > > Thanks, if I find

Re: [PATCH 1/1] virtio/s390: fix vritio-ccw device teardown

2021-09-20 Thread Halil Pasic
On Mon, 20 Sep 2021 12:30:39 +0200 Cornelia Huck wrote: > On Mon, Sep 20 2021, Halil Pasic wrote: [..] > > Basically, the vcdev is supposed to be around while the ccw device is > online (with a tail end until references have been given up, of course.) > It embeds a virtio device that has the

Re: [PATCH 1/1] virtio/s390: fix vritio-ccw device teardown

2021-09-20 Thread Cornelia Huck
On Mon, Sep 20 2021, Halil Pasic wrote: > On Fri, 17 Sep 2021 10:40:20 +0200 > Cornelia Huck wrote: > >> On Thu, Sep 16 2021, Halil Pasic wrote: >> >> > On Thu, 16 Sep 2021 10:59:15 +0200 >> > Cornelia Huck wrote: >> > >> >> > Since commit 48720ba56891 ("virtio/s390: use DMA memory for ccw

Re: [PATCH 1/1] virtio/s390: fix vritio-ccw device teardown

2021-09-20 Thread Cornelia Huck
On Mon, Sep 20 2021, Vineeth Vijayan wrote: > On Mon, 2021-09-20 at 00:39 +0200, Halil Pasic wrote: >> On Fri, 17 Sep 2021 10:40:20 +0200 >> Cornelia Huck wrote: >> > ...snip... >> > > >> > > Thanks, if I find time for it, I will try to understand this >> > > better and >> > > come back with

Re: [PATCH 1/1] virtio/s390: fix vritio-ccw device teardown

2021-09-19 Thread Halil Pasic
On Fri, 17 Sep 2021 10:40:20 +0200 Cornelia Huck wrote: > On Thu, Sep 16 2021, Halil Pasic wrote: > > > On Thu, 16 Sep 2021 10:59:15 +0200 > > Cornelia Huck wrote: > > > >> > Since commit 48720ba56891 ("virtio/s390: use DMA memory for ccw I/O and > >> > classic notifiers") we were supposed

Re: [PATCH 1/1] virtio/s390: fix vritio-ccw device teardown

2021-09-17 Thread Cornelia Huck
On Thu, Sep 16 2021, Halil Pasic wrote: > On Thu, 16 Sep 2021 10:59:15 +0200 > Cornelia Huck wrote: > >> > Since commit 48720ba56891 ("virtio/s390: use DMA memory for ccw I/O and >> > classic notifiers") we were supposed to make sure that >> > virtio_ccw_release_dev() completes before the ccw

Re: [PATCH 1/1] virtio/s390: fix vritio-ccw device teardown

2021-09-16 Thread Halil Pasic
On Thu, 16 Sep 2021 10:59:15 +0200 Cornelia Huck wrote: > > Since commit 48720ba56891 ("virtio/s390: use DMA memory for ccw I/O and > > classic notifiers") we were supposed to make sure that > > virtio_ccw_release_dev() completes before the ccw device, and the > > attached dma pool are torn

Re: [PATCH 1/1] virtio/s390: fix vritio-ccw device teardown

2021-09-16 Thread Cornelia Huck
On Wed, Sep 15 2021, Halil Pasic wrote: > Since commit 48720ba56891 ("virtio/s390: use DMA memory for ccw I/O and > classic notifiers") we were supposed to make sure that > virtio_ccw_release_dev() completes before the ccw device, and the > attached dma pool are torn down, but unfortunately we

Re: [PATCH 1/1] virtio/s390: fix vritio-ccw device teardown

2021-09-15 Thread Halil Pasic
s/vritio/virtio/ (subject) [..] On Wed, 15 Sep 2021 23:57:42 +0200 Halil Pasic wrote: > Since commit 48720ba56891 ("virtio/s390: use DMA memory for ccw I/O and > classic notifiers") we were supposed to make sure that > virtio_ccw_release_dev() completes before the ccw device, and the >

[PATCH 1/1] virtio/s390: fix vritio-ccw device teardown

2021-09-15 Thread Halil Pasic
Since commit 48720ba56891 ("virtio/s390: use DMA memory for ccw I/O and classic notifiers") we were supposed to make sure that virtio_ccw_release_dev() completes before the ccw device, and the attached dma pool are torn down, but unfortunately we did not. Before that commit it used to be OK to