Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751473AbaGPTo4 (ORCPT ); Wed, 16 Jul 2014 15:44:56 -0400 Received: from userp1040.oracle.com ([156.151.31.81]:35511 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750732AbaGPTox (ORCPT ); Wed, 16 Jul 2014 15:44:53 -0400 To: James Bottomley Cc: "martin.petersen\@oracle.com" , "linux-kernel\@vger.kernel.org" , "hch\@infradead.org" , "devel\@linuxdriverproject.org" , "apw\@canonical.com" , "kys\@microsoft.com" , "stable\@vger.kernel.org" , "linux-scsi\@vger.kernel.org" , "ohering\@suse.com" , "jasowang\@redhat.com" Subject: Re: [PATCH 4/8] Drivers: scsi: storvsc: Filter WRITE_SAME_16 From: "Martin K. Petersen" Organization: Oracle Corporation References: <20140709084300.GD6012@infradead.org> <1404935792.2184.5.camel@dabdike.int.hansenpartnership.com> <2f3ae589e6f149acbe4c5dd79f905971@BY2PR03MB299.namprd03.prod.outlook.com> <1404944843.2184.8.camel@dabdike.int.hansenpartnership.com> <20140711063216.GA20660@infradead.org> <328b7a6174ef4dd8a54a7db5ac959834@BY2PR03MB299.namprd03.prod.outlook.com> <20140716110111.GA7382@infradead.org> <20140716173827.GB20528@infradead.org> <1405533734.3165.12.camel@dabdike.int.hansenpartnership.com> <1405536631.3165.23.camel@dabdike.int.hansenpartnership.com> <1405538056.3165.27.camel@dabdike.int.hansenpartnership.com> Date: Wed, 16 Jul 2014 15:44:13 -0400 In-Reply-To: <1405538056.3165.27.camel@dabdike.int.hansenpartnership.com> (James Bottomley's message of "Wed, 16 Jul 2014 19:14:16 +0000") Message-ID: User-Agent: Gnus/5.130012 (Ma Gnus v0.12) Emacs/24.2 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Source-IP: acsinet22.oracle.com [141.146.126.238] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org >>>>> "James" == James Bottomley writes: James> Well, your judgement: is this situation (support for UNMAP but James> not for WRITE_SAME) in what is effectively a RAID driver (hv James> drivers count as RAID) just a silly Microsoft one off? I only recall seeing one or two devices that supported LBP but not WRITE SAME w/UNMAP. James> However, if we get any RAID drivers with strange discard support, James> we'll likely get the same problem The LBP VPD page is mandatory now. It wasn't for the first couple of generations of devices that we still have to support. I think that if a vendor were to support LBP, adding the mandatory VPD page would be a given. And so far nobody has messed up the LBP VPD page contents. My main gripe about linking no_write_same and discard functionality is that the heuristics for the latter are already excessively complex thanks to having to support devices that predate the spec. I'm wary of adding another dimension to that. Also, linking the two use cases we can get into inconsistent states where no_write_same is set but the device does not support UNMAP and has LBPWS=1 and LBPWS10=1 set in the LBP VPD. I'll contemplate the LBPME => mandatory VPD lookup thing for bit. -- Martin K. Petersen Oracle Linux Engineering -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/