[PATCH 1/1] staging: hv: Add support for >2 TB LUN in storage driver.

Mike Sterling Mike.Sterling at microsoft.com
Tue Sep 6 19:14:01 UTC 2011


On Tuesday, September 06, 2011 12:06 PM, Greg KH wrote:
>On Thu, Sep 01, 2011 at 03:11:09PM -0700, Mike Sterling wrote:
>> If a LUN larger than 2 TB is attached to a Linux VM on Hyper-V, we 
>> currently report a maximum size of 2 TB. This patch resolves the issue in hv_storvsc.
>> Thanks to Robert Scheck <robert.scheck at etes.de> for reporting the issue.
>> 
>> Reported-by: Robert Scheck <robert.scheck at etes.de>
>> Signed-off-by: Mike Sterling <mike.sterling at microsoft.com>
>> Signed-off-by: K.Y. Srinivasan <kys at microsoft.com>
>> Signed-off-by: Haiyang Zhang <haiyangz at microsoft.com>
>> ---
>>  drivers/staging/hv/hyperv_storage.h |    1 +
>
>This file is no longer in the latest kernel tree (linux-next or staging-next).  What tree did you generate this patch against?  And >how come Haiyang and KY didn't catch such a basic mistake in their review...
>
>I'm guessing that this patch was not even tested :(
>
>Please fix this and resend.

Greg,

This patch was reviewed by both Ky and Haiyang. However, it looks like I built the patch against an older tree that didn't have Ky's changes. The "how" is still unclear, even after reviewing my steps with Ky. I've discussed this with Ky, I'm rebuilding the patch today, and will resubmit once we confirm.

-Mike



More information about the devel mailing list