From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tejun Heo Subject: Re: [PATCH v3 7/8] cgroup: Assign subsystem IDs during compile time Date: Tue, 11 Sep 2012 14:27:20 -0700 Message-ID: <20120911212720.GD7677@google.com> References: <1347380774-9546-1-git-send-email-wagi@monom.org> <1347380774-9546-8-git-send-email-wagi@monom.org> <20120911210109.GZ7677@google.com> <504FA9D6.4050201@monom.org> Mime-Version: 1.0 Return-path: DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=sender:date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:in-reply-to:user-agent; bh=hJ7CQgb8LvQsre4AlwXBHOGQ75xcujKJSAgLZbPCkU8=; b=a3R3d4VWdVjCDq4jijZGn3zlGoKtEEgYbdI7yyPJl4zCxGPoSfK+ycNMslEJPduW8I lY49oAbKZDPQny2qsMlnjFY9TSxzgwkQ5ND/DxWFMnYAOZb+MLGv6UMu1Z6/xyLhObKE GLN7w5uJyczlNcbhLpgZfpZCZNBUfBpMvZ5a/aH8Ahiq5VdylDbsYwdtFn4312CaaxwB XoZf7RhxmbzSNpZt8CO22Kf25jPmrJLHDkVpKJHKYKWjTv/s6ETYvPCwkQ6LmfgUraBm u3/gS+0wiecKKHklWUSV+omkL0VpHc6h0PQgzz9VQ9wElF1o9CTKi8pGflruNYS5Lb9n +6og== Content-Disposition: inline In-Reply-To: <504FA9D6.4050201@monom.org> Sender: netdev-owner@vger.kernel.org List-ID: Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Daniel Wagner Cc: netdev@vger.kernel.org, cgroups@vger.kernel.org, Daniel Wagner , "David S. Miller" , Andrew Morton , Eric Dumazet , Gao feng , Glauber Costa , Jamal Hadi Salim , John Fastabend , Kamezawa Hiroyuki , Li Zefan , Neil Horman Hello, Daniel On Tue, Sep 11, 2012 at 11:15:02PM +0200, Daniel Wagner wrote: > If net_prio_subsys_id is changed to be an enum, then the compiler > will report an error: > > error: lvalue required as left operand of assignment > > that was the reason why I kept this change here. I think I just > don't get what you are trying to tell me. > > >Please separate these changes and explain them. > > I will do that as soon I figured out what you are telling me. Sorry about that. I was thinking that was a separate variable. Well, we can introduce a variable, change the id allocation and then swap it back to the constant, but that would be too much. Let's just try to explain it better. Thanks. -- tejun