Re: [Qemu-devel] [PATCH v4 0/3] Introduce vhost-user-scsi and sample application

2017-03-29 Thread Felipe Franciosi
> On 29 Mar 2017, at 13:40, Michael S. Tsirkin wrote: > > On Thu, Mar 02, 2017 at 08:43:00AM -0800, Felipe Franciosi wrote: >> Based on various discussions on the 2016 KVM Forum, I'm sending over a >> vhost-user-scsi implementation for your consideration. >> >> This patchset

Re: [Qemu-devel] [PATCH v4 0/3] Introduce vhost-user-scsi and sample application

2017-03-29 Thread Michael S. Tsirkin
On Thu, Mar 02, 2017 at 08:43:00AM -0800, Felipe Franciosi wrote: > Based on various discussions on the 2016 KVM Forum, I'm sending over a > vhost-user-scsi implementation for your consideration. > > This patchset introduces a new vhost-user SCSI device. While heavily > based on vhost-scsi, it is

Re: [Qemu-devel] [PATCH v4 0/3] Introduce vhost-user-scsi and sample application

2017-03-02 Thread Eric Blake
On 03/02/2017 11:01 AM, Felipe Franciosi wrote: > Hi Eric, > > Ping on this? > http://lists.nongnu.org/archive/html/qemu-devel/2016-10/msg06169.html > [context: Re: [Qemu-devel] [PATCH 1/2] compiler: add ignore_value() macro] I'm still in favor of the patch, but only when it has a client. And

Re: [Qemu-devel] [PATCH v4 0/3] Introduce vhost-user-scsi and sample application

2017-03-02 Thread Felipe Franciosi
> On 2 Mar 2017, at 18:08, Paolo Bonzini wrote: > > > > On 02/03/2017 18:39, Felipe Franciosi wrote: >>> Yeah, it wasn't previously noticed since these files weren't compiled >>> by default. Since you've added make rules to enable build, I'm afraid >>> you've got the

Re: [Qemu-devel] [PATCH v4 0/3] Introduce vhost-user-scsi and sample application

2017-03-02 Thread Paolo Bonzini
On 02/03/2017 18:39, Felipe Franciosi wrote: >> Yeah, it wasn't previously noticed since these files weren't compiled >> by default. Since you've added make rules to enable build, I'm afraid >> you've got the unfortunate job of cleaning up pre-existing build >> problems. I'd suggest just doing

Re: [Qemu-devel] [PATCH v4 0/3] Introduce vhost-user-scsi and sample application

2017-03-02 Thread Felipe Franciosi
> On 2 Mar 2017, at 17:17, Daniel P. Berrange wrote: > > On Thu, Mar 02, 2017 at 05:01:06PM +, Felipe Franciosi wrote: >> Hi Eric, >> >> Ping on this? >> http://lists.nongnu.org/archive/html/qemu-devel/2016-10/msg06169.html > > [snip] > >> CC

Re: [Qemu-devel] [PATCH v4 0/3] Introduce vhost-user-scsi and sample application

2017-03-02 Thread Felipe Franciosi
> On 2 Mar 2017, at 17:32, Daniel P. Berrange wrote: > > On Thu, Mar 02, 2017 at 05:21:46PM +, Felipe Franciosi wrote: >> >>> On 2 Mar 2017, at 17:17, Daniel P. Berrange wrote: >>> >>> On Thu, Mar 02, 2017 at 05:01:06PM +, Felipe Franciosi

Re: [Qemu-devel] [PATCH v4 0/3] Introduce vhost-user-scsi and sample application

2017-03-02 Thread Daniel P. Berrange
On Thu, Mar 02, 2017 at 05:21:46PM +, Felipe Franciosi wrote: > > > On 2 Mar 2017, at 17:17, Daniel P. Berrange wrote: > > > > On Thu, Mar 02, 2017 at 05:01:06PM +, Felipe Franciosi wrote: > >> Hi Eric, > >> > >> Ping on this? > >>

Re: [Qemu-devel] [PATCH v4 0/3] Introduce vhost-user-scsi and sample application

2017-03-02 Thread Daniel P. Berrange
On Thu, Mar 02, 2017 at 05:01:06PM +, Felipe Franciosi wrote: > Hi Eric, > > Ping on this? > http://lists.nongnu.org/archive/html/qemu-devel/2016-10/msg06169.html [snip] > CC contrib/ivshmem-client/main.o > >

Re: [Qemu-devel] [PATCH v4 0/3] Introduce vhost-user-scsi and sample application

2017-03-02 Thread Felipe Franciosi
is series failed build test on s390x host. Please find the details below. Type: series Subject: [Qemu-devel] [PATCH v4 0/3] Introduce vhost-user-scsi and sample application Message-id: 1488472983-9108-1-git-send-email-fel...@nutanix.com === TEST SCRIPT BEGIN === … …

Re: [Qemu-devel] [PATCH v4 0/3] Introduce vhost-user-scsi and sample application

2017-03-02 Thread no-reply
Hi, This series failed build test on s390x host. Please find the details below. Type: series Subject: [Qemu-devel] [PATCH v4 0/3] Introduce vhost-user-scsi and sample application Message-id: 1488472983-9108-1-git-send-email-fel...@nutanix.com === TEST SCRIPT BEGIN === #!/bin/bash # Testing

Re: [Qemu-devel] [PATCH v4 0/3] Introduce vhost-user-scsi and sample application

2017-03-02 Thread no-reply
Hi, This series failed automatic build test. Please find the testing commands and their output below. If you have docker installed, you can probably reproduce it locally. Message-id: 1488472983-9108-1-git-send-email-fel...@nutanix.com Type: series Subject: [Qemu-devel] [PATCH v4 0/3] Introduce

[Qemu-devel] [PATCH v4 0/3] Introduce vhost-user-scsi and sample application

2017-03-02 Thread Felipe Franciosi
Based on various discussions on the 2016 KVM Forum, I'm sending over a vhost-user-scsi implementation for your consideration. This patchset introduces a new vhost-user SCSI device. While heavily based on vhost-scsi, it is implemented using vhost's userspace counterpart. The device has been coded

Re: [Qemu-devel] [PATCH v4 0/3] Introduce vhost-user-scsi and sample application

2017-03-02 Thread no-reply
Hi, This series seems to have some coding style problems. See output below for more information: Message-id: 1488472983-9108-1-git-send-email-fel...@nutanix.com Type: series Subject: [Qemu-devel] [PATCH v4 0/3] Introduce vhost-user-scsi and sample application === TEST SCRIPT BEGIN === #!/bin