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

[Andrey Khavryuchenko <akhavr@kds.com.ua>]

[-- Attachment #1: Type: text/plain, Size: 1255 bytes --]

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> вокруг всего хозяйства...

Сделал.  Работает и не падает.

Патч приложен.  Каковы дальнейшие действия для того, чтобы он оказался в
сизифовых ядрах (и мне не нужно было бы раз за разом пересобирать для
себя)?


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: net/atm/br2684.c patch --]
[-- Type: text/x-patch, Size: 768 bytes --]

diff -ru kernel-source-2.4.25.orig/net/atm/br2684.c kernel-source-2.4.25/net/atm/br2684.c
--- kernel-source-2.4.25.orig/net/atm/br2684.c	
2004-04-01 14:17:51 +0300
+++ kernel-source-2.4.25/net/atm/br2684.c	
2004-04-01 15:05:58 +0300
@@ -559,11 +559,12 @@
 	atmvcc->push = br2684_push;
 	skb_queue_head_init(&copy);
 	skb_migrate(&atmvcc->sk->receive_queue, &copy);
-	while ((skb = skb_dequeue(&copy))) {
-		BRPRIV(skb->dev)->stats.rx_bytes -= skb->len;
-		BRPRIV(skb->dev)->stats.rx_packets--;
-		br2684_push(atmvcc, skb);
-	}
+	while ((skb = skb_dequeue(&copy))) 
+		if( skb->dev ) {
+			BRPRIV(skb->dev)->stats.rx_bytes -= skb->len;
+			BRPRIV(skb->dev)->stats.rx_packets--;
+			br2684_push(atmvcc, skb);
+		}
 	return 0;
     error:
 	write_unlock_irq(&devs_lock);

[-- Attachment #3: Type: text/plain, Size: 199 bytes --]


Теперь буду воевать дальше чтобы вся эта байда стартовала по hotplug.

-- 
Andrey V Khavryuchenko            http://www.kds.com.ua/
Silver Bullet Software Solutions  http://www.kds.com.ua/training/