From mboxrd@z Thu Jan 1 00:00:00 1970 From: Johannes Weiner Subject: Re: [RESEND, PATCH 4/6] memcg: fix broken boolean expression Date: Mon, 9 Jan 2012 15:04:04 +0100 Message-ID: <20120109140404.GG3588@cmpxchg.org> References: <1325883472-5614-1-git-send-email-kirill@shutemov.name> <1325883472-5614-4-git-send-email-kirill@shutemov.name> Mime-Version: 1.0 Return-path: Content-Disposition: inline In-Reply-To: <1325883472-5614-4-git-send-email-kirill-oKw7cIdHH8eLwutG50LtGA@public.gmane.org> Sender: cgroups-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-ID: Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: "Kirill A. Shutemov" Cc: linux-mm-Bw31MaZKKs3YtjvyW6yDsg@public.gmane.org, cgroups-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, containers-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org, KAMEZAWA Hiroyuki , Balbir Singh , Michal Hocko , stable-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org On Fri, Jan 06, 2012 at 10:57:50PM +0200, Kirill A. Shutemov wrote: > From: "Kirill A. Shutemov" > > action != CPU_DEAD || action != CPU_DEAD_FROZEN is always true. > > Signed-off-by: Kirill A. Shutemov > Cc: I think you don't need to actually CC stable via email. If you include that tag, they will pick it up once the patch hits mainline. The changelog is too terse, doubly so for a patch that should go into stable. How is the code supposed to work? What are the consequences of the bug? From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx115.postini.com [74.125.245.115]) by kanga.kvack.org (Postfix) with SMTP id 989ED6B005C for ; Mon, 9 Jan 2012 09:04:14 -0500 (EST) Date: Mon, 9 Jan 2012 15:04:04 +0100 From: Johannes Weiner Subject: Re: [RESEND, PATCH 4/6] memcg: fix broken boolean expression Message-ID: <20120109140404.GG3588@cmpxchg.org> References: <1325883472-5614-1-git-send-email-kirill@shutemov.name> <1325883472-5614-4-git-send-email-kirill@shutemov.name> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1325883472-5614-4-git-send-email-kirill@shutemov.name> Sender: owner-linux-mm@kvack.org List-ID: To: "Kirill A. Shutemov" Cc: linux-mm@kvack.org, cgroups@vger.kernel.org, linux-kernel@vger.kernel.org, containers@lists.linux-foundation.org, KAMEZAWA Hiroyuki , Balbir Singh , Michal Hocko , stable@kernel.org On Fri, Jan 06, 2012 at 10:57:50PM +0200, Kirill A. Shutemov wrote: > From: "Kirill A. Shutemov" > > action != CPU_DEAD || action != CPU_DEAD_FROZEN is always true. > > Signed-off-by: Kirill A. Shutemov > Cc: I think you don't need to actually CC stable via email. If you include that tag, they will pick it up once the patch hits mainline. The changelog is too terse, doubly so for a patch that should go into stable. How is the code supposed to work? What are the consequences of the bug? -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: email@kvack.org From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755206Ab2AIOEP (ORCPT ); Mon, 9 Jan 2012 09:04:15 -0500 Received: from zene.cmpxchg.org ([85.214.230.12]:37208 "EHLO zene.cmpxchg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752726Ab2AIOEO (ORCPT ); Mon, 9 Jan 2012 09:04:14 -0500 Date: Mon, 9 Jan 2012 15:04:04 +0100 From: Johannes Weiner To: "Kirill A. Shutemov" Cc: linux-mm@kvack.org, cgroups@vger.kernel.org, linux-kernel@vger.kernel.org, containers@lists.linux-foundation.org, KAMEZAWA Hiroyuki , Balbir Singh , Michal Hocko , stable@kernel.org Subject: Re: [RESEND, PATCH 4/6] memcg: fix broken boolean expression Message-ID: <20120109140404.GG3588@cmpxchg.org> References: <1325883472-5614-1-git-send-email-kirill@shutemov.name> <1325883472-5614-4-git-send-email-kirill@shutemov.name> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1325883472-5614-4-git-send-email-kirill@shutemov.name> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Jan 06, 2012 at 10:57:50PM +0200, Kirill A. Shutemov wrote: > From: "Kirill A. Shutemov" > > action != CPU_DEAD || action != CPU_DEAD_FROZEN is always true. > > Signed-off-by: Kirill A. Shutemov > Cc: I think you don't need to actually CC stable via email. If you include that tag, they will pick it up once the patch hits mainline. The changelog is too terse, doubly so for a patch that should go into stable. How is the code supposed to work? What are the consequences of the bug?