From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Howells Subject: Re: [PATCH [try #4] 8/9] KEYS: Extend TIF_NOTIFY_RESUME to (almost) all architectures Date: Tue, 01 Sep 2009 18:26:04 +0100 Message-ID: <14644.1251825964@redhat.com> References: <20090901171649.GA16779@flint.arm.linux.org.uk> <20090901163231.14180.43426.stgit@warthog.procyon.org.uk> <20090901163307.14180.64201.stgit@warthog.procyon.org.uk> Return-path: In-Reply-To: <20090901171649.GA16779@flint.arm.linux.org.uk> Sender: linux-security-module-owner@vger.kernel.org To: Russell King Cc: dhowells@redhat.com, jmorris@namei.org, linux-security-module@vger.kernel.org, linux-arch@vger.kernel.org List-Id: linux-arch.vger.kernel.org Russell King wrote: > > @@ -707,4 +707,9 @@ do_notify_resume(struct pt_regs *regs, unsigned int thread_flags, int syscall) > > { > > if (thread_flags & _TIF_SIGPENDING) > > do_signal(¤t->blocked, regs, syscall); > > + > > + if (thread_info_flags & _TIF_NOTIFY_RESUME) { > > Shouldn't this be thread_flags ? Yes, thanks. Fixed. David From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com ([209.132.183.28]:64048 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751795AbZIAR1E (ORCPT ); Tue, 1 Sep 2009 13:27:04 -0400 From: David Howells In-Reply-To: <20090901171649.GA16779@flint.arm.linux.org.uk> References: <20090901171649.GA16779@flint.arm.linux.org.uk> <20090901163231.14180.43426.stgit@warthog.procyon.org.uk> <20090901163307.14180.64201.stgit@warthog.procyon.org.uk> Subject: Re: [PATCH [try #4] 8/9] KEYS: Extend TIF_NOTIFY_RESUME to (almost) all architectures Date: Tue, 01 Sep 2009 18:26:04 +0100 Message-ID: <14644.1251825964@redhat.com> Sender: linux-arch-owner@vger.kernel.org List-ID: To: Russell King Cc: dhowells@redhat.com, jmorris@namei.org, linux-security-module@vger.kernel.org, linux-arch@vger.kernel.org Message-ID: <20090901172604.hQKb7Dd5XPlpIoPqP9HEclZNJxtNKSNZNlG2YseKB4M@z> Russell King wrote: > > @@ -707,4 +707,9 @@ do_notify_resume(struct pt_regs *regs, unsigned int thread_flags, int syscall) > > { > > if (thread_flags & _TIF_SIGPENDING) > > do_signal(¤t->blocked, regs, syscall); > > + > > + if (thread_info_flags & _TIF_NOTIFY_RESUME) { > > Shouldn't this be thread_flags ? Yes, thanks. Fixed. David