From mboxrd@z Thu Jan 1 00:00:00 1970 From: Steven Rostedt Subject: Re: [PATCH V11 2/5] vsprintf: refactor %pK code out of pointer() Date: Tue, 28 Nov 2017 21:39:57 -0500 Message-ID: <20171128213957.2edf5069@gandalf.local.home> References: <1511921105-3647-1-git-send-email-me@tobin.cc> <1511921105-3647-3-git-send-email-me@tobin.cc> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: kernel-hardening@lists.openwall.com, Linus Torvalds , "Jason A. Donenfeld" , Theodore Ts'o , Kees Cook , Paolo Bonzini , Tycho Andersen , "Roberts, William C" , Tejun Heo , Jordan Glover , Greg KH , Petr Mladek , Joe Perches , Ian Campbell , Sergey Senozhatsky , Catalin Marinas , Will Deacon , Chris Fries , Dave Weinstein , Daniel Micay Return-path: In-Reply-To: <1511921105-3647-3-git-send-email-me@tobin.cc> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Wed, 29 Nov 2017 13:05:02 +1100 "Tobin C. Harding" wrote: > + /* > + * kptr_restrict==1 cannot be used in IRQ context > + * because its test for CAP_SYSLOG would be meaningless. > + */ > + if (in_irq() || in_serving_softirq() || in_nmi()) This could be replaced with: if (!in_task()) Which is actually more efficient. -- Steve > + return string(buf, end, "pK-error", spec);