On 26.04.2013 19:47, Peter Wemm wrote:
On Thu, Apr 18, 2013 at 2:44 AM, Alexander Motin <m...@freebsd.org> wrote:
Author: mav
Date: Thu Apr 18 09:44:00 2013
New Revision: 249611
URL: http://svnweb.freebsd.org/changeset/base/249611
Log:
MFC r248872, r249048:
Make pre-shutdown flush and spindown routines to not use xpt_polled_action(),
but execute the commands in regular way. There is no any reason to cook CPU
while the system is still fully operational. After this change polling in
CAM is used only for kernel dumping.
FYI, this causes some drivers to deadlock when you attempt to cleanly
reboot the machine. eg: mpt based systems.
Thank you for the report, but I've seen your first email 23.04 and
replied you with proposed solution the same day.
Adding new assumptions about interrupt-driven hooks continuing to work
after the post-sync shutdown hooks don't seem like a -stable
candidate.
That is not "after the post-sync", but the post-sync itself. Is it
written somewhere that it should not work? Because several GEOM modules
are also doing some disk writes at post-sync stage and they expect to be
handled in normal way.
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?
--
Alexander Motin
_______________________________________________
svn-src-stable-9@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-stable-9
To unsubscribe, send any mail to "svn-src-stable-9-unsubscr...@freebsd.org"
