From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754794AbeDWLc4 (ORCPT ); Mon, 23 Apr 2018 07:32:56 -0400 Received: from mx3-rdu2.redhat.com ([66.187.233.73]:36554 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1754624AbeDWLcv (ORCPT ); Mon, 23 Apr 2018 07:32:51 -0400 Date: Mon, 23 Apr 2018 13:32:45 +0200 From: Cornelia Huck To: Dong Jia Shi Cc: linux-kernel@vger.kernel.org, linux-s390@vger.kernel.org, kvm@vger.kernel.org, borntraeger@de.ibm.com, bjsdjshi@linux.ibm.com, pasic@linux.ibm.com, pmorel@linux.ibm.com Subject: Re: [PATCH v2 0/5] vfio: ccw: error handling fixes and improvements Message-ID: <20180423133245.164c64b4.cohuck@redhat.com> In-Reply-To: <20180423110113.59385-1-bjsdjshi@linux.vnet.ibm.com> References: <20180423110113.59385-1-bjsdjshi@linux.vnet.ibm.com> Organization: Red Hat GmbH MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 23 Apr 2018 13:01:08 +0200 Dong Jia Shi wrote: > Dear Reviewers, > > Here is a new version for this patch series. > > We didn't get agreement on patch #5 (#4 in v1) in the former cycle though, > I made it based on my understanding. We can continue discussing on it. > > Changelog: > v1->v2: > - #1. Reworded commit message and comment, plus some typo fixes. > - #2. New patch. > - #3. Added the missing suggested-by Pierre. > Fixed typos. > Added sanity check on pa->pa_iova_pfn and updated comments accordingly. > - #4. Removed unused idaw_nr. > - #5. Replaced leading white spaces with TABs. > Traced the function in anycase. > > Dong Jia Shi (3): > vfio: ccw: shorten kernel doc description for pfn_array_pin() > vfio: ccw: refactor and improve pfn_array_alloc_pin() > vfio: ccw: set ccw->cda to NULL defensively > > Halil Pasic (2): > vfio: ccw: fix cleanup if cp_prefetch fails > vfio: ccw: add traceponits for interesting error paths > > drivers/s390/cio/Makefile | 1 + > drivers/s390/cio/vfio_ccw_cp.c | 134 ++++++++++++++++++++------------------ > drivers/s390/cio/vfio_ccw_fsm.c | 16 ++++- > drivers/s390/cio/vfio_ccw_trace.h | 77 ++++++++++++++++++++++ > 4 files changed, 164 insertions(+), 64 deletions(-) > create mode 100644 drivers/s390/cio/vfio_ccw_trace.h > Out of this series, patch 1 is a fix, while the rest are improvements, correct? So patch 1 would be material for 4.17 (and maybe stable?), the rest for 4.18?