[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH RFC v3 00/15] basic vfio-ccw infrastructure
From: |
Dong Jia Shi |
Subject: |
Re: [Qemu-devel] [PATCH RFC v3 00/15] basic vfio-ccw infrastructure |
Date: |
Mon, 13 Mar 2017 15:16:52 +0800 |
User-agent: |
Mutt/1.7.0 (2016-08-17) |
* Cornelia Huck <address@hidden> [2017-03-10 11:46:05 +0100]:
> On Thu, 9 Mar 2017 17:25:25 +0800
> Dong Jia Shi <address@hidden> wrote:
>
> > * Dong Jia Shi <address@hidden> [2017-02-17 09:29:24 +0100]:
> >
> > > vfio-ccw: the basic infrastructure
> > > ==================================
> > >
> > > Introduction
> > > ------------
> > >
> > > Here we describe the vfio support for I/O subchannel devices for
> > > Linux/s390. Motivation for vfio-ccw is to passthrough subchannels to a
> > > virtual machine, while vfio is the means.
> > >
> >
> > Ping.
> >
> > Any more comment for this version?
> >
> > Thanks.
> >
> > [...]
> >
Hey Conny,
>
> I just walked through the patch set again and did not notice anything
> else. There's still the area of channel path handling, but I don't see
> why it shouldn't be able to fit into the infrastructure you introduce
> here. (I'd expect most of the interesting stuff there to be handled in
> user space anyway.)
Thanks! I have the same expectation for the channel path handling part.
>
> You should update MAINTAINERS, though. I'm not sure the s390 cio folks
> are prepared to handle vfio-ccw :) (although I think you should cc:
> them on the patches exporting cio interfaces.) It's probably best to
> take vfio-ccw through the kvm-s390 tree.
Beside the fixes for former comments form you and Alex, I will also do
the following stuff in the next version:
1. Remove the "RFC" tag from the mail subject.
2. Cc the s390 CIO layer maintainers for patch 1 and 2:
Sebastian Ott <address@hidden>
Peter Oberparleiter <address@hidden>
3. For the new created files, update MAINTAINERS by adding:
S390 VFIO-CCW DRIVER
M: Cornelia Huck <address@hidden>
L: address@hidden
L: address@hidden
S: Supported
F: drivers/s390/cio/vfio_ccw*
F: Documentation/s390/vfio-ccw.txt
F: include/uapi/linux/vfio_ccw.h
--
Dong Jia