Message ID | 20160908192736.GA4356@gmail.com (mailing list archive) |
---|---|
State | Not Applicable, archived |
Headers | show |
Hello, On Thu, Sep 08, 2016 at 10:27:37PM +0300, Mike Krinkin wrote: > Hi, > > i tried recent linux-next on my laptop, and after boot system is > almost unusable because most of apps just crash with segfaults and > in dmesg output there are a lot of errors like this: ... > git bisect points at commit 7b20309428598df00ffe ("libata: Add support for SCT > Write Same". I temporary fixed problem with the following change: Shaun, any ideas? If this isn't an easy fix, I'm gonna disable write_same for all devices for now. Thanks.
Hello, Shaun. On Fri, Sep 09, 2016 at 10:26:44AM -0500, Shaun Tancheff wrote: > I'm looking into it now. Let me see if I can reproduce this on any of my > hardware. > > If not there are a couple of options ... one is to only enable for ZBC > devices > where this explicitly required by the spec. > > Or disable for devices that report support trim? I'd much prefer enabling this only on ZBC devices. There isn't any real benefits to !ZBC devices, right? Using non-essential features on ATA never goes well. Thanks.
On Fri, Sep 9, 2016 at 10:36 AM, Tejun Heo <tj@kernel.org> wrote: > Hello, Shaun. > > On Fri, Sep 09, 2016 at 10:26:44AM -0500, Shaun Tancheff wrote: >> I'm looking into it now. Let me see if I can reproduce this on any of my >> hardware. >> >> If not there are a couple of options ... one is to only enable for ZBC >> devices >> where this explicitly required by the spec. >> >> Or disable for devices that report support trim? > > I'd much prefer enabling this only on ZBC devices. There isn't any > real benefits to !ZBC devices, right? Using non-essential features on > ATA never goes well. Sure I'm fine with that. I'll move the WRITE SAME support to be conditional on ZBC. Sending a patch as soon as it's tested. > Thanks. > > -- > tejun Thanks -- Shaun -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Fri, Sep 9, 2016 at 10:36 AM, Tejun Heo <tj@kernel.org> wrote: > Hello, Shaun. > > On Fri, Sep 09, 2016 at 10:26:44AM -0500, Shaun Tancheff wrote: >> I'm looking into it now. Let me see if I can reproduce this on any of my >> hardware. >> >> If not there are a couple of options ... one is to only enable for ZBC >> devices >> where this explicitly required by the spec. >> >> Or disable for devices that report support trim? > > I'd much prefer enabling this only on ZBC devices. There isn't any > real benefits to !ZBC devices, right? Using non-essential features on > ATA never goes well. I've posted a patch for !ZBC. Mike, can you confirm if this works for you? > > Thanks. > > -- > tejun Thanks! (Apologies for the html reply earlier)
diff --git a/drivers/ata/libata-scsi.c b/drivers/ata/libata-scsi.c index 08d9c8e..0dcd59e 100644 --- a/drivers/ata/libata-scsi.c +++ b/drivers/ata/libata-scsi.c @@ -1159,6 +1159,7 @@ static void ata_scsi_sdev_config(struct scsi_device *sdev) { sdev->use_10_for_rw = 1; sdev->use_10_for_ms = 1; + sdev->no_write_same = 1; /* Schedule policy is determined by ->qc_defer() callback and * it needs to see every deferred qc. Set dev_blocked to 1 to