Page MenuHomeFreeBSD

xen: expose support for poweroff/reboot/suspend on xenbus
ClosedPublic

Authored by royger on Oct 1 2024, 9:14 AM.
Tags
None
Referenced Files
F108753285: D46859.diff
Mon, Jan 27, 5:38 PM
Unknown Object (File)
Fri, Jan 24, 5:26 PM
Unknown Object (File)
Sat, Jan 18, 5:58 PM
Unknown Object (File)
Dec 22 2024, 7:45 PM
Unknown Object (File)
Nov 26 2024, 7:03 PM
Unknown Object (File)
Nov 18 2024, 6:57 PM
Unknown Object (File)
Oct 16 2024, 10:35 AM
Unknown Object (File)
Oct 8 2024, 11:26 PM
Subscribers

Details

Summary

Some toolstacks won't attempt the signal power actions on xenbus unless the VM
explicitly exposes support for them. FreeBSD supports all power actions, hence
signal on xenbus such support by setting the nodes to the value of "1".

Sponsored by: Cloud Software Group

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

royger requested review of this revision.Oct 1 2024, 9:14 AM

This looks ok to me aside from style nits. I don't know Xen well enough to properly review the substance of the change.

sys/dev/xen/control/control.c
401

Extra newline.

502

Extra newline.

508

Extra newline.

Thanks for the review, will get the extra newlines removed and this committed.

sys/dev/xen/control/control.c
401

Oh, we finally changed the style to not mandate an empty newline if there are no local varaibles in a function:

static void
usage(void)
{
       /* Optional blank line goes here. */

Optionally, insert a blank line at the beginning of functions with no
local variables.   Older versions of this style document required the
blank line convention, so it is widely used in existing code.
This revision is now accepted and ready to land.Nov 5 2024, 3:33 PM