Re: [PATCH v2 1/5] vfio: ccw: fix cleanup if cp_prefetch fails

2018-04-25 Thread Halil Pasic
On 04/25/2018 04:43 AM, Dong Jia Shi wrote: [Some procedural notes: I've created a new vfio-ccw-fixes branch based on the s390 fixes branch for easier handling. Things targeted for the next release will go on the vfio-ccw branch on top of the s390 features branch, as before. Does that work for

Re: [PATCH v2 1/5] vfio: ccw: fix cleanup if cp_prefetch fails

2018-04-24 Thread Cornelia Huck
On Mon, 23 Apr 2018 13:01:09 +0200 Dong Jia Shi wrote: > From: Halil Pasic > > If the translation of a channel program fails, we may end up attempting > to clean up (free, unpin) stuff that never got translated (and allocated, > pinned) in the first place. > > By adjusting the lengths of the c

Re: [PATCH v2 1/5] vfio: ccw: fix cleanup if cp_prefetch fails

2018-04-23 Thread Halil Pasic
On 04/23/2018 01:40 PM, Cornelia Huck wrote: On Mon, 23 Apr 2018 13:01:09 +0200 Dong Jia Shi wrote: From: Halil Pasic If the translation of a channel program fails, we may end up attempting to clean up (free, unpin) stuff that never got translated (and allocated, pinned) in the first place

Re: [PATCH v2 1/5] vfio: ccw: fix cleanup if cp_prefetch fails

2018-04-23 Thread Cornelia Huck
On Mon, 23 Apr 2018 13:01:09 +0200 Dong Jia Shi wrote: > From: Halil Pasic > > If the translation of a channel program fails, we may end up attempting > to clean up (free, unpin) stuff that never got translated (and allocated, > pinned) in the first place. > > By adjusting the lengths of the c

Re: [PATCH v2 1/5] vfio: ccw: fix cleanup if cp_prefetch fails

2018-04-23 Thread Halil Pasic
On 04/23/2018 01:01 PM, Dong Jia Shi wrote: From: Halil Pasic If the translation of a channel program fails, we may end up attempting to clean up (free, unpin) stuff that never got translated (and allocated, pinned) in the first place. By adjusting the lengths of the chains accordingly (so t

[PATCH v2 1/5] vfio: ccw: fix cleanup if cp_prefetch fails

2018-04-23 Thread Dong Jia Shi
From: Halil Pasic If the translation of a channel program fails, we may end up attempting to clean up (free, unpin) stuff that never got translated (and allocated, pinned) in the first place. By adjusting the lengths of the chains accordingly (so the element that failed, and all subsequent eleme