lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 22 Jul 2008 12:10:02 -0700 From: Ben Greear <greearb@...delatech.com> To: Stephen Hemminger <shemminger@...tta.com> CC: David Miller <davem@...emloft.net>, netdev@...r.kernel.org, Patrick McHardy <kaber@...sh.net> Subject: Re: [PATCH 1/6] ipv6: use timer pending Stephen Hemminger wrote: > On Mon, 21 Jul 2008 16:16:15 -0700 > Ben Greear <greearb@...delatech.com> wrote: > >> Stephen Hemminger wrote: >>> This fixes the bridge reference count problem and cleanups ipv6 FIB timer management. >>> Don't use expires field, because it is not a proper way to test, >>> instead use timer_pending(). >> I patched this into my 2.6.25, and it does not seem to help >> my refcount problem. I did not apply any of your other >> patches as it seemed they were mostly cleanups. >> >> This is not to say that this patch is wrong...but >> it seems we'll have to keep looking for other problems >> as well. > > I agree this patch is not a direct fix for the the problem. What happened > was that I was retesting on latest 2.6 net kernel and adding tracing > and checking. In the process, the problem went away. Either > a Heisenbug or some other part of the configuration changed. Patrick sent me a debugging patch and we're poking at the problem. Currently, it seems we are leaking in6_dev refcounts...possibly due to a timer leak, but it's tricky code (for me, at least). I'm trying to add more debugging now, as I have a test case that easily reproduces the problem. Thanks, Ben -- Ben Greear <greearb@...delatech.com> Candela Technologies Inc http://www.candelatech.com -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists