From patchwork Fri Feb 15 20:14:15 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Paul Burton X-Patchwork-Id: 1043159 X-Patchwork-Delegate: bpf@iogearbox.net Return-Path: X-Original-To: patchwork-incoming-netdev@ozlabs.org Delivered-To: patchwork-incoming-netdev@ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=netdev-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dmarc=none (p=none dis=none) header.from=mips.com Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=wavesemi.onmicrosoft.com header.i=@wavesemi.onmicrosoft.com header.b="MkIvVyp+"; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 441PcS6PyPz9sDr for ; Sat, 16 Feb 2019 07:14:32 +1100 (AEDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2390908AbfBOUOU (ORCPT ); Fri, 15 Feb 2019 15:14:20 -0500 Received: from mail-eopbgr700091.outbound.protection.outlook.com ([40.107.70.91]:40096 "EHLO NAM04-SN1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1726317AbfBOUOT (ORCPT ); Fri, 15 Feb 2019 15:14:19 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=wavesemi.onmicrosoft.com; s=selector1-wavecomp-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=rjHFBc2hzhSQnUBNbmFr9Y1oTt1CX+dU/cSyNASEng4=; b=MkIvVyp+SNkULW67gtDNawBkrGR+ETpXecl7wGdGfn9WJxPYhXbBlyqeIbXqUy20kWyJBqtw4Fm7CIvxtOZnq+yCnmsTsgCoahXJahl9WxvCelafyaWgAgffqwoMNnzV2C8nzkAg6xv7yY/wmHNI368CGtaJRwDgTkyyMA4C5gI= Received: from MWHPR2201MB1277.namprd22.prod.outlook.com (10.174.162.17) by MWHPR2201MB1582.namprd22.prod.outlook.com (10.174.167.31) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1622.16; Fri, 15 Feb 2019 20:14:15 +0000 Received: from MWHPR2201MB1277.namprd22.prod.outlook.com ([fe80::7d5e:f3b0:4a5:4636]) by MWHPR2201MB1277.namprd22.prod.outlook.com ([fe80::7d5e:f3b0:4a5:4636%9]) with mapi id 15.20.1601.023; Fri, 15 Feb 2019 20:14:15 +0000 From: Paul Burton To: "linux-mips@vger.kernel.org" , Alexei Starovoitov , Daniel Borkmann , Martin KaFai Lau , Song Liu , Yonghong Song , "netdev@vger.kernel.org" CC: "linux-kernel@vger.kernel.org" , Jiong Wang , Paul Burton , "stable@vger.kernel.org" Subject: [PATCH 1/2] MIPS: eBPF: Always return sign extended 32b values Thread-Topic: [PATCH 1/2] MIPS: eBPF: Always return sign extended 32b values Thread-Index: AQHUxWsFAaDSuQE4g0KueZPNLuu8Cg== Date: Fri, 15 Feb 2019 20:14:15 +0000 Message-ID: <20190215201321.15725-1-paul.burton@mips.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-clientproxiedby: BYAPR07CA0036.namprd07.prod.outlook.com (2603:10b6:a02:bc::49) To MWHPR2201MB1277.namprd22.prod.outlook.com (2603:10b6:301:24::17) authentication-results: spf=none (sender IP is ) smtp.mailfrom=pburton@wavecomp.com; x-ms-exchange-messagesentrepresentingtype: 1 x-mailer: git-send-email 2.20.1 x-originating-ip: [67.207.99.198] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 3349ca96-d610-4e11-c451-08d693822844 x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600110)(711020)(4605077)(2017052603328)(7153060)(7193020); SRVR:MWHPR2201MB1582; x-ms-traffictypediagnostic: MWHPR2201MB1582: x-microsoft-exchange-diagnostics: =?iso-8859-1?q?1=3BMWHPR2201MB1582=3B23?= =?iso-8859-1?q?=3AFtSurcM6FU1QDqlkIFwTqueMLDSgAyDJzPrmu2xKrQUttEn5?= =?iso-8859-1?q?ab8iS6UQB6wpDuhAowVOUJk0eE2N3iP0ovLK2ZLK/lCH+u1bZ8L?= =?iso-8859-1?q?VwgwKXTrAq6+wYW9lUohOpVl6pg2gbhwreZ5twNya1baLWIaGQY?= =?iso-8859-1?q?O1LUUQiQmZVkkrSsdnwA4WEf3YgfJSDiORLE0bZCjmPecZIoz4J?= =?iso-8859-1?q?8oeM+imkf6P4/SpIsUzwNHue3qMlEBJgt8crmByT8pkrY2RpBEG?= =?iso-8859-1?q?yN6lHjzcdgwuAk+i44LcOUxWvdt0u3NFFdHmNBdKZH8JZsoZQ+W?= =?iso-8859-1?q?3vTZtkFjICoiEbFxmewIeHXy3ROPMJ+sVawkTDfbYuAUOLrnI1C?= =?iso-8859-1?q?JImbJot7e6dWjcrJ/VtDuOVIRs/JUCgVVo/Su0yhWdcuhiRsyNb?= =?iso-8859-1?q?GCJL389SxsBHedDt4XjBb07xPLr8fgljJlaQIJJNgLUGvWxLX6q?= =?iso-8859-1?q?ljpaj24+06bzr5e9hoK9mJ14Z+FfnWCSkkOhgg89rAqvK6jH108?= =?iso-8859-1?q?NW/QAsupu9L+J6417XWDwUVVD5UW6Fk0PSVsDUxijf2zvNKEaSc?= =?iso-8859-1?q?d/vy84qUSktfytcTNmy9KB4zbV/5W9p6ae6LUHwnqhZT0oicfBH?= =?iso-8859-1?q?bkdIXrTD2Kdi1Eu4UA5Is8IRwa5AAnZgmFZTT1icJ6HCxMUX57h?= =?iso-8859-1?q?pm+GikFdhiaTy3ffIiu7tIn2BYIEy43NhohF0qdSnLRF4GlH8qV?= =?iso-8859-1?q?gUOK0i8bLQezVvxN7w+BFTc0jSRnB32ntIu8q82F47c+D50iV1k?= =?iso-8859-1?q?hbJo+qyr6DWnS7rPZpeDXii4sshlxBNqFCesOiKMpYc8BVEyThl?= =?iso-8859-1?q?2ypwJ70JjuO6LTrIRHeXj7V9RyoCsZGPRhOxU0CagZiVPsAM/CO?= =?iso-8859-1?q?/O3jB97ar42ZjkDdV5BX1fFzB11mh0p/G2t9ZuMubVZw6sdj+2B?= =?iso-8859-1?q?WbTRLwropwYQJHVcLHqzfb8YIgsZ0T63lFDJbTvJFFXiGiaJBl+?= =?iso-8859-1?q?gueUCLAX3q4xKzR3dnGm0R7k3zmnUnljMwRtxoYWswJhUwMWLt9?= =?iso-8859-1?q?jFXo0v0MDSBaDmuSoSnM0OR2H8CUSvAQG//GDuuSS7WXFNMxqLG?= =?iso-8859-1?q?loZK/dnPV3SequqpDMGlbuO2ONEBwD4/LjovTq/fMeLzVde4C3+?= =?iso-8859-1?q?kjcIfaYhWNB+M/BlUNmKrIJ7MDsXq9megoiKuuomDNPHjoscDmm?= =?iso-8859-1?q?lKyEzrBdq1EK54DFYtLtX2XMQGBbCxbECk3uJy5ntOqopP?= x-microsoft-antispam-prvs: x-forefront-prvs: 09497C15EB x-forefront-antispam-report: SFV:NSPM; SFS:(10019020)(136003)(346002)(376002)(39850400004)(396003)(366004)(199004)(189003)(71190400001)(256004)(186003)(66066001)(68736007)(50226002)(36756003)(486006)(102836004)(71200400001)(7416002)(386003)(6506007)(7736002)(305945005)(4326008)(316002)(1076003)(6116002)(3846002)(25786009)(44832011)(478600001)(2501003)(105586002)(97736004)(52116002)(42882007)(2906002)(476003)(53936002)(26005)(2616005)(110136005)(14444005)(8676002)(81156014)(99286004)(8936002)(6512007)(14454004)(54906003)(6436002)(106356001)(6486002)(81166006); DIR:OUT; SFP:1102; SCL:1; SRVR:MWHPR2201MB1582; H:MWHPR2201MB1277.namprd22.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1; received-spf: None (protection.outlook.com: wavecomp.com does not designate permitted sender hosts) x-ms-exchange-senderadcheck: 1 x-microsoft-antispam-message-info: BuiknC6m2558JmXBua/ANhh/TWg6bUr7TX4ISY9Wk93SsPBmQts9Ce+4FaO6PLbA+Si2zjmReYqB+zjdHUhJTJgPay+hDKULWg+0DQk6J3VzmGepn0VXm+14IEGdZO8vaZOEDLw0v/TXe4+DeOOqNVWeENnftPgUhbV8JAiliRIE5nO0qQi5a/+sy1OYSa6cMNzXBc6mv8o2XkegoOvwTpTSvejiCDHwCxdWa1esPLP0IRxrDbOBcYKaYSA5PBpRTsXb0UQ/yFmL5utQmasVwaiAqq2/3vN1YRzkTrCaSbDKwEGun52vEEDy7Y0FQMydP3YHeNMfs2eCehuk9EjVS7aO8Cdb6J985g9yVCbXHvVe0LF65ec44pJQVnQ+FCmam3q3b8I2RUoZ2yv3wGIr0ppDGC65IBcCqTjecDcZNTc= MIME-Version: 1.0 X-OriginatorOrg: mips.com X-MS-Exchange-CrossTenant-Network-Message-Id: 3349ca96-d610-4e11-c451-08d693822844 X-MS-Exchange-CrossTenant-originalarrivaltime: 15 Feb 2019 20:14:14.6567 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-id: 463607d3-1db3-40a0-8a29-970c56230104 X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR2201MB1582 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org The function prototype used to call JITed eBPF code (ie. the type of the struct bpf_prog bpf_func field) returns an unsigned int. The MIPS n64 ABI that MIPS64 kernels target defines that 32 bit integers should always be sign extended when passed in registers as either arguments or return values. This means that when returning any value which may not already be sign extended (ie. of type REG_64BIT or REG_32BIT_ZERO_EX) we need to perform that sign extension in order to comply with the n64 ABI. Without this we see strange looking test failures from test_bpf.ko, such as: test_bpf: #65 ALU64_MOV_X: dst = 4294967295 jited:1 ret -1 != -1 FAIL (1 times) Although the return value printed matches the expected value, this is only because printf is only examining the least significant 32 bits of the 64 bit register value we returned. The register holding the expected value is sign extended whilst the v0 register was set to a zero extended value by our JITed code, so when compared by a conditional branch instruction the values are not equal. We already handle this when the return value register is of type REG_32BIT_ZERO_EX, so simply extend this to also cover REG_64BIT. Signed-off-by: Paul Burton Fixes: b6bd53f9c4e8 ("MIPS: Add missing file for eBPF JIT.") Cc: stable@vger.kernel.org # v4.13+ --- arch/mips/net/ebpf_jit.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/arch/mips/net/ebpf_jit.c b/arch/mips/net/ebpf_jit.c index b16710a8a9e7..715415fa2345 100644 --- a/arch/mips/net/ebpf_jit.c +++ b/arch/mips/net/ebpf_jit.c @@ -343,12 +343,15 @@ static int build_int_epilogue(struct jit_ctx *ctx, int dest_reg) const struct bpf_prog *prog = ctx->skf; int stack_adjust = ctx->stack_size; int store_offset = stack_adjust - 8; + enum reg_val_type td; int r0 = MIPS_R_V0; - if (dest_reg == MIPS_R_RA && - get_reg_val_type(ctx, prog->len, BPF_REG_0) == REG_32BIT_ZERO_EX) + if (dest_reg == MIPS_R_RA) { /* Don't let zero extended value escape. */ - emit_instr(ctx, sll, r0, r0, 0); + td = get_reg_val_type(ctx, prog->len, BPF_REG_0); + if (td == REG_64BIT || td == REG_32BIT_ZERO_EX) + emit_instr(ctx, sll, r0, r0, 0); + } if (ctx->flags & EBPF_SAVE_RA) { emit_instr(ctx, ld, MIPS_R_RA, store_offset, MIPS_R_SP); From patchwork Fri Feb 15 20:14:16 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Paul Burton X-Patchwork-Id: 1043158 X-Patchwork-Delegate: bpf@iogearbox.net Return-Path: X-Original-To: patchwork-incoming-netdev@ozlabs.org Delivered-To: patchwork-incoming-netdev@ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=netdev-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dmarc=none (p=none dis=none) header.from=mips.com Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=wavesemi.onmicrosoft.com header.i=@wavesemi.onmicrosoft.com header.b="WRcajbpm"; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 441PcN0lkqz9sML for ; Sat, 16 Feb 2019 07:14:28 +1100 (AEDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2390960AbfBOUOW (ORCPT ); Fri, 15 Feb 2019 15:14:22 -0500 Received: from mail-eopbgr700091.outbound.protection.outlook.com ([40.107.70.91]:40096 "EHLO NAM04-SN1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S2390667AbfBOUOV (ORCPT ); Fri, 15 Feb 2019 15:14:21 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=wavesemi.onmicrosoft.com; s=selector1-wavecomp-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=dAYJvweJO52SnUzzJwO3VW/UkhT/zhfX0oytuWwkhAM=; b=WRcajbpmJQ8Dn34tpWhvxjSbvbOR4BW493O/wxGdHx882uTw8pq/PCwLE+iZRluJfb6kFICz2nejN99TY+yt5JFa9S9y+QQW6EzTs2jkx6r9xfxtL4GIHsa3XdL1WEVFoo8KuZlLtmIttqM4ZJ0cGv8ufKE9zkFMTMPwcGEZEwM= Received: from MWHPR2201MB1277.namprd22.prod.outlook.com (10.174.162.17) by MWHPR2201MB1582.namprd22.prod.outlook.com (10.174.167.31) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1622.16; Fri, 15 Feb 2019 20:14:16 +0000 Received: from MWHPR2201MB1277.namprd22.prod.outlook.com ([fe80::7d5e:f3b0:4a5:4636]) by MWHPR2201MB1277.namprd22.prod.outlook.com ([fe80::7d5e:f3b0:4a5:4636%9]) with mapi id 15.20.1601.023; Fri, 15 Feb 2019 20:14:16 +0000 From: Paul Burton To: "linux-mips@vger.kernel.org" , Alexei Starovoitov , Daniel Borkmann , Martin KaFai Lau , Song Liu , Yonghong Song , "netdev@vger.kernel.org" CC: "linux-kernel@vger.kernel.org" , Jiong Wang , Paul Burton Subject: [PATCH 2/2] MIPS: eBPF: Remove REG_32BIT_ZERO_EX Thread-Topic: [PATCH 2/2] MIPS: eBPF: Remove REG_32BIT_ZERO_EX Thread-Index: AQHUxWsGSv3U5Mub50SxgUEaRy+wZQ== Date: Fri, 15 Feb 2019 20:14:16 +0000 Message-ID: <20190215201321.15725-2-paul.burton@mips.com> References: <20190215201321.15725-1-paul.burton@mips.com> In-Reply-To: <20190215201321.15725-1-paul.burton@mips.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-clientproxiedby: BYAPR07CA0036.namprd07.prod.outlook.com (2603:10b6:a02:bc::49) To MWHPR2201MB1277.namprd22.prod.outlook.com (2603:10b6:301:24::17) authentication-results: spf=none (sender IP is ) smtp.mailfrom=pburton@wavecomp.com; x-ms-exchange-messagesentrepresentingtype: 1 x-mailer: git-send-email 2.20.1 x-originating-ip: [67.207.99.198] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: f86e0e67-8488-42c4-6deb-08d6938228b0 x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600110)(711020)(4605077)(2017052603328)(7153060)(7193020); SRVR:MWHPR2201MB1582; x-ms-traffictypediagnostic: MWHPR2201MB1582: x-microsoft-exchange-diagnostics: =?iso-8859-1?q?1=3BMWHPR2201MB1582=3B23?= =?iso-8859-1?q?=3A/8KPaz3cKnn8W5gtxx2Pu8gT5jtcPHoyNUGBzwMAwsBwYISr?= =?iso-8859-1?q?yXvzypaKxopw2pdI4/EJcvfvbsIK8PsKpeN+0SE4Ch1y2VZF3fy?= =?iso-8859-1?q?e1C+9sB1JTQaXXqwXSCQivB0yUtyGNGE9+rdqRlEmxef68NwWEE?= =?iso-8859-1?q?CHI2yzT2RuwzMWy7AbQUh/1aCBasJPadVWLOFlbR8IjelNFsO5R?= =?iso-8859-1?q?PWaaK7ziYqMMc0jnBHyfdy7pqzWzQUrY+sAwXe6LDCv2aVcdspH?= =?iso-8859-1?q?Y/PxrcLWC7RIWDdQSOIJb65sdUPAIWpSaqgbEQJAR6o0b7MPaBS?= =?iso-8859-1?q?da2OvftAtMjNJyezarwKoHj4zBaXbqxzSYVCk83413h2hT/8Ci9?= =?iso-8859-1?q?hdDt07uaR2jgxLFSnvD/cKWtMuVjX4W10vTLZoOmoeBvaiE0f5p?= =?iso-8859-1?q?TZM1vCg7EtZeJsgcC3L7rhiUTy9VYtONIIBMzwIl7PFoTb8JV7o?= =?iso-8859-1?q?Y/GDnMCea/OV3RUq7JKNACLaD7wqiF1oDQ+7E08klBVCbZJ8JfG?= =?iso-8859-1?q?kCjuHYLadKN+4uQTheKDG2IyNzrcOeo4l7D3I5r5PDYQJX/N3SO?= =?iso-8859-1?q?+bWGyBhoeNQ67klOk6eSxMgPtkMXZCS7dhRDUaJI8DlfCTnOg/Z?= =?iso-8859-1?q?1S+wp+PVBHMoqwKZ+qDCVQxOxCl6RUB8YWVYCAqqX0A0YU1uEDJ?= =?iso-8859-1?q?MVOWMnQWaI5dD5KsehMQuEv3mxsWecr/1/QBEu3p0qHwZH/fc7u?= =?iso-8859-1?q?RWZHpbKNpvT6tSt6eSJYBa9+BjPIWmVSAuKdcgyJ41Eiufb++W+?= =?iso-8859-1?q?IybBSqjwFb8XGQXdqDKu430pSQb1MBiaHDfzEaTscNG8huh2vQ+?= =?iso-8859-1?q?ma3tzXfUUf491Cg5oUiaGXA2BpGRs5H5rfiJljrOTF+SkEaaXCD?= =?iso-8859-1?q?3tj6Y+2WVHI/oWzkzLndtrMx/H6eijbXHB7AonmQKu0slnliwHt?= =?iso-8859-1?q?IJRRNykw0FF7ydgwB3bPfRVkgbL6t6N9JfamHetG+UziWI1U9nG?= =?iso-8859-1?q?eTN+7FfLjcZucrIhmZYK9sSBgD0GN0e+xd1QCVP9aNfIJhQllTK?= =?iso-8859-1?q?GiGs/YBasGTnHTyHdZF4nxYhJ6hZ+V1OZJpA712QRdnFaqm6fOB?= =?iso-8859-1?q?dhTp5jlNap1vM0RwNMvq5aL/v/WdYAWmVmO4Zg6KWK7heVXhokv?= =?iso-8859-1?q?aJtvTTRp6Av6K5iiAP3iETXBerTlnr3EUR11AZzqyxtSDQQDgFk?= =?iso-8859-1?q?yFCyfOLJqaWFVqBovE9vseIh+AbFRPydIG/KMzIzyWSUk/uDJCm?= =?iso-8859-1?q?hINb1XHOuVJykbAXBH2fhqlo10aYkswzQYJZtw=3D?= x-microsoft-antispam-prvs: x-forefront-prvs: 09497C15EB x-forefront-antispam-report: SFV:NSPM; SFS:(10019020)(136003)(346002)(376002)(39850400004)(396003)(366004)(199004)(189003)(71190400001)(76176011)(256004)(186003)(66066001)(68736007)(50226002)(36756003)(486006)(107886003)(102836004)(71200400001)(386003)(6506007)(7736002)(305945005)(4326008)(316002)(1076003)(6116002)(3846002)(25786009)(44832011)(11346002)(478600001)(2501003)(105586002)(97736004)(446003)(52116002)(42882007)(2906002)(476003)(53936002)(26005)(2616005)(110136005)(8676002)(81156014)(99286004)(8936002)(6512007)(14454004)(54906003)(6436002)(106356001)(6486002)(81166006); DIR:OUT; SFP:1102; SCL:1; SRVR:MWHPR2201MB1582; H:MWHPR2201MB1277.namprd22.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1; received-spf: None (protection.outlook.com: wavecomp.com does not designate permitted sender hosts) x-ms-exchange-senderadcheck: 1 x-microsoft-antispam-message-info: sYavKm3EnElS9JWUfsrN2gieQpdn0mUlQUDe7rQMmdPTTffVkVhmzPWprCMVtvMNouMnu9zxz3EtyDqB0kpXoiip32eyCGd1q5d2788cvhOeCTuyRQQNfcwYxtGLlFcNN4KNYDTj1OJtxEw3qRTM4iRbgRkpoR03nDt5Cj25KlmkdI92aDzanYjPdV7CrulqZCekuXEqBFwTi7uxJl2vsABTl2lUs3DzahTEkYy5W8D9I9wh9xcqzkzkzGh7jXCrvkDOIORAwlDQiitONrdW4GvgUMFaV9ffH6RTdR/I4RABf5MAZIO+uMMcGIiB5M5mP2e/m/EiiYzRJhgnao7s0LrE2tsDGiDB3Lmc46C8pE4Q2KGZT6NEG5UUxckzsxztUxZNMN+Fy7rsTeQePp9tRDDrwd0MAJMGHCQKaTLNx/g= MIME-Version: 1.0 X-OriginatorOrg: mips.com X-MS-Exchange-CrossTenant-Network-Message-Id: f86e0e67-8488-42c4-6deb-08d6938228b0 X-MS-Exchange-CrossTenant-originalarrivaltime: 15 Feb 2019 20:14:15.4013 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-id: 463607d3-1db3-40a0-8a29-970c56230104 X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR2201MB1582 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org REG_32BIT_ZERO_EX and REG_64BIT are always handled in exactly the same way, and reg_val_propagate_range() never actually sets any register to type REG_32BIT_ZERO_EX. Remove the redundant & unused REG_32BIT_ZERO_EX. Signed-off-by: Paul Burton --- arch/mips/net/ebpf_jit.c | 17 +++++++---------- 1 file changed, 7 insertions(+), 10 deletions(-) diff --git a/arch/mips/net/ebpf_jit.c b/arch/mips/net/ebpf_jit.c index 715415fa2345..76e9bf88d3b9 100644 --- a/arch/mips/net/ebpf_jit.c +++ b/arch/mips/net/ebpf_jit.c @@ -79,8 +79,6 @@ enum reg_val_type { REG_64BIT_32BIT, /* 32-bit compatible, need truncation for 64-bit ops. */ REG_32BIT, - /* 32-bit zero extended. */ - REG_32BIT_ZERO_EX, /* 32-bit no sign/zero extension needed. */ REG_32BIT_POS }; @@ -349,7 +347,7 @@ static int build_int_epilogue(struct jit_ctx *ctx, int dest_reg) if (dest_reg == MIPS_R_RA) { /* Don't let zero extended value escape. */ td = get_reg_val_type(ctx, prog->len, BPF_REG_0); - if (td == REG_64BIT || td == REG_32BIT_ZERO_EX) + if (td == REG_64BIT) emit_instr(ctx, sll, r0, r0, 0); } @@ -695,7 +693,7 @@ static int build_one_insn(const struct bpf_insn *insn, struct jit_ctx *ctx, if (dst < 0) return dst; td = get_reg_val_type(ctx, this_idx, insn->dst_reg); - if (td == REG_64BIT || td == REG_32BIT_ZERO_EX) { + if (td == REG_64BIT) { /* sign extend */ emit_instr(ctx, sll, dst, dst, 0); } @@ -710,7 +708,7 @@ static int build_one_insn(const struct bpf_insn *insn, struct jit_ctx *ctx, if (dst < 0) return dst; td = get_reg_val_type(ctx, this_idx, insn->dst_reg); - if (td == REG_64BIT || td == REG_32BIT_ZERO_EX) { + if (td == REG_64BIT) { /* sign extend */ emit_instr(ctx, sll, dst, dst, 0); } @@ -724,7 +722,7 @@ static int build_one_insn(const struct bpf_insn *insn, struct jit_ctx *ctx, if (dst < 0) return dst; td = get_reg_val_type(ctx, this_idx, insn->dst_reg); - if (td == REG_64BIT || td == REG_32BIT_ZERO_EX) + if (td == REG_64BIT) /* sign extend */ emit_instr(ctx, sll, dst, dst, 0); if (insn->imm == 1) { @@ -863,13 +861,13 @@ static int build_one_insn(const struct bpf_insn *insn, struct jit_ctx *ctx, if (src < 0 || dst < 0) return -EINVAL; td = get_reg_val_type(ctx, this_idx, insn->dst_reg); - if (td == REG_64BIT || td == REG_32BIT_ZERO_EX) { + if (td == REG_64BIT) { /* sign extend */ emit_instr(ctx, sll, dst, dst, 0); } did_move = false; ts = get_reg_val_type(ctx, this_idx, insn->src_reg); - if (ts == REG_64BIT || ts == REG_32BIT_ZERO_EX) { + if (ts == REG_64BIT) { int tmp_reg = MIPS_R_AT; if (bpf_op == BPF_MOV) { @@ -1257,8 +1255,7 @@ static int build_one_insn(const struct bpf_insn *insn, struct jit_ctx *ctx, if (insn->imm == 64 && td == REG_32BIT) emit_instr(ctx, dinsu, dst, MIPS_R_ZERO, 32, 32); - if (insn->imm != 64 && - (td == REG_64BIT || td == REG_32BIT_ZERO_EX)) { + if (insn->imm != 64 && td == REG_64BIT) { /* sign extend */ emit_instr(ctx, sll, dst, dst, 0); }