From: Tejun Heo <tj-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
To: Aleksa Sarai <cyphar-gVpy/LI/lHzQT0dZR+AlfA@public.gmane.org>
Cc: lizefan-hv44wF8Li93QT0dZR+AlfA@public.gmane.org,
mingo-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org,
peterz-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org,
richard-/L3Ra7n9ekc@public.gmane.org,
"Frédéric Weisbecker"
<fweisbec-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
cgroups-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH v9 3/4] cgroups: allow a cgroup subsystem to reject a fork
Date: Tue, 14 Apr 2015 08:17:32 -0400 [thread overview]
Message-ID: <20150414121732.GL2596@htj.duckdns.org> (raw)
In-Reply-To: <CAOviyajqc_ip2Lpa5NTLLXxQ7JghYXg1UL3vXDdpbbnhmsfXEQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
Hello,
On Tue, Apr 14, 2015 at 10:33:41AM +1000, Aleksa Sarai wrote:
> Hi Tejun,
>
> > Ummm... is this really necessary? Can't we do something like the
> > following?
> >
> > #define SUBSYS_TAG(tag) \
> > __##tag, \
> > tag = __##tag - 2,
>
> Is which part necessary? That's what TAG() does. I can remove the
The complex macros.
> UNUSED_IDENT() if you want, that was just to make sure you don't get clashes
> with other unused idents. COUNT() is so that we don't have to manually add
> CGROUP_<TAG>_COUNT for every tag we add.
Let's please do the simple thing. It just takes one two-liner macro.
Thanks.
--
tejun
next prev parent reply other threads:[~2015-04-14 12:17 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-12 0:51 [PATCH v9 0/4] cgroups: add pids subsystem Aleksa Sarai
[not found] ` <1428799917-15141-1-git-send-email-cyphar-gVpy/LI/lHzQT0dZR+AlfA@public.gmane.org>
2015-04-12 0:51 ` [PATCH v9 1/4] cgroups: use bitmask to filter for_each_subsys Aleksa Sarai
2015-04-12 0:51 ` [PATCH v9 3/4] cgroups: allow a cgroup subsystem to reject a fork Aleksa Sarai
[not found] ` <1428799917-15141-4-git-send-email-cyphar-gVpy/LI/lHzQT0dZR+AlfA@public.gmane.org>
2015-04-13 14:24 ` Tejun Heo
2015-04-14 0:33 ` Aleksa Sarai
[not found] ` <CAOviyajqc_ip2Lpa5NTLLXxQ7JghYXg1UL3vXDdpbbnhmsfXEQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-04-14 12:17 ` Tejun Heo [this message]
[not found] ` <20150414121732.GL2596-piEFEHQLUPpN0TnZuCh8vA@public.gmane.org>
2015-04-16 13:57 ` Aleksa Sarai
[not found] ` <CAOviyaj6eQ6s50+DrQN8yFAp7q5Y6b4ReFOT8XT0UufPb5J0xQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-04-17 14:45 ` Tejun Heo
[not found] ` <20150417144519.GC1954-piEFEHQLUPpN0TnZuCh8vA@public.gmane.org>
2015-04-17 20:35 ` Aleksa Sarai
2015-04-17 20:42 ` Tejun Heo
[not found] ` <20150417204206.GI16743-piEFEHQLUPpN0TnZuCh8vA@public.gmane.org>
2015-04-17 20:48 ` Aleksa Sarai
[not found] ` <CAOviyah2dBbc5MecxzCfOt36dVfCaN7F7u+5ycPmp3eVDc_J2Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-04-17 20:55 ` Tejun Heo
[not found] ` <20150417205545.GJ16743-piEFEHQLUPpN0TnZuCh8vA@public.gmane.org>
2015-04-17 21:06 ` Aleksa Sarai
2015-04-12 0:51 ` [PATCH v9 2/4] cgroups: replace explicit ss_mask checking with for_each_subsys_which Aleksa Sarai
2015-04-12 0:51 ` [PATCH v9 4/4] cgroups: implement the PIDs subsystem Aleksa Sarai
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=20150414121732.GL2596@htj.duckdns.org \
--to=tj-dgejt+ai2ygdnm+yrofe0a@public.gmane.org \
--cc=cgroups-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=cyphar-gVpy/LI/lHzQT0dZR+AlfA@public.gmane.org \
--cc=fweisbec-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=lizefan-hv44wF8Li93QT0dZR+AlfA@public.gmane.org \
--cc=mingo-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
--cc=peterz-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org \
--cc=richard-/L3Ra7n9ekc@public.gmane.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).