Re: [RFC] ahci(4) patch

From: Alexander Motin <mav_at_FreeBSD.org>
Date: Thu, 17 Nov 2011 00:35:38 +0200
Hi.

On 16.11.2011 23:59, Maksim Yevmenkin wrote:
> would anyone object to the following ahci(4) patch?
>
> ==
>
> --- ahci.c.orig	2011-11-16 21:35:26.000000000 +0000
> +++ ahci.c	2011-11-16 21:35:41.000000000 +0000
> _at__at_ -500,7 +500,7 _at__at_
>   	for (unit = 0; unit<  ctlr->channels; unit++) {
>   		if ((ctlr->ichannels&  (1<<  unit)) == 0)
>   			continue;
> -		child = device_add_child(dev, "ahcich", -1);
> +		child = device_add_child(dev, "ahcich", unit);
>   		if (child == NULL)
>   			device_printf(dev, "failed to add channel device\n");
>   		else
>
> ==
>
> the idea is to have "static" numbering for ada(4) disks.

I do. The only way I see this useful is if you have BIOS configured for 
non-hot-swappable disks, in which case you have some AHCI channels 
disabled, but want to keep numbers of the rest. While I don't like this 
mode in general, especially when it can't be disabled, that patch could 
be useful in these cases. But in other cases, when you have several AHCI 
controllers, it just wont not work. You will receive error on attempt to 
create second ahcich0.

-- 
Alexander Motin
Received on Wed Nov 16 2011 - 21:35:40 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:20 UTC