Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 23 Apr 2013 14:15:31 +0100
From:      Tom Evans <tevans.uk@googlemail.com>
To:        Jeremy Chadwick <jdc@koitsu.org>
Cc:        freebsd-stable@freebsd.org
Subject:   Re: ada(4) and ahci(4) quirk printing
Message-ID:  <CAFHbX1JPhfeZMv5t=jLr_2xHVz5XSrhq7Nipd-Qg7xfT%2Boo7nA@mail.gmail.com>
In-Reply-To: <20130423125144.GA62949@icarus.home.lan>
References:  <20130422051452.GA2148@icarus.home.lan> <51763BF9.2000506@FreeBSD.org> <20130423092602.GA58831@icarus.home.lan> <51765466.4040209@FreeBSD.org> <4D28DBAE46424C268AA22FCDD8657946@multiplay.co.uk> <20130423114722.GA61919@icarus.home.lan> <CC7BB743D5AB4312BB3A0FE37AC1C566@multiplay.co.uk> <20130423125144.GA62949@icarus.home.lan>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Apr 23, 2013 at 1:51 PM, Jeremy Chadwick <jdc@koitsu.org> wrote:
> On Tue, Apr 23, 2013 at 01:20:31PM +0100, Steven Hartland wrote:
>> >4) camcontrol wouldn't address the need/interest for ahci(4) quirks to
>> >be made available.
>>
>> Why?
>
> Because camcontrol is for CAM.  ahci(4) is not part of CAM.  The last
> place I'd look for "poking at AHCI" (as in *actual AHCI*) is camcontrol.
>
> This is one of the reasons sysctl exists -- it's a sort of "covers
> everything" tree, on a per-device basis.

Just on this point, these quirks aren't actually quirks of ahci(4) are
they? They are quirks of the disks that are attached to ahci(4), and
presumably should apply regardless of whether the disk in question is
hooked up to ahci(4), siis(4) or even mps(4).

To my mind, this means the quirks should belong to da(4), or at the
very least ada(4). ada(4) and da(4) are both manipulated by CAM, so
having this in camcontrol seems logical to me. To clarify, my mind is
completely oblivious of how all this is implemented in software, so
perhaps my mind needs to be changed rather than the code!

Cheers

Tom



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAFHbX1JPhfeZMv5t=jLr_2xHVz5XSrhq7Nipd-Qg7xfT%2Boo7nA>