Re: [PATCH v4 00/11] s390/vfio-ccw rework

From: Alex Williamson <alex.williamson@redhat.com>
To: Eric Farman <farman@linux.ibm.com>
Cc: Matthew Rosato <mjrosato@linux.ibm.com>,
	Jason Gunthorpe <jgg@nvidia.com>,
	Cornelia Huck <cohuck@redhat.com>,
	Halil Pasic <pasic@linux.ibm.com>,
	kvm@vger.kernel.org, linux-s390@vger.kernel.org,
	Kirti Wankhede <kwankhede@nvidia.com>
Subject: Re: [PATCH v4 00/11] s390/vfio-ccw rework
Date: Thu, 7 Jul 2022 15:32:53 -0600	[thread overview]
Message-ID: <20220707153253.13deffe3.alex.williamson@redhat.com> (raw)
In-Reply-To: <20220707135737.720765-1-farman@linux.ibm.com>

On Thu,  7 Jul 2022 15:57:26 +0200
Eric Farman <farman@linux.ibm.com> wrote:

> Alex,
> 
> Here's a final pass through some of the vfio-ccw rework.
> I'm hoping that because of the intersection with the extern
> removal, you could grab this directly? [1]

Done.  Applied to vfio next branch for v5.20.  Thanks,

Alex

> There were no code changes since v3, I simply rebased this
> onto your linux-vfio/next tree, currently on commit 7654a8881a54
> ("Merge branches
>  'v5.20/vfio/migration-enhancements-v3',
>  'v5.20/vfio/simplify-bus_type-determination-v3',
>  'v5.20/vfio/check-vfio_register_iommu_driver-return-v2',
>  'v5.20/vfio/check-iommu_group_set_name_return-v1',
>  'v5.20/vfio/clear-caps-buf-v3',
>  'v5.20/vfio/remove-useless-judgement-v1' and
>  'v5.20/vfio/move-device_open-count-v2'
>  into
>  v5.20/vfio/next")
> 
> v3->v4:
>  - Rebased to vfio/next tree
>    - Tweak patch 6 to fit with extern removal
>    - The rest applied directly
>  - [MR] Added r-b's (Thank you!)
>  - [EF] Add a comment about cp_free() call in fsm_notoper()
> v3: https://lore.kernel.org/r/20220630203647.2529815-1-farman@linux.ibm.com/
> v2: https://lore.kernel.org/r/20220615203318.3830778-1-farman@linux.ibm.com/
> v1: https://lore.kernel.org/r/20220602171948.2790690-1-farman@linux.ibm.com/
> 
> Footnotes:
> [1] https://lore.kernel.org/r/e1ead3e4-9e7d-f026-485b-157d7dc004d3@linux.ibm.com/
> 
> Cc: Kirti Wankhede <kwankhede@nvidia.com>
> 
> Eric Farman (10):
>   vfio/ccw: Fix FSM state if mdev probe fails
>   vfio/ccw: Do not change FSM state in subchannel event
>   vfio/ccw: Remove private->mdev
>   vfio/ccw: Pass enum to FSM event jumptable
>   vfio/ccw: Flatten MDEV device (un)register
>   vfio/ccw: Update trace data for not operational event
>   vfio/ccw: Create an OPEN FSM Event
>   vfio/ccw: Create a CLOSE FSM event
>   vfio/ccw: Refactor vfio_ccw_mdev_reset
>   vfio/ccw: Move FSM open/close to MDEV open/close
> 
> Michael Kawano (1):
>   vfio/ccw: Remove UUID from s390 debug log
> 
>  drivers/s390/cio/vfio_ccw_async.c   |  1 -
>  drivers/s390/cio/vfio_ccw_drv.c     | 59 +++++------------
>  drivers/s390/cio/vfio_ccw_fsm.c     | 99 ++++++++++++++++++++++++-----
>  drivers/s390/cio/vfio_ccw_ops.c     | 77 +++++++---------------
>  drivers/s390/cio/vfio_ccw_private.h |  9 +--
>  include/linux/mdev.h                |  5 --
>  6 files changed, 125 insertions(+), 125 deletions(-)
> 