Re: [Qemu-devel] [PATCHv2 3/4] scsi: clarify sense codes for LUN0 emulation

2017-08-17 Thread Hannes Reinecke
On 08/18/2017 02:57 AM, Laszlo Ersek wrote: > On 08/18/17 02:16, Laszlo Ersek wrote: >> On 08/17/17 22:57, Laszlo Ersek wrote: >>> On 08/04/17 12:49, Paolo Bonzini wrote: On 04/08/2017 10:36, Hannes Reinecke wrote: > The LUN0 emulation is just that, an emulation for a non-existing > LU

Re: [Qemu-devel] [PATCHv2 3/4] scsi: clarify sense codes for LUN0 emulation

2017-08-17 Thread Laszlo Ersek
On 08/18/17 02:16, Laszlo Ersek wrote: > On 08/17/17 22:57, Laszlo Ersek wrote: >> On 08/04/17 12:49, Paolo Bonzini wrote: >>> On 04/08/2017 10:36, Hannes Reinecke wrote: The LUN0 emulation is just that, an emulation for a non-existing LUN0. So we should be returning LUN_NOT_SUPPORTED for

Re: [Qemu-devel] [PATCHv2 3/4] scsi: clarify sense codes for LUN0 emulation

2017-08-17 Thread Laszlo Ersek
On 08/17/17 22:57, Laszlo Ersek wrote: > On 08/04/17 12:49, Paolo Bonzini wrote: >> On 04/08/2017 10:36, Hannes Reinecke wrote: >>> The LUN0 emulation is just that, an emulation for a non-existing >>> LUN0. So we should be returning LUN_NOT_SUPPORTED for any request >>> coming from any other LUN. >

Re: [Qemu-devel] [PATCHv2 3/4] scsi: clarify sense codes for LUN0 emulation

2017-08-17 Thread Laszlo Ersek
On 08/04/17 12:49, Paolo Bonzini wrote: > On 04/08/2017 10:36, Hannes Reinecke wrote: >> The LUN0 emulation is just that, an emulation for a non-existing >> LUN0. So we should be returning LUN_NOT_SUPPORTED for any request >> coming from any other LUN. >> And we should be aborting unhandled command

Re: [Qemu-devel] [PATCHv2 3/4] scsi: clarify sense codes for LUN0 emulation

2017-08-04 Thread Paolo Bonzini
On 04/08/2017 10:36, Hannes Reinecke wrote: > The LUN0 emulation is just that, an emulation for a non-existing > LUN0. So we should be returning LUN_NOT_SUPPORTED for any request > coming from any other LUN. > And we should be aborting unhandled commands with INVALID OPCODE, > not LUN NOT SUPPORTED