Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 16 Jan 2002 01:41:10 +0000
From:      Ian Dowse <iedowse@maths.tcd.ie>
To:        Emiel Kollof <coolvibe@hackerheaven.org>
Cc:        John Baldwin <jhb@FreeBSD.org>, freebsd-current@FreeBSD.org
Subject:   Re: Netatalk broken in current? Lock order reversal? 
Message-ID:   <200201160141.aa95174@salmon.maths.tcd.ie>
In-Reply-To: Your message of "Wed, 16 Jan 2002 02:34:48 %2B0100." <20020116013448.GA584@laptop.hackerheaven.org> 

next in thread | previous in thread | raw e-mail | index | archive | help
In message <20020116013448.GA584@laptop.hackerheaven.org>, Emiel Kollof writes:
>
>Oh, on another note, is someone working at that netatalk breakage? Who
>do I have to discipline for that? :-)

Could you try the following patch in src/sys/netatalk? The problem
was caused by the -fno-common compiler option that was added to
the kernel build flags recently.

This compiles for me, but I haven't checked that it actually works.

Ian

Index: ddp_input.c
===================================================================
RCS file: /dump/FreeBSD-CVS/src/sys/netatalk/ddp_input.c,v
retrieving revision 1.12
diff -u -r1.12 ddp_input.c
--- ddp_input.c	13 Feb 2000 03:31:58 -0000	1.12
+++ ddp_input.c	16 Jan 2002 01:30:50 -0000
@@ -27,8 +27,6 @@
 static struct ddpstat	ddpstat;
 static struct route	forwro;
 
-const int atintrq1_present = 1, atintrq2_present = 1;
-
 static void     ddp_input(struct mbuf *, struct ifnet *, struct elaphdr *, int);
 
 /*
Index: ddp_usrreq.c
===================================================================
RCS file: /dump/FreeBSD-CVS/src/sys/netatalk/ddp_usrreq.c,v
retrieving revision 1.22
diff -u -r1.22 ddp_usrreq.c
--- ddp_usrreq.c	17 Nov 2001 03:07:08 -0000	1.22
+++ ddp_usrreq.c	16 Jan 2002 01:32:34 -0000
@@ -14,6 +14,7 @@
 #include <sys/protosw.h>
 #include <net/if.h>
 #include <net/route.h>
+#include <net/intrq.h>
 
 #include <netatalk/at.h>
 #include <netatalk/at_var.h>
@@ -547,6 +548,8 @@
 {
     atintrq1.ifq_maxlen = IFQ_MAXLEN;
     atintrq2.ifq_maxlen = IFQ_MAXLEN;
+    atintrq1_present = 1;
+    atintrq2_present = 1;
     mtx_init(&atintrq1.ifq_mtx, "at1_inq", MTX_DEF);
     mtx_init(&atintrq2.ifq_mtx, "at2_inq", MTX_DEF);
 }

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-current" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi? <200201160141.aa95174>