Thread (12 messages) 12 messages, 4 authors, 2021-03-10

Re: [PATCH 1/9] memcg: accounting for allocations called with disabled BH

From: Roman Gushchin <hidden>
Date: 2021-03-10 19:09:21
Also in: linux-mm

On Wed, Mar 10, 2021 at 10:42:29AM +0100, Michal Hocko wrote:
On Tue 09-03-21 12:18:28, Roman Gushchin wrote:
quoted
On Tue, Mar 09, 2021 at 11:39:41AM -0800, Shakeel Butt wrote:
quoted
On Tue, Mar 9, 2021 at 12:04 AM Vasily Averin [off-list ref] wrote:
quoted
in_interrupt() check in memcg_kmem_bypass() is incorrect because
it does not allow to account memory allocation called from task context
with disabled BH, i.e. inside spin_lock_bh()/spin_unlock_bh() sections

Signed-off-by: Vasily Averin <redacted>
In that file in_interrupt() is used at other places too. Should we
change those too?
Yes, it seems so. Let me prepare a fix (it seems like most of them were
introduced by me).
Does this affect any existing in-tree users?
I'll double check this, but I doubt. We should fix it anyway, but I don't think
we need any stable backports.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help