Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 3 Jan 2014 11:32:37 +0100
From:      Zbigniew Bodek <zbb@freebsd.org>
To:        John Baldwin <jhb@freebsd.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r260163 - head/sys/dev/ahci
Message-ID:  <CALF_TxkP7A8xZRSEkbfAm1SGw0nhGwR12aKrOf0EY_c3LCnEWQ@mail.gmail.com>
In-Reply-To: <201401021345.39250.jhb@freebsd.org>
References:  <201401012018.s01KI31L014441@svn.freebsd.org> <201401021345.39250.jhb@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
2014/1/2 John Baldwin <jhb@freebsd.org>:
> On Wednesday, January 01, 2014 3:18:03 pm Zbigniew Bodek wrote:
>> Author: zbb
>> Date: Wed Jan  1 20:18:03 2014
>> New Revision: 260163
>> URL: http://svnweb.freebsd.org/changeset/base/260163
>>
>> Log:
>>   Do not attach to PCI bridges in AHCI driver
>>
>>   Some vendors use the same VID:PID combination in AHCI and PCI bridge cards
>
> Would it be better to require the class to be PCIC_STORAGE instead?
>
> --
> John Baldwin

Hello John.

As you can see this is required indirectly when "checking for possible
AHCI candidate" but in the loop below the device can still be probed.
I will need to contact the person from "Submitted by" to ensure
whether adding AHCI_Q_NOFORCE quirk is not going to solve this issue
instead.

Best regards
zbb



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CALF_TxkP7A8xZRSEkbfAm1SGw0nhGwR12aKrOf0EY_c3LCnEWQ>