Page MenuHomeFreeBSD

bhyve: Add CPU pinning diagnostic message
AcceptedPublic

Authored by chuck on Wed, Jun 17, 5:59 PM.
Tags
None
Referenced Files
F161015882: D57619.diff
Mon, Jun 29, 9:10 PM
F161015615: D57619.diff
Mon, Jun 29, 9:07 PM
Unknown Object (File)
Sat, Jun 27, 10:52 AM
Unknown Object (File)
Fri, Jun 26, 7:09 AM
Unknown Object (File)
Thu, Jun 25, 11:05 PM
Unknown Object (File)
Wed, Jun 24, 9:50 PM
Unknown Object (File)
Wed, Jun 24, 7:55 AM
Unknown Object (File)
Tue, Jun 23, 2:37 AM

Details

Reviewers
markj
corvink
bnovkov
Group Reviewers
bhyve
Summary

When pinning a vcpu to a hostcpu fails, print out a diagnostic message
to stderr indicating the failing CPU pair.

MFC after: 1 month

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 73939
Build 70822: arc lint + arc unit

Event Timeline

chuck requested review of this revision.Wed, Jun 17, 5:59 PM
bnovkov added inline comments.
usr.sbin/bhyve/bhyverun.c
575–577

We don't really end need error after removing the assertion so we can get rid of it.

579

I suggest adding in sterror(errno) to the warning, the extra information will come in handy.

chuck added inline comments.
usr.sbin/bhyve/bhyverun.c
579

My reading of ERR(3) suggests this is what warn(...) does, but perhaps I'm misunderstanding.

The err(), errc(), verr(), verrc(), warn(), warnc(), vwarn(), and vwarnc() functions append an error message obtained from strerror(3) based on a supplied error code value or the global variable errno, preceded by another colon and space unless the fmt argument is NULL.

Update based on review feedback

bnovkov added inline comments.
usr.sbin/bhyve/bhyverun.c
579

ah right, my bad, I always confuse it with warnx.

This revision is now accepted and ready to land.Fri, Jun 19, 7:49 PM