netdev - Re: neigh_periodic_timer expires too often

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 30 Jul 2009 17:33:01 +0300
From:	Luciano Coelho <luciano.coelho@...ia.com>
To:	ext Eric Dumazet <eric.dumazet@...il.com>
CC:	"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
	"Kristo Tero (Nokia-D/Tampere)" <Tero.Kristo@...ia.com>
Subject: Re: neigh_periodic_timer expires too often

Hi Eric,

Thanks for your very quick response!

ext Eric Dumazet wrote:
> You are mixing two different problems here.
>
> First patch you link (from Stephen Hemminger) was using deferrable timer for
> individual entry (function neigh_timer_handler()). Probably not your
> current concern.
>
> Second patch you link (from me) was using a workqueue (instead of a timer)
> for garbage collection of whole table.
>   

Yes, it was the second patch that I saw as the most efficient way to 
deal with the problem.  I mentioned the first one because we were 
considering using deferrable timers until we saw your workqueue patch. ;)

> I bet you are more intested about the garbage collection problem
> (function neigh_periodic_timer()) so I could try to resurrect
> my patch. I guess at that time Parag didnt replied to my mail...
>
> If nobody cared, patch was not considered for inclusion. Thats all.
>   

Indeed, the interest seemed to have died out.  But I was not sure 
whether there had been some other discussions elsewhere and the decision 
had been not to take that patch because of some real technical reasons.  
I'm glad that was not the case ;)

> So please test it and tell me/us you like it :)
>   

I applied your patch on top of the 2.6.28 I'm using and will start 
testing it now.  I can also try to test it on top of 2.6.31-rc4 plus 
wireless-testing stuff, if needed.

I'll tell you that I liked your patch in a moment (I actually already 
liked it, but I'll wait until I know it works before I tell you ;)

-- 
Cheers,
Luca.

--
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