Date: Wed, 23 Jan 2008 15:50:02 GMT From: wforms@Safe-mail.net To: freebsd-net@FreeBSD.org Subject: Re: kern/112654: [pcn] Kernel panic upon if_pcn module load on a Netfinity 5000 Message-ID: <200801231550.m0NFo2sg027992@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
The following reply was made to PR kern/112654; it has been noted by GNATS. From: wforms@Safe-mail.net To: bug-followup@FreeBSD.org Cc: marius@alchemy.franken.de, chuzzwassa@gmail.com Subject: Re: kern/112654: [pcn] Kernel panic upon if_pcn module load on a Netfinity 5000 Date: Wed, 23 Jan 2008 16:47:23 +0100 BINGO! Adding this latest nsphyter.c edition of Marius to his former patchset resulted = in a reliably working pcn driver. So whatever you tried there Marius, is the = key to the kernel panic problem I reported this PR for. The kernel I built = with your latest update now loads the if_pcn module without the kernel panic, = regardless of the RJ45 cable being connected or disconnected, and the pcn0 = adapter reliably recognizes the presence of the cable. The adapter worked fine = for basic network testings. I do not know how ready you consider the current code to be the final fix for = my PR, or if you need to incorporate the things we just learnt here into a = larger and more uniformly written patch before it can migrate to the src system. = I also do not know if a proper fix for FreeBSD 6 needs to be much different = than for FreeBSD 7, I only recognized that your RELENG_6 patchset introduced = this new nsphyter thing which did not exist in 6.x before. So I am uncertain = how much of this are you or the FreeBSD Team willing_to/can inject into RELENG_6 = about now. However, judging from your latest followup I assume that there may = not be necessarry to add nsphyter to RELENG_6 just now, because a simplified = fix implementing this latest reset function change of yours in the good old = ukphy section of the current 6.3-STABLE code might be perfectly good enough = for the time being. But this is not for me to decide, I simply mentioned this as a possible option. Anyway, GOOD JOB guys! Thank you! P.S.: That backtrace thing I should have given you in case your patch does = not work, is unknown for me. I assume it may be usefull to know this in the = future and I am happy to improve my knowledge, so if you can point me to a = page where I can learn what that is, I would appreciate it.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200801231550.m0NFo2sg027992>