Re: [d-kernel] Re: ?????? oops

[Anton Farygin <rider@altlinux.com>]

On Thu, Apr 01, 2004 at 07:35:57PM +0300, Andrey Khavryuchenko wrote:
> Hi!
> 
> "AVK" == Andrey Khavryuchenko wrote:
> 
>  AVK> "SV" == Sergey Vlasov wrote:
> 
>  SV> On Mon, Mar 29, 2004 at 04:09:35PM +0300, Andrey Khavryuchenko wrote:
>  >>> Background: опакечиваю user-space поддержку Zyxel Prestige 630 EE (630-11)
>  >>> Запускаемая ручками инициализация/прибитие работает без проблем.  При
>  >>> попытке запустить то же из hotplug поймал oops:
> 
>  SV> static int br2684_regvcc(struct atm_vcc *atmvcc, unsigned long arg)
>  SV> ...
>  SV> skb_queue_head_init(&copy);
>  SV> skb_migrate(&atmvcc->sk->receive_queue, &copy);
>  SV> while ((skb = skb_dequeue(&copy))) {
>  SV> BRPRIV(skb->dev)->stats.rx_bytes -= skb->len;
>  SV> BRPRIV(skb->dev)->stats.rx_packets--;
>  SV> br2684_push(atmvcc, skb);
>  SV> }
>  SV> ...
> 
>  SV> и вот в этом цикле поймали skb->dev == NULL.  Интересно, откуда
>  SV> могли взяться такие пакеты?  Никогда не имел дела с ATM...
> 
>  AVK> А ХЗ!  Это не ATM, это aDSL, который у нас поддерживается через atm-ную
>  AVK> байду. 
> 
>  AVK> br2684_regvcc - это в одноименном модуле?  И там IMO стооит поставить 
>  AVK> if(skb->dev) 
>  AVK> вокруг всего хозяйства...
> 
> Сделал.  Работает и не падает.

В 2.6, кстати, такая же беда.

Так что можно и в bugzilla.kernel.org, вместе с патчем.

Rgds,
Rider