Re: [Qemu-devel] [PATCH 00/27] vhost-user-scsi: code clean-up

2017-09-19 Thread Paolo Bonzini
On 19/09/2017 14:45, Marc-André Lureau wrote: > Now missing 3, 8, 10, 11, 13, 17, 22, 24. Done, only some doubts on patch 13. Paolo

Re: [Qemu-devel] [PATCH 00/27] vhost-user-scsi: code clean-up

2017-09-19 Thread Marc-André Lureau
Hi On Mon, Sep 11, 2017 at 3:18 PM Marc-André Lureau < marcandre.lur...@gmail.com> wrote: > Hi > > On Thu, Aug 24, 2017 at 2:41 AM Liu, Changpeng > wrote: > >> Thanks for helping the example code clean-up. I can rebase the >> vhost-user-blk patch >> set after your commits. For the issue you ment

Re: [Qemu-devel] [PATCH 00/27] vhost-user-scsi: code clean-up

2017-09-11 Thread Liu, Changpeng
017 9:18 PM To: Liu, Changpeng ; qemu-devel@nongnu.org Cc: fel...@nutanix.com; Philippe Mathieu-Daudé Subject: Re: [Qemu-devel] [PATCH 00/27] vhost-user-scsi: code clean-up Hi On Thu, Aug 24, 2017 at 2:41 AM Liu, Changpeng wrote: Thanks for helping the example code clean-up. I can rebase the vhost

Re: [Qemu-devel] [PATCH 00/27] vhost-user-scsi: code clean-up

2017-09-11 Thread Marc-André Lureau
Hi On Thu, Aug 24, 2017 at 2:41 AM Liu, Changpeng wrote: > Thanks for helping the example code clean-up. I can rebase the > vhost-user-blk patch > set after your commits. For the issue you mentioned below, I think the > vhost-user-scsi example > can only support 1 LUN, so LUN0 is exist, LUN1 rep

Re: [Qemu-devel] [PATCH 00/27] vhost-user-scsi: code clean-up

2017-08-23 Thread Liu, Changpeng
Thanks for helping the example code clean-up. I can rebase the vhost-user-blk patch set after your commits. For the issue you mentioned below, I think the vhost-user-scsi example can only support 1 LUN, so LUN0 is exist, LUN1 reported error should be okay. > -Original Message- > From: Ma

Re: [Qemu-devel] [PATCH 00/27] vhost-user-scsi: code clean-up

2017-08-23 Thread no-reply
Hi, This series failed build test on FreeBSD host. Please find the details below. Message-id: 20170823162004.27337-1-marcandre.lur...@redhat.com Type: series Subject: [Qemu-devel] [PATCH 00/27] vhost-user-scsi: code clean-up === TEST SCRIPT BEGIN === #!/bin/sh # Testing script will be invoked

Re: [Qemu-devel] [PATCH 00/27] vhost-user-scsi: code clean-up

2017-08-23 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. Type: series Message-id: 20170823162004.27337-1-marcandre.lur...@redhat.com Subject: [Qemu-devel] [PATCH 00/27] vhost-user

Re: [Qemu-devel] [PATCH 00/27] vhost-user-scsi: code clean-up

2017-08-23 Thread no-reply
Hi, This series failed build test on s390x host. Please find the details below. Message-id: 20170823162004.27337-1-marcandre.lur...@redhat.com Subject: [Qemu-devel] [PATCH 00/27] vhost-user-scsi: code clean-up Type: series === TEST SCRIPT BEGIN === #!/bin/bash # Testing script will be invoked

[Qemu-devel] [PATCH 00/27] vhost-user-scsi: code clean-up

2017-08-23 Thread Marc-André Lureau
Hi, While reviewing vhost-user-blk, I realized a lot of code was based on vhost-user-scsi, and I found a number of improvements could be made. As a result in this series, I tried to move common glib code in libvhost-user-glib. (I originally made libvhost-user glib-free, so if external projects wan