{"id":818717,"url":"http://patchwork.ozlabs.org/api/patches/818717/?format=json","web_url":"http://patchwork.ozlabs.org/project/netdev/patch/f27b1dfe-4ea2-5737-d3d5-21cb581d1927@solarflare.com/","project":{"id":7,"url":"http://patchwork.ozlabs.org/api/projects/7/?format=json","name":"Linux network development","link_name":"netdev","list_id":"netdev.vger.kernel.org","list_email":"netdev@vger.kernel.org","web_url":null,"scm_url":null,"webscm_url":null,"list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"","list_archive_url":null,"date":"2017-09-26T15:35:29","name":"[v2,net-next,2/2] bpf/verifier: improve disassembly of BPF_NEG instructions","commit_ref":null,"pull_url":null,"state":"accepted","archived":true,"hash":"18e3caf6d37155991e75e975604e1fed81dc5134","submitter":{"id":61177,"url":"http://patchwork.ozlabs.org/api/people/61177/?format=json","name":"Edward Cree","email":"ecree@solarflare.com"},"delegate":{"id":34,"url":"http://patchwork.ozlabs.org/api/users/34/?format=json","username":"davem","first_name":"David","last_name":"Miller","email":"davem@davemloft.net"},"mbox":"http://patchwork.ozlabs.org/project/netdev/patch/f27b1dfe-4ea2-5737-d3d5-21cb581d1927@solarflare.com/mbox/","series":[{"id":5167,"url":"http://patchwork.ozlabs.org/api/series/5167/?format=json","web_url":"http://patchwork.ozlabs.org/project/netdev/list/?series=5167","date":"2017-09-26T15:32:15","name":"bpf/verifier: disassembly improvements","version":2,"mbox":"http://patchwork.ozlabs.org/series/5167/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/818717/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/818717/checks/","tags":{},"related":[],"headers":{"Return-Path":"","X-Original-To":"patchwork-incoming@ozlabs.org","Delivered-To":"patchwork-incoming@ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=vger.kernel.org\n\t(client-ip=209.132.180.67; helo=vger.kernel.org;\n\tenvelope-from=netdev-owner@vger.kernel.org;\n\treceiver=)","Received":["from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 3y1lQm65Lfz9t3R\n\tfor ;\n\tWed, 27 Sep 2017 01:35:44 +1000 (AEST)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1031438AbdIZPfn (ORCPT );\n\tTue, 26 Sep 2017 11:35:43 -0400","from dispatch1-us1.ppe-hosted.com ([67.231.154.164]:37862 \"EHLO\n\tdispatch1-us1.ppe-hosted.com\" rhost-flags-OK-OK-OK-OK)\n\tby vger.kernel.org with ESMTP id S1031425AbdIZPfj (ORCPT\n\t); Tue, 26 Sep 2017 11:35:39 -0400","from pure.maildistiller.com (unknown [10.110.50.29])\n\tby dispatch1-us1.ppe-hosted.com (Proofpoint Essentials ESMTP Server)\n\twith ESMTP id 3A8E26008C; Tue, 26 Sep 2017 15:35:39 +0000 (UTC)","from mx4-us4.ppe-hosted.com (unknown [10.110.49.251])\n\tby pure.maildistiller.com (Proofpoint Essentials ESMTP Server) with\n\tESMTPS id 947DC6004C; Tue, 26 Sep 2017 15:35:38 +0000 (UTC)","from webmail.solarflare.com (uk.solarflare.com [193.34.186.16])\n\t(using TLSv1 with cipher ECDHE-RSA-AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby mx4-us4.ppe-hosted.com (Proofpoint Essentials ESMTP Server) with\n\tESMTPS id 9C13C600A2; Tue, 26 Sep 2017 15:35:38 +0000 (UTC)","from ec-desktop.uk.solarflarecom.com (10.17.20.45) by\n\tukex01.SolarFlarecom.com (10.17.10.4) with Microsoft SMTP Server\n\t(TLS) id 15.0.1044.25; Tue, 26 Sep 2017 16:35:33 +0100"],"X-Virus-Scanned":"Proofpoint Essentials engine","From":"Edward Cree ","Subject":"[PATCH v2 net-next 2/2] bpf/verifier: improve disassembly of BPF_NEG\n\tinstructions","To":"","References":"<52270348-67f1-4e7a-cd2f-9d611ae94064@solarflare.com>","CC":", ,\n\t, ","Message-ID":"","Date":"Tue, 26 Sep 2017 16:35:29 +0100","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101\n\tThunderbird/45.1.0","MIME-Version":"1.0","In-Reply-To":"<52270348-67f1-4e7a-cd2f-9d611ae94064@solarflare.com>","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"7bit","X-Originating-IP":"[10.17.20.45]","X-ClientProxiedBy":"ocex03.SolarFlarecom.com (10.20.40.36) To\n\tukex01.SolarFlarecom.com (10.17.10.4)","X-TM-AS-Product-Ver":"SMEX-11.0.0.1191-8.100.1062-23352.000","X-TM-AS-Result":"No--2.359300-0.000000-31","X-TM-AS-User-Approved-Sender":"Yes","X-TM-AS-User-Blocked-Sender":"No","X-MDID":"1506440139-d2be7ESIJDQ9","Sender":"netdev-owner@vger.kernel.org","Precedence":"bulk","List-ID":"","X-Mailing-List":"netdev@vger.kernel.org"},"content":"BPF_NEG takes only one operand, unlike the bulk of BPF_ALU[64] which are\n compound-assignments. So give it its own format in print_bpf_insn().\n\nSigned-off-by: Edward Cree \n---\n kernel/bpf/verifier.c | 5 +++++\n 1 file changed, 5 insertions(+)","diff":"diff --git a/kernel/bpf/verifier.c b/kernel/bpf/verifier.c\nindex 3aaa3262..04e0508 100644\n--- a/kernel/bpf/verifier.c\n+++ b/kernel/bpf/verifier.c\n@@ -344,6 +344,11 @@ static void print_bpf_insn(const struct bpf_verifier_env *env,\n \t\t\t\tverbose(\"BUG_alu64_%02x\\n\", insn->code);\n \t\t\telse\n \t\t\t\tprint_bpf_end_insn(env, insn);\n+\t\t} else if (BPF_OP(insn->code) == BPF_NEG) {\n+\t\t\tverbose(\"(%02x) r%d = %s-r%d\\n\",\n+\t\t\t\tinsn->code, insn->dst_reg,\n+\t\t\t\tclass == BPF_ALU ? \"(u32) \" : \"\",\n+\t\t\t\tinsn->dst_reg);\n \t\t} else if (BPF_SRC(insn->code) == BPF_X) {\n \t\t\tverbose(\"(%02x) %sr%d %s %sr%d\\n\",\n \t\t\t\tinsn->code, class == BPF_ALU ? \"(u32) \" : \"\",\n","prefixes":["v2","net-next","2/2"]}