Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 29 Jun 2009 13:06:37 +0300
From:      Alexander Motin <mav@FreeBSD.org>
To:        Kamigishi Rei <spambox@haruhiism.net>
Cc:        Goran Lowkrantz <glz@hidden-powers.com>, freebsd-current@freebsd.org, Alexander Best <alexbestms@math.uni-muenster.de>
Subject:   Re: RFC: ATA to CAM integration patch
Message-ID:  <4A48922D.5090507@FreeBSD.org>
In-Reply-To: <4A487BF7.8060103@haruhiism.net>
References:  <1246206181.00132972.1246192801@10.7.7.3>	<1246206186.00132982.1246194002@10.7.7.3>	<1246209783.00133001.1246197001@10.7.7.3>	<1246260183.00133237.1246247402@10.7.7.3> <4A48798A.5070604@FreeBSD.org> <4A487BF7.8060103@haruhiism.net>

next in thread | previous in thread | raw e-mail | index | archive | help
Kamigishi Rei wrote:
> Alexander Motin wrote:
>>> Kernel builds fine for me after "sed 
>>> 's/AHCI_DMA_ENTRIES/AHCI_SG_ENTRIES/' /usr/src/sys/dev/ahci/ahci.c" 
>>> (only one occurrence of that macro, aside from the .h file, anyway).
>> You are right. Thank you. Committed to P4.
> On a side note: the two secondary patches mentioned in the discussion 
> (camcontrol and the other one) do not apply cleanly to r195137, even 
> though the files' contents match. I had to edit the files manually, 
> however, kernel still panics in xpt during boot right after the AHCI bus 
> poll.
> I didn't remove atapicam from the kernel configuration, because 
> according to your post with that patch earlier the patch should have 
> fixed that behaviour.

Sorry, my fault, it is the different bug, due to disabled invariants I 
have missed two locking issues. Here is regenerated patch, including 
this fix and all previous fixes and improvements:
http://people.freebsd.org/~mav/cam-ata.20090629.patch

-- 
Alexander Motin



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