{"id":816366,"url":"http://patchwork.ozlabs.org/api/patches/816366/?format=json","web_url":"http://patchwork.ozlabs.org/project/linuxppc-dev/patch/20170920162910.32053-8-sergey.senozhatsky@gmail.com/","project":{"id":2,"url":"http://patchwork.ozlabs.org/api/projects/2/?format=json","name":"Linux PPC development","link_name":"linuxppc-dev","list_id":"linuxppc-dev.lists.ozlabs.org","list_email":"linuxppc-dev@lists.ozlabs.org","web_url":"https://github.com/linuxppc/wiki/wiki","scm_url":"https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git","webscm_url":"https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git/","list_archive_url":"https://lore.kernel.org/linuxppc-dev/","list_archive_url_format":"https://lore.kernel.org/linuxppc-dev/{}/","commit_url_format":"https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git/commit/?id={}"},"msgid":"<20170920162910.32053-8-sergey.senozhatsky@gmail.com>","list_archive_url":"https://lore.kernel.org/linuxppc-dev/20170920162910.32053-8-sergey.senozhatsky@gmail.com/","date":"2017-09-20T16:29:09","name":"[RFC,v2,7/7] checkpatch: add pF/pf deprecation warning","commit_ref":null,"pull_url":null,"state":"not-applicable","archived":false,"hash":"c1115e7643a324540ae0b05ac7583b4e7c74768d","submitter":{"id":3905,"url":"http://patchwork.ozlabs.org/api/people/3905/?format=json","name":"Sergey Senozhatsky","email":"sergey.senozhatsky@gmail.com"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/linuxppc-dev/patch/20170920162910.32053-8-sergey.senozhatsky@gmail.com/mbox/","series":[{"id":4171,"url":"http://patchwork.ozlabs.org/api/series/4171/?format=json","web_url":"http://patchwork.ozlabs.org/project/linuxppc-dev/list/?series=4171","date":"2017-09-20T16:29:02","name":"printk/ia64/ppc64/parisc64: let's deprecate %pF/%pf printk specifiers","version":2,"mbox":"http://patchwork.ozlabs.org/series/4171/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/816366/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/816366/checks/","tags":{},"related":[],"headers":{"Return-Path":"","X-Original-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Delivered-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Received":["from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\t(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xy5Fz1sn5z9t2Q\n\tfor ;\n\tThu, 21 Sep 2017 02:45:27 +1000 (AEST)","from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\tby lists.ozlabs.org (Postfix) with ESMTP id 3xy5Fz0XpBzDqj8\n\tfor ;\n\tThu, 21 Sep 2017 02:45:27 +1000 (AEST)","from mail-pg0-x242.google.com (mail-pg0-x242.google.com\n\t[IPv6:2607:f8b0:400e:c05::242])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128\n\tbits)) (No client certificate requested)\n\tby lists.ozlabs.org (Postfix) with ESMTPS id 3xy4wH0rbhzDqYY\n\tfor ;\n\tThu, 21 Sep 2017 02:30:07 +1000 (AEST)","by mail-pg0-x242.google.com with SMTP id u18so1919413pgo.1\n\tfor ;\n\tWed, 20 Sep 2017 09:30:07 -0700 (PDT)","from localhost.localdomain ([121.137.63.184])\n\tby smtp.gmail.com with ESMTPSA id\n\tc185sm8890567pfb.57.2017.09.20.09.30.00\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tWed, 20 Sep 2017 09:30:04 -0700 (PDT)"],"Authentication-Results":["ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"g33OEuK6\"; dkim-atps=neutral","lists.ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"g33OEuK6\"; dkim-atps=neutral","ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=gmail.com\n\t(client-ip=2607:f8b0:400e:c05::242; helo=mail-pg0-x242.google.com;\n\tenvelope-from=sergey.senozhatsky@gmail.com; receiver=)","lists.ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"g33OEuK6\"; dkim-atps=neutral"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=from:to:cc:subject:date:message-id:in-reply-to:references;\n\tbh=HJqhUlNnWHdffPLzDjDRPKA3+7BArB5/r6DxGIZj5oI=;\n\tb=g33OEuK6nTtEpDQOIPbrRhPIn/I4HgD2NHO2mWXr3Q4fbDS0yC58g79e5XVh6pfII/\n\tSl6TatUiOsHhqn7s0Uo7NLmkbt+V+4uYZ9N4+/zcfOCsi9kjD4x1qPzeIbEl6YOTuK5r\n\tntqJkG+2GEaXMYJL7h1uQ1jAfdq2uxSra1h9Zj650pYR0aaDq9BNI7Dg4K1aDcNg0l1z\n\tvrVdc2kvTioAPArW/A/qZIfjWzdeyAPHxbjfwV2kR0pqHRZGBepFkot6seoR7nC9u2La\n\t04AWWWRqDyTiEuIEQH3mhlAUb6/vJZMExDPb2VSn41W1uEZzCK8WqIZHQBc9yuRjQ6cC\n\tVBfA==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to\n\t:references;\n\tbh=HJqhUlNnWHdffPLzDjDRPKA3+7BArB5/r6DxGIZj5oI=;\n\tb=Ue/wxaMqdGlq5iV1jngQ0GUBQSvNzcOxzdYYNzt1gfnOxFUOvS69hBThJx5+vK85CN\n\t16qkoyJSK0q0cq0CTU8QB5II1BKQ8nvZhMZ8coDErxTlFhG7CUkvvudb50M0REAZwyap\n\t4pLPzhc0k24TX+M6d0LGdcoegB2dmKUei7ayfYGR5g+/WJEZFa8Ucg50RT8b9Yr24UT0\n\t2hRduamGVaciyfdmtW9UBk5NpyZtH8jd49Zeyu05MU+VhzzGzUmoXqHADZaikEl21xgA\n\ttXf0mrLBB1mCpDgCrsHDZ+alqJtZyGqKv2dHaVeeJJEJ4rnga2XfyPAp3CyM/Bpibx46\n\t1Brg==","X-Gm-Message-State":"AHPjjUh+TB5e0+HkX83GdnfD3rny8fUA9hgohIAVdnaPtAd9TCttBlmy\n\tdPsdHZXNEhu9wIg4gm7KSbw=","X-Google-Smtp-Source":"AOwi7QDFueRMZKFGzhb2953NehyciXW4SyKQCVnC2ZD58b12+vb/xEkXNNtIeP6AR0LqOv5K7lFGXQ==","X-Received":"by 10.98.12.84 with SMTP id u81mr2735453pfi.93.1505925005415;\n\tWed, 20 Sep 2017 09:30:05 -0700 (PDT)","From":"Sergey Senozhatsky ","To":"Tony Luck , Fenghua Yu ,\n\tBenjamin Herrenschmidt ,\n\tPaul Mackerras ,\n\tMichael Ellerman , \n\tJames Bottomley , Helge Deller ","Subject":"[RFC][PATCH v2 7/7] checkpatch: add pF/pf deprecation warning","Date":"Thu, 21 Sep 2017 01:29:09 +0900","Message-Id":"<20170920162910.32053-8-sergey.senozhatsky@gmail.com>","X-Mailer":"git-send-email 2.14.1","In-Reply-To":"<20170920162910.32053-1-sergey.senozhatsky@gmail.com>","References":"<20170920162910.32053-1-sergey.senozhatsky@gmail.com>","X-BeenThere":"linuxppc-dev@lists.ozlabs.org","X-Mailman-Version":"2.1.24","Precedence":"list","List-Id":"Linux on PowerPC Developers Mail List\n\t","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"Petr Mladek , linux-ia64@vger.kernel.org,\n\tlinux-parisc@vger.kernel.org, Alexei Starovoitov ,\n\tSteven Rostedt , linux-kernel@vger.kernel.org,\n\tSergey Senozhatsky ,\n\tJessica Yu , Andy Whitcroft ,\n\tJoe Perches ,\n\tAndrew Morton , linuxppc-dev@lists.ozlabs.org","Errors-To":"linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org","Sender":"\"Linuxppc-dev\"\n\t"},"content":"We deprecated '%pF/%pf' printk specifiers, since '%pS/%ps' is now smart\nenough to handle function pointer dereference on platforms where such\ndereference is required.\n\ncheckpatch warning example:\n\nWARNING: Use '%pS/%ps' instead. This pointer extension was deprecated: '%pF'\n\nSigned-off-by: Sergey Senozhatsky \nCc: Andy Whitcroft \nCc: Joe Perches \n---\n scripts/checkpatch.pl | 6 ++++--\n 1 file changed, 4 insertions(+), 2 deletions(-)","diff":"diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl\nindex dd2c262aebbf..5945e4843466 100755\n--- a/scripts/checkpatch.pl\n+++ b/scripts/checkpatch.pl\n@@ -5762,18 +5762,20 @@ sub process {\n \t\t for (my $count = $linenr; $count <= $lc; $count++) {\n \t\t\t\tmy $fmt = get_quoted_string($lines[$count - 1], raw_line($count, 0));\n \t\t\t\t$fmt =~ s/%%//g;\n-\t\t\t\tif ($fmt =~ /(\\%[\\*\\d\\.]*p(?![\\WFfSsBKRraEhMmIiUDdgVCbGNO]).)/) {\n+\t\t\t\tif ($fmt =~ /(\\%[\\*\\d\\.]*p(?![\\WSsBKRraEhMmIiUDdgVCbGNO]).)/) {\n \t\t\t\t\t$bad_extension = $1;\n \t\t\t\t\tlast;\n \t\t\t\t}\n \t\t\t}\n \t\t\tif ($bad_extension ne \"\") {\n \t\t\t\tmy $stat_real = raw_line($linenr, 0);\n+\t\t\t\tmy $error_msg = \"Invalid vsprintf pointer extension \";\n \t\t\t\tfor (my $count = $linenr + 1; $count <= $lc; $count++) {\n \t\t\t\t\t$stat_real = $stat_real . \"\\n\" . raw_line($count, 0);\n \t\t\t\t}\n+\t\t\t\t$error_msg = \"Use '%pS/%ps' instead. This pointer extension was deprecated:\" if ($bad_extension =~ /pF|pf/);\n \t\t\t\tWARN(\"VSPRINTF_POINTER_EXTENSION\",\n-\t\t\t\t \"Invalid vsprintf pointer extension '$bad_extension'\\n\" . \"$here\\n$stat_real\\n\");\n+\t\t\t\t \"$error_msg '$bad_extension'\\n\" . \"$here\\n$stat_real\\n\");\n \t\t\t}\n \t\t}\n \n","prefixes":["RFC","v2","7/7"]}