Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1620838

Re: Race to power off harming SATA SSDs

Path csiph.com!usenet.pasdenom.info!paganini.bofh.team!weretis.net!feeder4.news.weretis.net!news.mixmin.net!aioe.org!bofh.it!news.nic.it!robomod
From Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Newsgroups linux.kernel
Subject Re: Race to power off harming SATA SSDs
Date Tue, 11 Apr 2017 04:10:02 +0200
Message-ID <tuTpL-Kz-3@gated-at.bofh.it> (permalink)
References <tuQUW-7sQ-9@gated-at.bofh.it> <tuRnX-7Fz-5@gated-at.bofh.it> <tuRnX-7Fz-3@gated-at.bofh.it>
Dkim-Signature v=1; a=rsa-sha256; c=relaxed/relaxed; d=hmh.eng.br; h=cc :content-type:date:from:in-reply-to:message-id:mime-version :references:subject:to:x-me-sender:x-me-sender:x-sasl-enc :x-sasl-enc; s=fm1; bh=zmhs3pynI18rkfUd10JJW+hbThibmuzrYawweFYAh 0Y=; b=nIF+AlW6WZa4/JRJ5oBXKEUYpv2jh+O+wIkJkw9cXj9C/bLSF9+E3gkMS I1V/PI4pjYqa6MPP7dSqHwO1k+u9Ta0flzBjNSf0SMJm/kLQSJja5Ma/N6dd5jL8 Rwpy93Ibbsi6WAOX4+0nfUaM2e1O8GF+rGRAeljH7LZOW8H3VAFv9WFaVVWNgdAX 4S8cvkz/Qgb20M65z7jC3WniEFO2ljpNzaKG5rDVYrzv2+2ItqP9FqXOHZS0aCI+ prqlCrlHvFB71Mou2P0VM085wk9jKI6zvmzl7LzYx4gWlMz9U6qsh7R1nb+t7RhE +LmiSH4cuiIB8iTFesml4D/W95Abg==
Dkim-Signature v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-sender :x-me-sender:x-sasl-enc:x-sasl-enc; s=fm1; bh=zmhs3pynI18rkfUd10 JJW+hbThibmuzrYawweFYAh0Y=; b=Q6eTfwDYYpkazU05BkIwoptDYQWiL/B8Bm hMrFHVOJ8JiQv81CkkOaPNKfBLyN69eLupw/pUxnqMPpZBej4SiFmlZynLgg8s3L wzwSD+LPUyNDq+1L1wASkX5sV7diW5cEfEfY4XzJNdghWEOi3wDprz1PMakbJyEt g4Q7oKziUOzF8wEw34e26bv07oHkHPGTgDjcq2a7abL/qeMui4nVf7ojmqig0AkF LckKckhGCG5B93HXfCatexCs6Rh8rnOPeByGHMqnYkuhB8s64uqU8t4uxn0Cxb58 rdP3JyZbszD4kCF9Zqk6DiIHT35KvHgcurQVCoSg9xeyY36AL0PQ==
X-Me-Sender <xms:PjnsWFFX2ifU_tm6FNE9vZvQF_ikEhZ5VYLsPjIlV3Wifbmst8Dl2A>
X-Sasl-Enc Rp5yuKc7vR960r1r3IG60OuDmUfjYq/jEPujCYSSnmvF 1491876157
MIME-Version 1.0
Content-Type text/plain; charset=us-ascii
Content-Disposition inline
X-Gpg-Fingerprint1 4096R/0x0BD9E81139CB4807: C467 A717 507B BAFE D3C1 6092 0BD9 E811 39CB 4807
User-Agent Mutt/1.5.23 (2014-03-12)
Sender robomod@news.nic.it
List-ID <linux-kernel.vger.kernel.org>
X-Mailing-List linux-kernel@vger.kernel.org
Approved robomod@news.nic.it
Lines 32
Organization linux.* mail to news gateway
X-Original-Cc Tejun Heo <tj@kernel.org>, linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org, linux-ide@vger.kernel.org, Hans de Goede <hdegoede@redhat.com>
X-Original-Date Mon, 10 Apr 2017 23:02:35 -0300
X-Original-Message-ID <20170411020235.GD10185@khazad-dum.debian.net>
X-Original-References <20170410232118.GA4816@khazad-dum.debian.net> <20170410235206.GA28603@wtj.duckdns.org> <1491868675.2473.22.camel@HansenPartnership.com>
X-Original-Sender linux-kernel-owner@vger.kernel.org
Xref csiph.com linux.kernel:1620838

Show key headers only | View raw


On Mon, 10 Apr 2017, James Bottomley wrote:
> On Tue, 2017-04-11 at 08:52 +0900, Tejun Heo wrote:
> [...]
> > > Any comments?  Any clues on how to make the delay "smarter" to 
> > > trigger only once during platform shutdown, but still trigger per
> > > -device when doing per-device hotswapping ?
> > 
> > So, if this is actually an issue, sure, we can try to work around;
> > however, can we first confirm that this has any other consequences
> > than a SMART counter being bumped up?  I'm not sure how meaningful
> > that is in itself.
> 
> Seconded; especially as the proposed patch is way too invasive: we run

It is a proof of concept thing.  It even says so in the patch commit
log, and in the cover text.

I don't want an one second delay per device.  I never proposed that,
either.  In fact, I *specifically* asked for something else in the
paragraph you quoted.

I would much prefer an one- or two-seconds delay per platform *power
off*.  And that's for platforms that do ACPI-like heavy-duty S3/S4/S5
like x86/x86-64.  Opportunistic high-frequency suspend on mobile likely
requires no such handling.

The per-device delay would be needed only for hotplug removal (device
delete), and that's just because some hardware powers down bays (like
older thinkpads with ATA-compatible bays, and some industrial systems).

-- 
  Henrique Holschuh

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread


Thread

Race to power off harming SATA SSDs Henrique de Moraes Holschuh <hmh@hmh.eng.br> - 2017-04-11 01:30 +0200
  Re: Race to power off harming SATA SSDs Bart Van Assche <Bart.VanAssche@sandisk.com> - 2017-04-11 01:40 +0200
    Re: Race to power off harming SATA SSDs Henrique de Moraes Holschuh <hmh@hmh.eng.br> - 2017-04-11 02:00 +0200
  sd: wait for slow devices on shutdown path Henrique de Moraes Holschuh <hmh@hmh.eng.br> - 2017-04-11 02:00 +0200
  Re: Race to power off harming SATA SSDs James Bottomley <James.Bottomley@HansenPartnership.com> - 2017-04-11 02:00 +0200
    Re: Race to power off harming SATA SSDs Henrique de Moraes Holschuh <hmh@hmh.eng.br> - 2017-04-11 04:10 +0200
  Re: Race to power off harming SATA SSDs Tejun Heo <tj@kernel.org> - 2017-04-11 02:00 +0200
    Re: Race to power off harming SATA SSDs Henrique de Moraes Holschuh <hmh@hmh.eng.br> - 2017-04-11 03:30 +0200
    Re: Race to power off harming SATA SSDs Martin Steigerwald <martin@lichtvoll.de> - 2017-04-11 12:50 +0200
      Re: Race to power off harming SATA SSDs Henrique de Moraes Holschuh <hmh@hmh.eng.br> - 2017-04-11 16:40 +0200
        Re: Race to power off harming SATA SSDs Martin Steigerwald <martin@lichtvoll.de> - 2017-04-12 09:50 +0200

csiph-web