On Fri, Apr 25, 2014 at 3:24 AM, Alessandro DE LAURENZIS < just22....@gmail.com> wrote:
> On Fri 25/04, Stefan Sperling wrote: > > On Fri, Apr 25, 2014 at 08:53:42AM +0200, Alessandro DE LAURENZIS wrote: > > > Some other debug here, and this time with (I hope) more useful info. > > > > Thanks! > > > > > First of all, I confirm that the debug messages triggered by the added > > > printf instructions are *NOT* logged by dmesg (and /var/log/*), but are > > > instead visible on the screen (it takes a while to find out that, > > > 'cause this laptop has an almost totally broken LCD, as I said > > > previously). > > > > > > Now: > > > > > > 1) after a "halt -p" command, the bge_activate function is triggered > > > with value 5: > > > > > > BGE WOL DEBUG > > > bge_activate:5 > > > > Great. This means we're missing the POWERDOWN case. > > > > Can you try adding this 'case' block to the switch (act) statement > > in bge_activate? > > > > case DVACT_POWERDOWN: > > rv = config_activate_children(self, act); > > if (ifp->if_flags & IFF_RUNNING) > > bge_stop(sc); > > #ifndef SMALL_KERNEL > > bge_wol_power(sc); > > #endif > > break; > > It finally works! > > Stefan, really: I'm impressed; this is a great example of kindness, > availability and commitment; I'm an OpenBSD noob (and probably I'll ever > be, considering that I use Unix and Unix-like systems for about 20 yrs > now and I feel that there are still so many things to learn...), but > I rarely received so much attention and competent support in other > communities. > > I'm attaching a tarball with all the modified files, 'cause we proceeded > iteratively and it's very easy to miss something; it would be great if > you could revisit the code and prepare a patch usable with 5.4-Rel > (I applied some hunks by hand because they were not directly manageable > by the patch command). > FYI: " Most OpenBSD mailing lists strip messages of MIME content before sending them out to the rest of the list." http://www.openbsd.org/mail.html > > I don't know if you're still in time to discuss the modifications and > commit them for the incoming 5.5; the only thing I can add is that, > differently from what you said initially, Broadcom BCM57xx chips are > anything but rare: they have been, for example, extensively used on Dell > Latitude D series (in my case, it is a D810), so enabling WoL on them in > the mainstream would be a significant improvement. > > Keep me informed and thanks again. > > All the best > > -- > Alessandro DE LAURENZIS > [mailto:just22....@gmail.com] > LinkedIn: http://it.linkedin.com/in/delaurenzis > > [demime 1.01d removed an attachment of type application/x-tar-gz]
