AsiaBSDCon DEVSUMMIT patch
John Baldwin
jhb at freebsd.org
Thu Mar 27 13:00:58 PDT 2008
On Thursday 27 March 2008 03:32:29 am M. Warner Losh wrote:
> Greetings,
>
> We've been talking about the situation with suspend/resume in the
> tree. Here's a quick hack to allow one to suspend/resume an
> individual device. This may or may not work too well, but it is
> offered up for testing and criticism.
>
> http://people.freebsd.org/~imp/devctl.diff
>
> devctl -s ath 0 suspend ath0
> devctl -r ath 0 resume ath0
Unfortunately, what you really need is to power down the device to D3 for
suspend and then bring it up. Otherwise you might not lose enough state to
notice that resume isn't restoring all of it. bge(4) doesn't survive resume
on my laptop I think because brgphy doesn't re-patch the firmware on resume,
and you'd need a full power down to run into that sort of thing.
What I would actually prefer would be this:
devctl ath0 power off (maps to D3 on PCI/ACPI)
devctl ath0 power D1 (PCI/ACPI-specific)
devctl ath0 power on (maps to D0 on PCI/APCI)
You'd probably need a 'int BUS_SET_POWERSTATE(device_t parent, device_t child,
const char *state)' and implement it for ACPI and PCI. You would then have
the ioctl handler do:
/* copy in state string */
/* lookup device */
error = BUS_SET_POWERSTATE(device_get_parent(device), device, state);
I would also make devctl take a plain device name and figure out the
devclass/unit from that. Either that or pass the device name as a string to
the kernel and have it do a lookup (for a userland ioctl I don't think an
O(n) walk over the device list is all that evil).
If you want to do named commands (like 'power') rather than getopt args for
everything you can use a linker set to build a table of commands (I've done
this for RAID management utils at work) that let you do something like:
struct devctl_power_request {
const char device[MAXDEVNAME];
const char state[32];
}
#define DEVIOC_POWER _IOW('d', 1, struct devctl_power_request)
/* av[0] will be 'power' */
static void
power_command(int ac, char **av)
{
struct devctl_power_request req;
if (ac != 3)
errx(1, "Usage: devctl power <device> <state>");
strlcpy(req.device, av[1], sizeof(req.device));
strlcpy(req.state, av[2], sizeof(req.state));
if (ioctl(fd, DEVIOC_POWER, &req) < 0)
err(1, "Set power state failed");
}
DEVCTL_COMMAND(power);
(Using a linker set makes it easier to add new commands later and have them
all be self-contained.)
--
John Baldwin
More information about the freebsd-arch
mailing list