Re: Deterministic panic due to non-sleepable lock with if_alc when reconfiguring interfaces

From: YongHyeon PYUN <pyunyh_at_gmail.com>
Date: Mon, 22 Aug 2011 14:35:49 -0700
On Mon, Aug 22, 2011 at 02:26:47PM -0700, Garrett Cooper wrote:
> On Mon, Aug 22, 2011 at 1:40 PM, YongHyeon PYUN <pyunyh_at_gmail.com> wrote:
> > On Sun, Aug 21, 2011 at 04:48:56PM -0700, YongHyeon PYUN wrote:
> >> On Fri, Aug 19, 2011 at 12:17:12AM -0700, Garrett Cooper wrote:
> >> > On Thu, Aug 18, 2011 at 9:31 PM, ?<mdf_at_freebsd.org> wrote:
> >> > > On Thu, Aug 18, 2011 at 5:50 PM, Garrett Cooper <yanegomi_at_gmail.com> wrote:
> >> > >> ? ?When loading if_alc as a module on my netbook and running
> >> > >> /etc/rc.d/netif restart, I can deterministically panic my netbook with
> >> > >> the following message:
> >> >
> >> > ? ? These repro steps were overly simplified. The complete steps are:
> >> >
> >> > 1. Attach ethernet cable to alc(4) enabled NIC.
> >> > 2. Boot up machine.
> >> > 3. Login.
> >> > 4. Physically remove ethernet cable from alc(4) enabled NIC.
> >> > 5. Run `/etc/rc.d/netif restart' as root.
> >> >
> >>
> >> I can't reproduce this with AR8151 sample board. Could you give me
> >> dmesg output to know exact controller revision?
> >> One issue I'm aware of is lack of re-establishing link when
> >> controller firmware put its PHY to deep sleep mode. ?The deep sleep
> >> mode seems to be automatically activated by firmware when it
> >> detects no energy signal(i.e. cable unplugged) so I had to down and
> >> up the interface again to take the PHY out of the sleep mode.
> >>
> >
> > Not re-establishing link issue was fixed in r225088. ?I'm not sure
> > whether this also fixes kern/148772 though. Because you also seem
> > to have the same issue of the PR, it would be good to know whether
> > it makes any difference or not.
> 
>     The panic no longer occurs with that commit when running
> /etc/rc.d/netif restart after inserting and reinserting the ethernet
> cable (I've done it several times for good measure); the failing case
> was potentially being triggered somehow by the hibernation code path.

Hmm, have no idea how this can be related with the panic. :-(
BTW, does the commit also fix kern/148772?

> Thanks,
> -Garrett
> 
Received on Mon Aug 22 2011 - 19:35:57 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:17 UTC