All of lore.kernel.org
 help / color / mirror / Atom feed
From: tj@kernel.org (Tejun Heo)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 07/16] cgroup: fix warning when building without any subsys
Date: Sat, 6 Oct 2012 11:14:51 +0900	[thread overview]
Message-ID: <20121006021451.GA2601@localhost> (raw)
In-Reply-To: <1349448930-23976-8-git-send-email-arnd@arndb.de>

On Fri, Oct 05, 2012 at 04:55:21PM +0200, Arnd Bergmann wrote:
> In a configuration where the base cgroup support is enabled but
> every single cgroup subsys is turned off, CGROUP_BUILTIN_SUBSYS_COUNT
> is zero, which causes the sanity check code in cgroup_load_subsys
> to trigger:
> 
>        BUG_ON(ss->subsys_id >= CGROUP_BUILTIN_SUBSYS_COUNT);
>        BUG_ON(subsys[ss->subsys_id] != ss);

CGROUP_BUILTIN_SUBSYS_COUNT is already gone in mainline.  I suppose
this is no longer necessary?

Thanks.

-- 
tejun

WARNING: multiple messages have this Message-ID (diff)
From: Tejun Heo <tj@kernel.org>
To: Arnd Bergmann <arnd@arndb.de>
Cc: linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, arm@kernel.org,
	Li Zefan <lizefan@huawei.com>, Ben Blum <bblum@andrew.cmu.edu>
Subject: Re: [PATCH 07/16] cgroup: fix warning when building without any subsys
Date: Sat, 6 Oct 2012 11:14:51 +0900	[thread overview]
Message-ID: <20121006021451.GA2601@localhost> (raw)
In-Reply-To: <1349448930-23976-8-git-send-email-arnd@arndb.de>

On Fri, Oct 05, 2012 at 04:55:21PM +0200, Arnd Bergmann wrote:
> In a configuration where the base cgroup support is enabled but
> every single cgroup subsys is turned off, CGROUP_BUILTIN_SUBSYS_COUNT
> is zero, which causes the sanity check code in cgroup_load_subsys
> to trigger:
> 
>        BUG_ON(ss->subsys_id >= CGROUP_BUILTIN_SUBSYS_COUNT);
>        BUG_ON(subsys[ss->subsys_id] != ss);

CGROUP_BUILTIN_SUBSYS_COUNT is already gone in mainline.  I suppose
this is no longer necessary?

Thanks.

-- 
tejun

  parent reply	other threads:[~2012-10-06  2:14 UTC|newest]

Thread overview: 83+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-10-05 14:55 [PATCH 00/16] ARM: mostly harmless gcc warnings Arnd Bergmann
2012-10-05 14:55 ` Arnd Bergmann
2012-10-05 14:55 ` [PATCH 01/16] ARM: warnings in arch/arm/include/asm/uaccess.h Arnd Bergmann
2012-10-05 14:55   ` Arnd Bergmann
2012-10-08  5:49   ` Greg Ungerer
2012-10-08  5:49     ` Greg Ungerer
2012-10-09 12:08     ` Arnd Bergmann
2012-10-09 12:08       ` Arnd Bergmann
2012-10-05 14:55 ` [PATCH 02/16] ARM: binfmt_flat: unused variable 'persistent' Arnd Bergmann
2012-10-05 14:55   ` Arnd Bergmann
2012-10-08  5:50   ` Greg Ungerer
2012-10-08  5:50     ` Greg Ungerer
2012-10-05 14:55 ` [PATCH 03/16] SCSI: ARM: ncr5380/oak uses no interrupts Arnd Bergmann
2012-10-05 14:55   ` Arnd Bergmann
2012-10-05 14:55   ` Arnd Bergmann
2012-10-05 14:55 ` [PATCH 04/16] SCSI: ARM: make fas216_dumpinfo function conditional Arnd Bergmann
2012-10-05 14:55   ` Arnd Bergmann
2012-10-05 14:55 ` [PATCH 05/16] vfs: bogus warnings in fs/namei.c Arnd Bergmann
2012-10-05 14:55   ` Arnd Bergmann
2012-10-08 11:51   ` Jan Kara
2012-10-08 11:51     ` Jan Kara
2012-10-09 12:27     ` Arnd Bergmann
2012-10-09 12:27       ` Arnd Bergmann
2012-10-09 13:07       ` Arnd Bergmann
2012-10-09 13:07         ` Arnd Bergmann
2012-10-09 13:43         ` Jan Kara
2012-10-09 13:43           ` Jan Kara
2012-10-11  4:37         ` Al Viro
2012-10-11  4:37           ` Al Viro
2012-10-11 13:20           ` [PATCH v2] " Arnd Bergmann
2012-10-11 13:20             ` Arnd Bergmann
2012-10-05 14:55 ` [PATCH 06/16] mm/slob: use min_t() to compare ARCH_SLAB_MINALIGN Arnd Bergmann
2012-10-05 14:55   ` Arnd Bergmann
2012-10-05 16:46   ` Christoph Lameter
2012-10-05 16:46     ` Christoph Lameter
2012-10-31  7:25     ` Pekka Enberg
2012-10-31  7:25       ` Pekka Enberg
2012-10-05 14:55 ` [PATCH 07/16] cgroup: fix warning when building without any subsys Arnd Bergmann
2012-10-05 14:55   ` Arnd Bergmann
2012-10-05 15:50   ` Ben Blum
2012-10-05 15:50     ` Ben Blum
2012-10-06  2:14   ` Tejun Heo [this message]
2012-10-06  2:14     ` Tejun Heo
2012-10-06  6:19     ` Arnd Bergmann
2012-10-06  6:19       ` Arnd Bergmann
2012-10-05 14:55 ` [PATCH 08/16] ipvs: fix ip_vs_set_timeout debug messages Arnd Bergmann
2012-10-05 14:55   ` Arnd Bergmann
2012-10-05 20:39   ` Julian Anastasov
2012-10-05 20:39     ` Julian Anastasov
2012-10-06  6:45     ` Arnd Bergmann
2012-10-06  6:45       ` Arnd Bergmann
2012-10-06  8:09       ` Julian Anastasov
2012-10-06  8:09         ` Julian Anastasov
2012-10-06  9:54         ` Arnd Bergmann
2012-10-06  9:54           ` Arnd Bergmann
2012-10-09  1:48           ` Simon Horman
2012-10-09  1:48             ` Simon Horman
2012-10-05 14:55 ` [PATCH 09/16] USB: EHCI: mark ehci_orion_conf_mbus_windows __devinit Arnd Bergmann
2012-10-05 14:55   ` Arnd Bergmann
2012-10-05 18:05   ` Alan Stern
2012-10-05 18:05     ` Alan Stern
2012-10-05 14:55 ` [PATCH 10/16] clk: don't mark clkdev_add_table as init Arnd Bergmann
2012-10-05 14:55   ` Arnd Bergmann
2012-10-05 14:55 ` [PATCH 11/16] pcmcia: sharpsl: don't discard sharpsl_pcmcia_ops Arnd Bergmann
2012-10-05 14:55   ` Arnd Bergmann
2012-10-05 14:55 ` [PATCH 12/16] video: mark nuc900fb_map_video_memory as __devinit Arnd Bergmann
2012-10-05 14:55   ` Arnd Bergmann
2012-10-05 14:55   ` Arnd Bergmann
2012-10-05 14:55 ` [PATCH 13/16] ARM: be really quiet when building with 'make -s' Arnd Bergmann
2012-10-05 14:55   ` Arnd Bergmann
2012-10-05 16:36   ` Nicolas Pitre
2012-10-05 16:36     ` Nicolas Pitre
2012-10-05 14:55 ` [PATCH 14/16] ARM: pxa: armcore: fix PCI PIO warnings Arnd Bergmann
2012-10-05 14:55   ` Arnd Bergmann
2012-10-05 15:32   ` Igor Grinberg
2012-10-05 15:32     ` Igor Grinberg
2012-10-05 14:55 ` [PATCH 15/16] spi/s3c64xx: use correct dma_transfer_direction type Arnd Bergmann
2012-10-05 14:55   ` Arnd Bergmann
2012-10-05 14:55   ` Arnd Bergmann
2012-10-05 14:55 ` [PATCH 16/16] ARM: pass -marm to gcc by default for both C and assembler Arnd Bergmann
2012-10-05 14:55   ` Arnd Bergmann
2012-10-05 18:42   ` Nicolas Pitre
2012-10-05 18:42     ` Nicolas Pitre

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=20121006021451.GA2601@localhost \
    --to=tj@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.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 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.