]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
Several small fixes for the amd64 minidump code.
authorKonstantin Belousov <kib@FreeBSD.org>
Sun, 27 Oct 2013 16:31:12 +0000 (16:31 +0000)
committerKonstantin Belousov <kib@FreeBSD.org>
Sun, 27 Oct 2013 16:31:12 +0000 (16:31 +0000)
commite20f049b877aa41086f56edfec24499fccf3cf58
tree7a7247d61a47de0bb44c9974eaa3905cff43133c
parent75bf2db38065cafe6b1716e5e66a00962866d5bf
Several small fixes for the amd64 minidump code.

In report_progress(), use nitems(progress_track) instead of manually
hard-coding array size.  Wrap long line.

In blk_write(), code verifies that ptr and pa cannot be non-zero
simultaneously.  The later check for the page-alignment of the ptr
argument never triggers due to pa != 0 always implying ptr == NULL.  I
believe that the intent was to ensure that physicall address passed is
page-aligned, since the address is (temporary) mapped for the duration
of the page write.

Clear the progress_track.visited fields when starting minidump.  If
minidump is restarted or taken second time during the system lifetime,
progress is not printed otherwise, making operator suspectible to the
dump status.

Tested by: pho
Sponsored by: The FreeBSD Foundation
MFC after: 1 week
sys/amd64/amd64/minidump_machdep.c