Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753085AbbELHbO (ORCPT ); Tue, 12 May 2015 03:31:14 -0400 Received: from a.ns.miles-group.at ([95.130.255.143]:65275 "EHLO radon.swed.at" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752731AbbELHbN (ORCPT ); Tue, 12 May 2015 03:31:13 -0400 Message-ID: <5551AC3D.3070100@nod.at> Date: Tue, 12 May 2015 09:31:09 +0200 From: Richard Weinberger User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Andrew Morton , Joe Perches CC: linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/2] vsprintf: Add support for userspace strings References: <1431286936-4333-1-git-send-email-richard@nod.at> <1431286936-4333-3-git-send-email-richard@nod.at> <1431288580.29257.29.camel@perches.com> <554FBB8F.3070204@nod.at> <1431288982.29257.31.camel@perches.com> <20150511162316.10df1028c6afae6f4a37747f@linux-foundation.org> In-Reply-To: <20150511162316.10df1028c6afae6f4a37747f@linux-foundation.org> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1518 Lines: 38 Am 12.05.2015 um 01:23 schrieb Andrew Morton: > On Sun, 10 May 2015 13:16:22 -0700 Joe Perches wrote: > >> On Sun, 2015-05-10 at 22:11 +0200, Richard Weinberger wrote: >>> Am 10.05.2015 um 22:09 schrieb Joe Perches: >>>> On Sun, 2015-05-10 at 21:42 +0200, Richard Weinberger wrote: >>>>> Add %pL format string to print userspace strings. >>>>> It works like %s but does copy_from_user() instead >>>>> of a memcpy(). >>>> >>>> I think this would be much simpler in a new >>>> function rather than complicating string() >>> >>> -ENOPATCH. >> >> It's your patch, I'm just commenting on it. >> >> I'm not sure there's much value in it. > > I can't say I'm terribly excited either. Are there any existing > callsites which might use this? The %p namespace is already pretty > crowded and if %pL is just for non-production developer debug use then > it can reasonably live outside the main kernel tree... My indention was to use it for two things: a) Debugging (at least I use such code from time to time when debugging strange stuff) b) Using this new format string in ftrace to print strings provided to syscalls instead of raw addresses. Masami showed me that I can use "perf probe" for for both a) and b). So let'S drop this patch. Thanks, //richard -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/