LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Cornelia Huck <cohuck@redhat.com>
To: Dong Jia Shi <bjsdjshi@linux.ibm.com>
Cc: linux-kernel@vger.kernel.org, linux-s390@vger.kernel.org,
	kvm@vger.kernel.org, borntraeger@de.ibm.com, pasic@linux.ibm.com,
	pmorel@linux.ibm.com
Subject: Re: [PATCH v4 0/4] vfio: ccw: error handling fixes and improvements
Date: Wed, 23 May 2018 12:21:29 +0200	[thread overview]
Message-ID: <20180523122129.5f765d24.cohuck@redhat.com> (raw)
In-Reply-To: <20180523025645.8978-1-bjsdjshi@linux.ibm.com>

On Wed, 23 May 2018 04:56:41 +0200
Dong Jia Shi <bjsdjshi@linux.ibm.com> wrote:

> Dear Reviewers,
> 
> Here is a new version for this patch series.
> 
> We didn't get agreement on patch #4 (#5 in v2) in the former cycle though,
> I made it based on my understanding. We can continue discussing on it.

I'll probably go ahead and queue 1-3 (would not mind r-bs, though) in
any case.

I hope we can queue the tracing patch as well, as any tracing beats no
tracing :)

> 
> Changelog:
> v3->v4:
>  - Change mail address from @linux.vnet.ibm.com to @linux.ibm.com
> 
> v2->v3:
>  - #2. Reworded comment message.
>  - #4. Trace failure only, trace all error path, and a common knob for all
>        failure paths.
> 
> 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 (1):
>   vfio: ccw: add traceponits for interesting error paths
> 
>  drivers/s390/cio/Makefile         |   1 +
>  drivers/s390/cio/vfio_ccw_cp.c    | 121 +++++++++++++++++++-------------------
>  drivers/s390/cio/vfio_ccw_fsm.c   |  17 +++++-
>  drivers/s390/cio/vfio_ccw_trace.h |  54 +++++++++++++++++
>  4 files changed, 130 insertions(+), 63 deletions(-)
>  create mode 100644 drivers/s390/cio/vfio_ccw_trace.h
> 

  parent reply	other threads:[~2018-05-23 10:21 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-05-23  2:56 Dong Jia Shi
2018-05-23  2:56 ` [PATCH v4 1/4] vfio: ccw: shorten kernel doc description for pfn_array_pin() Dong Jia Shi
2018-05-23  2:56 ` [PATCH v4 2/4] vfio: ccw: refactor and improve pfn_array_alloc_pin() Dong Jia Shi
2018-05-23  2:56 ` [PATCH v4 3/4] vfio: ccw: set ccw->cda to NULL defensively Dong Jia Shi
2018-05-23  2:56 ` [PATCH v4 4/4] vfio: ccw: add traceponits for interesting error paths Dong Jia Shi
2018-05-23 10:25   ` Cornelia Huck
2018-05-23 10:21 ` Cornelia Huck [this message]
2018-05-24 14:29   ` [PATCH v4 0/4] vfio: ccw: error handling fixes and improvements Cornelia Huck

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180523122129.5f765d24.cohuck@redhat.com \
    --to=cohuck@redhat.com \
    --cc=bjsdjshi@linux.ibm.com \
    --cc=borntraeger@de.ibm.com \
    --cc=kvm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-s390@vger.kernel.org \
    --cc=pasic@linux.ibm.com \
    --cc=pmorel@linux.ibm.com \
    --subject='Re: [PATCH v4 0/4] vfio: ccw: error handling fixes and improvements' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).