Re: [Qemu-devel] [PATCH 0/4] scsi-generic: fixes for Block Limits emulation

2018-11-07 Thread no-reply
Hi, This series failed docker-quick@centos7 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: 20181029173437.32559-1-pbonz...@redhat.com Subject: [Qemu-devel] [PATCH 0/4] scsi

Re: [Qemu-devel] [PATCH 0/4] scsi-generic: fixes for Block Limits emulation

2018-11-07 Thread no-reply
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20181029173437.32559-1-pbonz...@redhat.com Subject: [Qemu-devel] [PATCH 0/4] scsi-generic: fixes for Block Limits emulation === TEST SCRIPT BEGIN === #!/bin/bash BASE=base

Re: [Qemu-devel] [PATCH 0/4] scsi-generic: fixes for Block Limits emulation

2018-11-06 Thread Paolo Bonzini
On 06/11/2018 16:54, Daniel Henrique Barboza wrote: > Hi, > > How did you find all those issues, Max? First patch is something that > I missed out from the SCSI spec (ordering of the VPD pages) > and could have been detected by code inspection,  but I am curious about > the other fixes. It was fo

Re: [Qemu-devel] [PATCH 0/4] scsi-generic: fixes for Block Limits emulation

2018-11-06 Thread Daniel Henrique Barboza
Hi, How did you find all those issues, Max? First patch is something that I missed out from the SCSI spec (ordering of the VPD pages) and could have been detected by code inspection,  but I am curious about the other fixes. Thanks, Daniel On 10/29/18 2:34 PM, Paolo Bonzini wrote: scsi-gener

Re: [Qemu-devel] [PATCH 0/4] scsi-generic: fixes for Block Limits emulation

2018-11-06 Thread no-reply
Hi, This series failed docker-quick@centos7 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: 20181029173437.32559-1-pbonz...@redhat.com Subject: [Qemu-devel] [PATCH 0/4] scsi

Re: [Qemu-devel] [PATCH 0/4] scsi-generic: fixes for Block Limits emulation

2018-11-06 Thread no-reply
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20181029173437.32559-1-pbonz...@redhat.com Subject: [Qemu-devel] [PATCH 0/4] scsi-generic: fixes for Block Limits emulation === TEST SCRIPT BEGIN === #!/bin/bash BASE=base

Re: [Qemu-devel] [PATCH 0/4] scsi-generic: fixes for Block Limits emulation

2018-10-31 Thread no-reply
Hi, This series failed docker-quick@centos7 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: 20181029173437.32559-1-pbonz...@redhat.com Subject: [Qemu-devel] [PATCH 0/4] scsi

Re: [Qemu-devel] [PATCH 0/4] scsi-generic: fixes for Block Limits emulation

2018-10-31 Thread no-reply
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20181029173437.32559-1-pbonz...@redhat.com Subject: [Qemu-devel] [PATCH 0/4] scsi-generic: fixes for Block Limits emulation === TEST SCRIPT BEGIN === #!/bin/bash BASE=base

[Qemu-devel] [PATCH 0/4] scsi-generic: fixes for Block Limits emulation

2018-10-29 Thread Paolo Bonzini
scsi-generic (pass through) devices are able to inject an artificial Block Limits VPD page in order to communicate host HBA limits to the guest. However, Max Reitz found a few issues with the implementation of this feature; this series should fix them all. Paolo Paolo Bonzini (4): scsi-generic: