Re: INFO: rcu detected stall in xfrm_hash_rebuild

2018-12-14 Thread David Miller
From: Florian Westphal Date: Fri, 14 Dec 2018 17:23:33 +0100 > Christophe Gouault wrote: >> The main use cases I have encountered and tried to address with the >> hash-based lookup were network operator use cases: >> - a lot of dynamic /32 <=> /32 policies (protecting GTP tunnels) >> - or a lot

Re: INFO: rcu detected stall in xfrm_hash_rebuild

2018-12-14 Thread Christophe Gouault
Le ven. 14 déc. 2018 à 17:23, Florian Westphal a écrit : > I won't send a patch to remove your work, at least not at this time. > > In case I'd do this removal (thresholds, hash table, or both) > i will make these tests to see how large the impact is. Perfect, thanks Christophe

Re: INFO: rcu detected stall in xfrm_hash_rebuild

2018-12-14 Thread Florian Westphal
Christophe Gouault wrote: > The main use cases I have encountered and tried to address with the > hash-based lookup were network operator use cases: > - a lot of dynamic /32 <=> /32 policies (protecting GTP tunnels) > - or a lot of dynamic policies with the same prefix lengths (e.g. /16 <=> /24) >

Re: INFO: rcu detected stall in xfrm_hash_rebuild

2018-12-14 Thread Christophe Gouault
Le ven. 14 déc. 2018 à 15:35, Florian Westphal a écrit : > Ok. An alternative would be to remove the support for > policy hash table thresholds (which decide what kinds of policies > go to exact table and which ones go into inexact ones), i.e. > partially revert 880a6fab8f6ba5b5abe59ea6 > ("xfrm:

Re: INFO: rcu detected stall in xfrm_hash_rebuild

2018-12-14 Thread Herbert Xu
On Fri, Dec 14, 2018 at 03:35:32PM +0100, Florian Westphal wrote: > > Ok. An alternative would be to remove the support for > policy hash table thresholds (which decide what kinds of policies > go to exact table and which ones go into inexact ones), i.e. > partially revert 880a6fab8f6ba5b5abe59ea6

Re: INFO: rcu detected stall in xfrm_hash_rebuild

2018-12-14 Thread Florian Westphal
Wolfgang Walter wrote: [ CCing Christophe ] > Am Montag, 10. Dezember 2018, 09:58:56 schrieb David Miller: > > From: Florian Westphal > > Date: Mon, 10 Dec 2018 13:47:24 +0100 > > > > > After recent tree conversion, we could probably make the exact policies > > > part of the 'inexact tree' (wh

Re: INFO: rcu detected stall in xfrm_hash_rebuild

2018-12-14 Thread Wolfgang Walter
Am Montag, 10. Dezember 2018, 09:58:56 schrieb David Miller: > From: Florian Westphal > Date: Mon, 10 Dec 2018 13:47:24 +0100 > > > After recent tree conversion, we could probably make the exact policies > > part of the 'inexact tree' (which would be renamed to 'policy tree' or > > some such). >

Re: INFO: rcu detected stall in xfrm_hash_rebuild

2018-12-10 Thread David Miller
From: Florian Westphal Date: Mon, 10 Dec 2018 13:47:24 +0100 > After recent tree conversion, we could probably make the exact policies > part of the 'inexact tree' (which would be renamed to 'policy tree' or > some such). > > Special-casing the exact policies made a lot of sense when we had > a

Re: INFO: rcu detected stall in xfrm_hash_rebuild

2018-12-10 Thread Florian Westphal
syzbot wrote: > Hello, > > syzbot found the following crash on: [..] > Workqueue: events xfrm_hash_rebuild Ignoring this report for a second -- I think it makes sense to see if we can just remove the entire hash table rebuild/resize code. After recent tree conversion, we could probably make th