linux-kernel - Re: net, phonet, rcu: rcu hang within gprs

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: <8427f661d1aa5707ee18596e812fd3c5@roundcube.remlab.net>
Date:	Fri, 25 Jul 2014 10:43:46 +0300
From:	Rémi Denis-Courmont <remi@...lab.net>
To:	Sasha Levin <sasha.levin@...cle.com>
Cc:	"Paul E. McKenney" <paulmck@...ux.vnet.ibm.com>,
	"David S. Miller" <davem@...emloft.net>,
	LKML <linux-kernel@...r.kernel.org>, <netdev@...r.kernel.org>,
	Dave Jones <davej@...hat.com>
Subject: Re: net, phonet, rcu: rcu hang within gprs_attach

    Hello,

Le 2014-07-25 01:19, Sasha Levin a écrit :
> This has reproduced couple of times, and has always originated from
> gprs_attach. I don't see any obvious
> issues with the code there, so I'm not sure if it's a fault of the
> phonet or the rcu code.

Beats me :-( Is it systematic or pseudo-random?

To be honest, I do not remember how well we tested that error path way 
back when. Nevertheless gprs_detach() does basically the same thing, in 
the non-error case, and it definitely worked well as far as I could 
tell.

Maybe Phonet should be dropped though. As much as I like to have my 
name in MAINTAINERS, I have not heard of any remaining users for 
up-to-date kernel versions. And with Broadcom shutting down what was 
Nokia Wireless Modems, I doubt we will see new users ever.

-- 
Rémi Denis-Courmont
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/