[PATCH 0/2] scsi: hyper-v storvsc changes by Ubuntu

KY Srinivasan kys at microsoft.com
Thu May 22 10:49:20 UTC 2014



> -----Original Message-----
> From: Andy Whitcroft [mailto:apw at canonical.com]
> Sent: Wednesday, May 21, 2014 7:25 AM
> To: Ian Abbott
> Cc: linux-scsi at vger.kernel.org; devel at linuxdriverproject.org; KY Srinivasan;
> Haiyang Zhang; James E.J. Bottomley; Tim Gardner
> Subject: Re: [PATCH 0/2] scsi: hyper-v storvsc changes by Ubuntu
> 
> On 16 May 2014 16:39, Ian Abbott <abbotti at mev.co.uk> wrote:
> > These changes to the Microsoft Hyper-V storage driver in Ubuntu
> > Saucy's
> > 3.13 kernel look useful for the mainline kernel, especially as they
> > enable 'TRIM' support.
> >
> > Andy Whitcroft (2):
> >   scsi: hyper-v storvsc switch up to SPC-3
> >   scsi: hyper-v storvsc -- host takes MAINTENANCE_IN commands badly --
> >     elide them
> >
> >  drivers/scsi/storvsc_drv.c | 10 ++++++++++
> >  1 file changed, 10 insertions(+)
> 
> 
> The back story here is a little complex.  The main issue is that the Hyper-V
> drives claim to be SPC-2, and yet implement the SPC-3 extensions for TRIM.
> We did attempt to upstream quirks to allow these features to be negotiated
> specifically for the Hyper-V virtual drives (minimum regression potential) but
> these were NAKd, and it was suggested that just overriding the Hyper-V
> drives to SPC-3 unconditionally was more appropriate.  The first of the
> patches here does does this.  This has been sitting in our tree for some time
> as it was not clear that this would be entirely safe, though the SPC-3 bits are
> in theory at least mostly detected.  That said this change has been in Ubuntu
> for a full cycle now and does not seem to have caused any issues.  If KY is
> happy we should likely submit it formally.  The second fix I believed was
> already being submitted to mainline.
> 
> KY?

The Windows guys are not currently comfortable claiming conformance to SPC-3, as they have not done
the necessary testing. This will change hopefully soon.

K. Y
> 
> -apw


More information about the devel mailing list