netdev - Re: [PATCH]: Kill qdisc rcu freeing...

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]

Message-ID: <20080818120725.GB7393@ff.dom.local>
Date:	Mon, 18 Aug 2008 12:07:25 +0000
From:	Jarek Poplawski <jarkao2@...il.com>
To:	David Miller <davem@...emloft.net>
Cc:	netdev@...r.kernel.org, herbert@...dor.apana.org.au
Subject: Re: [PATCH]: Kill qdisc rcu freeing...

On Sun, Aug 17, 2008 at 10:34:35PM -0700, David Miller wrote:
> 
> Ok, I flipped the switch.  I've checked in the following to
> net-2.6 and it seems to pass a bunch of smoke tests I just
> did (flood pings while adding and deleting various qdisc
> setups, etc.)
> 
> Jarek, I think we can possibly now revert that locking change
> you had to make because destroy was outside of RTNL?

Yes, I think there are these two patches only to revert:

----------
commit d4766692e72422f3b0f0e9ac6773d92baad07d51
Author: Jarek Poplawski <jarkao2@...il.com>
Date:   Wed Aug 13 15:20:24 2008 -0700

    pkt_sched: Protect gen estimators under est_lock.

----------

commit 1cfa26661a85549063e369e2b40275eeaa7b923c
Author: Jarek Poplawski <jarkao2@...il.com>
Date:   Mon Aug 11 18:11:06 2008 -0700

    pkt_sched: Add BH protection for qdisc_stab_lock.

----------

I guess, I don't need to send any patches, but let me know if I'm
wrong.

Thanks,
Jarek P.

> 
> pkt_sched: No longer destroy qdiscs from RCU.
> 
> We can now kill them synchronously with all of the
> previous dev_deactivate() cures.
> 
> This makes netdev destruction and shutdown saner as
> the qdiscs hold references to the device.
> 
> Signed-off-by: David S. Miller <davem@...emloft.net>
> ---
...
--
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