Page MenuHomeFreeBSD
Differential D6672

libthr: Enhance PANIC() to format messages
ClosedPublic
Actions

Authored by cem on Jun 1 2016, 6:27 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, May 16, 11:15 AM
Unknown Object (File)
Sat, May 2, 11:33 PM
Unknown Object (File)
Thu, Apr 30, 5:06 PM
Unknown Object (File)
Thu, Apr 30, 5:01 PM
Unknown Object (File)
Thu, Apr 30, 4:55 PM
Unknown Object (File)
Tue, Apr 28, 8:28 PM
Unknown Object (File)
Tue, Apr 28, 8:25 PM
Unknown Object (File)
Mon, Apr 27, 2:33 PM
View All Files
Subscribers
imp

Details

Reviewers
kib
Commits
rS301136: libthr: Use formatted PANIC()
rS301135: libthr: _thread_vprintf: Enhance support for %p, %#x
rS301134: libthr: Add vprintf variant of _thread_printf, formatted PANIC()
Summary

libthr: Add vprintf variant of _thread_printf, formatted PANIC()

libthr: _thread_vprintf: Enhance support for %p, %#x

libthr: Use formatted PANIC()

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

cem updated this revision to Diff 17174.Jun 1 2016, 6:27 AM
cem retitled this revision from to libthr: Enhance PANIC() to format messages.
cem updated this object.
cem edited the test plan for this revision. (Show Details)
cem added a reviewer: kib.
Herald added a subscriber: imp. · View Herald TranscriptJun 1 2016, 6:27 AM
kib accepted this revision.Jun 1 2016, 1:59 PM
kib edited edge metadata.
kib added inline comments.
lib/libthr/thread/thr_exit.c
184 ↗(On Diff #17174)

Replace '2' by 'STDERR_FILENO', since the line is modified anyway.

This revision is now accepted and ready to land.Jun 1 2016, 1:59 PM
cem updated this revision to Diff 17184.Jun 1 2016, 2:56 PM
cem edited edge metadata.

Use STDERR_FILENO instead of 2.

This revision now requires review to proceed.Jun 1 2016, 2:56 PM
kib accepted this revision.Jun 1 2016, 3:09 PM
kib edited edge metadata.
This revision is now accepted and ready to land.Jun 1 2016, 3:09 PM
Closed by commit rS301134: libthr: Add vprintf variant of _thread_printf, formatted PANIC() (authored by cem). · Explain WhyJun 1 2016, 4:10 PM
This revision was automatically updated to reflect the committed changes.
cem marked an inline comment as done.

Revision Contents
Changeset List

PathSize
head/
lib/
libthr/
thread/
22 lines
16 lines
7 lines

Diff 17187

View Options

head/lib/libthr/thread/thr_exit.c

Loading...
View Options

head/lib/libthr/thread/thr_printf.c

Loading...
View Options

head/lib/libthr/thread/thr_private.h

Loading...