On 20 April 2015 at 19:21, Konstantin Belousov <[email protected]> wrote: [..] > +struct { > + void *id_fn; > + char *id_name; > +} idle_tbl[] = { > + { cpu_idle_spin, "spin" }, > + { cpu_idle_mwait, "mwait" }, > + { cpu_idle_hlt, "hlt" }, > +#if !defined(__i386__) || !defined(PC98) > + { cpu_idle_acpi, "acpi" }, > +#endif > + { NULL, NULL } > +}; > +
I believe this conditional could be left unchanged as #ifndef PC98 (also in several other places), given that pc98 may not be present other than under i386. Otherwise, looks good. -- wbr, pluknet _______________________________________________ [email protected] mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-amd64 To unsubscribe, send any mail to "[email protected]"
