From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Michael S. Tsirkin" Subject: Re: [PATCH v4 0/8] s390: virtio: support protected virtualization Date: Tue, 11 Jun 2019 06:44:02 -0400 Message-ID: <20190611064354-mutt-send-email-mst@kernel.org> References: <20190606115127.55519-1-pasic@linux.ibm.com> <20190611123740.3d46f31b.cohuck@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <20190611123740.3d46f31b.cohuck@redhat.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: virtualization-bounces@lists.linux-foundation.org Errors-To: virtualization-bounces@lists.linux-foundation.org To: Cornelia Huck Cc: Vasily Gorbik , linux-s390@vger.kernel.org, Thomas Huth , Claudio Imbrenda , kvm@vger.kernel.org, Sebastian Ott , Christoph Hellwig , Farhan Ali , Heiko Carstens , Eric Farman , virtualization@lists.linux-foundation.org, Halil Pasic , Christian Borntraeger , "Jason J. Herne" , Michael Mueller , Viktor Mihajlovski , Janosch Frank List-Id: virtualization@lists.linuxfoundation.org On Tue, Jun 11, 2019 at 12:37:40PM +0200, Cornelia Huck wrote: > On Thu, 6 Jun 2019 13:51:19 +0200 > Halil Pasic wrote: > > > * Documentation is still very sketchy. I'm committed to improving this, > > but I'm currently hampered by some dependencies currently. > > Have the "dependencies" been resolved in the meantime? It probably > would be a good idea to include some documentation for what needs to be > dma and what doesn't somewhere in the kernel documentation (IIRC we > have a s390 drivers 'book' partially generated from kerneldoc; there's > some general document about the cio interfaces as well, but I'm not > sure how up-to-date that is.) > > I think the code in here looks sane from my point of view (except for > the one easy-to-fix issue I found); I would be fine with the virtio-ccw > patches making it into the kernel via the s390 tree (and not via the > virtio tree). Yes, me too.