Re: [PATCH 10/10] virtio/s390: make airq summary indicators DMA

2019-05-19 Thread Michael Mueller
On 08.05.19 17:11, Pierre Morel wrote: On 26/04/2019 20:32, Halil Pasic wrote: Hypervisor needs to interact with the summary indicators, so these need to be DMA memory as well (at least for protected virtualization guests). Signed-off-by: Halil Pasic ---   drivers/s390/virtio/virtio_ccw.c |

Re: [PATCH 10/10] virtio/s390: make airq summary indicators DMA

2019-05-19 Thread Michael Mueller
On 13.05.19 14:20, Cornelia Huck wrote: On Fri, 26 Apr 2019 20:32:45 +0200 Halil Pasic wrote: Hypervisor needs to interact with the summary indicators, so these need to be DMA memory as well (at least for protected virtualization guests). Signed-off-by: Halil Pasic ---

Re: [PATCH 10/10] virtio/s390: make airq summary indicators DMA

2019-05-19 Thread Pierre Morel
On 26/04/2019 20:32, Halil Pasic wrote: Hypervisor needs to interact with the summary indicators, so these need to be DMA memory as well (at least for protected virtualization guests). Signed-off-by: Halil Pasic --- drivers/s390/virtio/virtio_ccw.c | 24 +--- 1 file

Re: [PATCH 10/10] virtio/s390: make airq summary indicators DMA

2019-05-15 Thread Halil Pasic
On Wed, 15 May 2019 15:33:02 +0200 Michael Mueller wrote: > >> @@ -273,8 +281,9 @@ static unsigned long get_airq_indicator(struct > >> virtqueue *vqs[], int nvqs, > >>   unsigned long bit, flags; > >>   for (i = 0; i < MAX_AIRQ_AREAS && !indicator_addr; i++) { > >> +    /* TODO:

Re: [PATCH 10/10] virtio/s390: make airq summary indicators DMA

2019-05-15 Thread Halil Pasic
On Wed, 15 May 2019 15:43:23 +0200 Michael Mueller wrote: > > Hm, where is airq_areas_lock defined? If it was introduced in one of > > the previous patches, I have missed it. > > There is no airq_areas_lock defined currently. My assumption is that > this will be used in context with the

Re: [PATCH 10/10] virtio/s390: make airq summary indicators DMA

2019-05-15 Thread Cornelia Huck
On Wed, 15 May 2019 15:43:23 +0200 Michael Mueller wrote: > On 13.05.19 14:20, Cornelia Huck wrote: > > On Fri, 26 Apr 2019 20:32:45 +0200 > > Halil Pasic wrote: > > > >> Hypervisor needs to interact with the summary indicators, so these > >> need to be DMA memory as well (at least for

Re: [PATCH 10/10] virtio/s390: make airq summary indicators DMA

2019-05-13 Thread Cornelia Huck
On Fri, 26 Apr 2019 20:32:45 +0200 Halil Pasic wrote: > Hypervisor needs to interact with the summary indicators, so these > need to be DMA memory as well (at least for protected virtualization > guests). > > Signed-off-by: Halil Pasic > --- > drivers/s390/virtio/virtio_ccw.c | 24

[PATCH 10/10] virtio/s390: make airq summary indicators DMA

2019-04-26 Thread Halil Pasic
Hypervisor needs to interact with the summary indicators, so these need to be DMA memory as well (at least for protected virtualization guests). Signed-off-by: Halil Pasic --- drivers/s390/virtio/virtio_ccw.c | 24 +--- 1 file changed, 17 insertions(+), 7 deletions(-) diff