svn commit: r249611 - in stable/9/sys/cam: ata scsi

[Peter Wemm]

On Sat, Apr 27, 2013 at 5:56 PM, Peter Wemm <peter at wemm.org> wrote:
> On Fri, Apr 26, 2013 at 11:00 AM, Alexander Motin <mav at freebsd.org> wrote:
>> On 26.04.2013 19:47, Peter Wemm wrote:
>>>
>>> On Thu, Apr 18, 2013 at 2:44 AM, Alexander Motin <mav at freebsd.org> wrote:
>>>>
>>>> Author: mav
>>>> Date: Thu Apr 18 09:44:00 2013
>>>> New Revision: 249611
>>>> URL: http://svnweb.freebsd.org/changeset/base/249611
> [..]
>>> This breaks a number of machines in the freebsd.org cluster.  I have
>>> to back out both of these changes to get them to reboot.
>>
>>
>> I've made a search though the base system and found only two drivers
>> affected by this change: mpt and hptmv. I've patched both at head r249849
>> and going to merge fix to stable/9 tomorrow unless objected. Have you tried
>> that patch instead of reverting?
>
> I'm testing this on ns1.freebsd.org and ns2.freebsd.org as we speak.
> If the cluster goes dark, that's why :)

The machines have survived multiple reboots with > r249849.

I do wonder if perhaps "post_sync" isn't the ideal name.  Perhaps add
a "quiesce_hardware" eventhandler chain and make it clear that this is
the hook for what things like mpt were doing.

-- 
Peter Wemm - peter at wemm.org; peter at FreeBSD.org; peter at yahoo-inc.com; KI6FJV