From: David Rientjes <rientjes@google.com>
To: Vlastimil Babka <vbabka@suse.cz>
Cc: Rongwei Wang <rongwei.wang@linux.alibaba.com>,
Christoph Lameter <cl@linux.com>,
Joonsoo Kim <iamjoonsoo.kim@lge.com>,
Pekka Enberg <penberg@kernel.org>,
Hyeonggon Yoo <42.hyeyoo@gmail.com>,
Roman Gushchin <roman.gushchin@linux.dev>,
linux-mm@kvack.org,
Sebastian Andrzej Siewior <bigeasy@linutronix.de>,
Thomas Gleixner <tglx@linutronix.de>,
Mike Galbraith <efault@gmx.de>
Subject: Re: [PATCH 5/5] mm/slub: simplify __cmpxchg_double_slab() and slab_[un]lock()
Date: Sun, 14 Aug 2022 17:04:50 -0700 (PDT) [thread overview]
Message-ID: <82d7ef4e-92a0-318a-2939-67ffc1d99f03@google.com> (raw)
In-Reply-To: <20220812091426.18418-6-vbabka@suse.cz>
On Fri, 12 Aug 2022, Vlastimil Babka wrote:
> The PREEMPT_RT specific disabling of irqs in __cmpxchg_double_slab()
> (through slab_[un]lock()) makes it behave on PREEMPT_RT exactly as
> cmpxchg_double_slab() does, so it's simpler to just redirect the usage
> of the former to the latter on RT.
>
> That means we no longer need the slab_[un]lock() wrappers, so delete
> them and rename the current __slab_[un]lock() to slab_[un]lock().
>
> Signed-off-by: Vlastimil Babka <vbabka@suse.cz>
Acked-by: David Rientjes <rientjes@google.com>
next prev parent reply other threads:[~2022-08-15 0:04 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-08-12 9:14 [PATCH 0/5] fix validation races and cleanup locking Vlastimil Babka
2022-08-12 9:14 ` [PATCH 2/5] mm/slub: restrict sysfs validation to debug caches and make it safe Vlastimil Babka
2022-08-14 14:39 ` Hyeonggon Yoo
2022-08-23 16:39 ` Vlastimil Babka
2022-08-12 9:16 ` [PATCH 0/5] fix validation races and cleanup locking Vlastimil Babka
[not found] ` <20220812091426.18418-4-vbabka@suse.cz>
2022-08-14 14:54 ` [PATCH 3/5] mm/slub: remove slab_lock() usage for debug operations Hyeonggon Yoo
2022-08-15 0:04 ` David Rientjes
[not found] ` <20220812091426.18418-2-vbabka@suse.cz>
2022-08-14 13:42 ` [PATCH 1/5] mm/slub: move free_debug_processing() further Hyeonggon Yoo
2022-08-15 0:03 ` David Rientjes
[not found] ` <20220812091426.18418-6-vbabka@suse.cz>
2022-08-15 0:04 ` David Rientjes [this message]
[not found] ` <20220812091426.18418-5-vbabka@suse.cz>
2022-08-15 0:03 ` [PATCH 4/5] mm/slub: convert object_map_lock to non-raw spinlock David Rientjes
2022-08-15 12:53 ` Hyeonggon Yoo
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=82d7ef4e-92a0-318a-2939-67ffc1d99f03@google.com \
--to=rientjes@google.com \
--cc=42.hyeyoo@gmail.com \
--cc=bigeasy@linutronix.de \
--cc=cl@linux.com \
--cc=efault@gmx.de \
--cc=iamjoonsoo.kim@lge.com \
--cc=linux-mm@kvack.org \
--cc=penberg@kernel.org \
--cc=roman.gushchin@linux.dev \
--cc=rongwei.wang@linux.alibaba.com \
--cc=tglx@linutronix.de \
--cc=vbabka@suse.cz \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.