lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 16 Jul 2014 11:44:18 -0400 From: "Martin K. Petersen" <martin.petersen@...cle.com> To: "hch\@infradead.org" <hch@...radead.org> Cc: "Martin K. Petersen" <martin.petersen@...cle.com>, KY Srinivasan <kys@...rosoft.com>, James Bottomley <jbottomley@...allels.com>, "linux-kernel\@vger.kernel.org" <linux-kernel@...r.kernel.org>, "devel\@linuxdriverproject.org" <devel@...uxdriverproject.org>, "apw\@canonical.com" <apw@...onical.com>, "stable\@vger.kernel.org" <stable@...r.kernel.org>, "linux-scsi\@vger.kernel.org" <linux-scsi@...r.kernel.org>, "ohering\@suse.com" <ohering@...e.com>, "jasowang\@redhat.com" <jasowang@...hat.com> Subject: Re: [PATCH 4/8] Drivers: scsi: storvsc: Filter WRITE_SAME_16 >>>>> "hch" == hch@...radead org <hch@...radead.org> writes: hch> read_capacity_16 calls sd_config_discard(sdkp, SD_LBP_WS16) if the hch> LPBME bit is set. At least older SBC drafts left it wide open if a hch> target supports WRITE SAME with UNMAP or UNMAP in this case. Correct. hch> So I think we'd still want a patch to use UNMAP instead of WRITE hch> SAME for this case, which should also fix hyperv. There are lots of devices out there that support WRITE SAME(10) or (16) without the UNMAP bit. And there are devices that support WRITE SAME w/ UNMAP functionality but not "regular" WRITE SAME. no_write_same is there to prevent the REQ_WRITE_SAME use case (for which we have really weak heuristics). Your patch overloads no_write_same so it also governs a REQ_DISCARD use case. My proposed black list patch fixes the hyperv discard issue. So I don't see why we'd need to overload no_write_same which was meant for an entirely different purpose. -- 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@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists