public inbox for linux-ide@vger.kernel.org
 help / color / mirror / Atom feed
From: Phillip Susi <phill@thesusis.net>
To: Damien Le Moal <dlemoal@kernel.org>
Cc: linux-ide@vger.kernel.org
Subject: Re: [PATCH v8 04/23] scsi: sd: Differentiate system and runtime start/stop management
Date: Wed, 08 Nov 2023 17:25:31 -0500	[thread overview]
Message-ID: <8734xfubl0.fsf@vps.thesusis.net> (raw)
In-Reply-To: <6d2a2015-3e9b-4420-837e-e9fb8816ae78@kernel.org>

Damien Le Moal <dlemoal@kernel.org> writes:

> Yes. Many embedded systems and people using old PCs still use that.
> And that is not the only case. Port multiplier is the other major one since
> multiple drives share the same link. In both cases, the drive "think" it is
> talking to its own port, but the port is used to manage several devices. One
> consequence of this is that the link speed for the port MUST be limited to the
> lowest link speed of all devices. And to discover that, one has to probe all 15
> ports of the PMP to discover the devices attached... Hence the fact that
> bypassing revalidation being a really bad idea.

Really?  I thought that the PMP recieved the FIS on the upstream port,
buffered it, then sent it to the downstream port, and that could be at a
different speed.  So you could have a FBS PMP that is taking 3 Gbps in
on the upstream link, and splitting the FISes out to two different
downstream links at 1.5 Gbps.  Or does that only happen with SAS
switches?

I think the last time I had a PATA drive was before 2006.  It's really
hard to believe that anyone is still using them nearly 20 years later.

> Patching udisk and smartd would be a lot more sensible. An application using
> passthrough to talk to a drive without first checking if the drive is actually
> active is not a sensible thing to do. Especially considering that runtime

They already do that.  The problem is that even checking to see if the
drive is running with CHECK POWER CONDITION causes libata to wake up a
drive from SLEEP ( not STANDBY ) so that it can feed the drive the
command.  Thus why I patched libata to just complete the command without
waking the drive and tell user space that it is in standby.  Likewise,
when the filesystem issued a FLUSH CACHE while the drive is in SLEEP,
libata can just complete the request and ignore it, since there isn't
anything dirty in the cache.

  reply	other threads:[~2023-11-08 22:25 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <c0b086ab-dcd5-4b7b-b931-4d407dd7ad47()kernel!org>
2023-10-12 19:01 ` [PATCH v8 04/23] scsi: sd: Differentiate system and runtime start/stop management Phillip Susi
2023-10-13  0:57   ` Damien Le Moal
2023-10-13 14:36     ` Phillip Susi
2023-10-15 22:09       ` Damien Le Moal
2023-10-21 17:56         ` Phillip Susi
2023-10-23  5:49           ` Damien Le Moal
2023-11-03 18:05             ` Phillip Susi
2023-11-03 23:01               ` Phillip Susi
2023-11-06  2:32                 ` Damien Le Moal
2023-11-07 13:27                   ` Phillip Susi
2023-11-07 21:59                     ` Damien Le Moal
2023-11-08 22:07                       ` Phillip Susi
2023-11-06  3:00               ` Damien Le Moal
2023-11-07 13:45                 ` Phillip Susi
2023-11-07 21:48                   ` Phillip Susi
2023-11-07 23:11                     ` Damien Le Moal
2023-11-08 22:15                       ` Phillip Susi
2023-11-09 22:09                         ` Phillip Susi
2023-11-09 22:57                           ` Damien Le Moal
2023-11-10 16:41                             ` Phillip Susi
2023-11-10  0:43                           ` Damien Le Moal
2023-11-07 22:13                   ` Damien Le Moal
2023-11-08 22:25                     ` Phillip Susi [this message]
2023-09-27 14:18 [PATCH v8 00/23] Fix libata suspend/resume handling and code cleanup Damien Le Moal
2023-09-27 14:18 ` [PATCH v8 04/23] scsi: sd: Differentiate system and runtime start/stop management Damien Le Moal
2023-09-27 19:50   ` Martin K. Petersen
2023-10-10 13:09   ` Phillip Susi
2023-10-10 14:04     ` Damien Le Moal
2023-10-15 16:14   ` Phillip Susi
2023-10-15 22:44     ` Damien Le Moal
2023-10-16 12:39       ` Phillip Susi
2023-10-16 12:55         ` Damien Le Moal
2023-10-17 18:03           ` Phillip Susi
2023-10-17 23:32             ` Damien Le Moal
2023-10-20 19:00               ` Phillip Susi
2023-10-18  6:16             ` Damien Le Moal
2023-10-20 21:23               ` Phillip Susi
2023-10-23  5:51                 ` Damien Le Moal
2023-10-26 21:21                   ` Phillip Susi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8734xfubl0.fsf@vps.thesusis.net \
    --to=phill@thesusis.net \
    --cc=dlemoal@kernel.org \
    --cc=linux-ide@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox