From patchwork Wed Mar 15 12:52:56 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Fabio M. De Francesco" X-Patchwork-Id: 70212 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:5915:0:0:0:0:0 with SMTP id v21csp2321274wrd; Wed, 15 Mar 2023 06:13:38 -0700 (PDT) X-Google-Smtp-Source: AK7set9L+NIbfbK8PTWVyn2eu0+8hJRRW6/gYtVSBpcjpgKc2UUjHCjbwlpBupSeFZ6eHOz2hKCd X-Received: by 2002:a05:6870:d10b:b0:17a:d615:3969 with SMTP id e11-20020a056870d10b00b0017ad6153969mr489786oac.4.1678886017827; Wed, 15 Mar 2023 06:13:37 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1678886017; cv=none; d=google.com; s=arc-20160816; b=eDWKYG91Umc8ZrEFxTEYosg4q2gg2VEfWX2hlpNwMpyk7cfXOBSlE/4ys3im1RWpeV wmdEjBFnaKKjGwGIj9nwiU94Ihvc8Ci4RT9H27GtGuQhxj1bJnxKjCxvQTSDvJpHRw27 gMNsPKQTDLh9jZunjTmWqigvIQFTVyAUsym2BROXD8+PP2Yw/Cbf11QeQ3wBHHYpqksJ /5xsfgG5euaWN8si4OacaWg1ckdIpapqDFokgvBl1iXXUuyTDVu7iyoakRL6mZOYZJMo D0GqZ2YaJXbioKTx+wGDr1EimDtka2ztqgVVpakSM/SIQCY9K/BlE+dSsVxqPd/KdrO2 ufBA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=OmyRdtH3qiSO5QWPBLWz/8v2lIwyybKSmrlaokl//yI=; b=XYchR52vv4YFVAL+jvOP40c+l7oQD3v4blJbT1gAOV171WPhh4SbfOUQjpHx/fWYR1 MYIm8TlOfoa3cAHxNw6oUpL9hASPjodcZpBk/ghfsCgu1p9HCOh91/vJq0DEtl1SJSje N8e7Rji0OINhyr4w2AKpr6zliNiVl/lZ8yT9sA1mZ6v2saXUCiPTLU940pJaHCvvGX1I RjX4XpYWHYRMphQYzg648/64oSxA41wonn6aSdUJQg0eS8TSiXo7pKP22K6jziOQ9odn FqxRYVu+y6uOQH5tS6hbqQCtd5gaNwBTxRQvLtG1ejQXLCmLs8HPKguQQJfFMMNfED4v KKFw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=IKjEROzV; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id 23-20020a056870125700b00172a1cc90eesi3970753oao.169.2023.03.15.06.13.23; Wed, 15 Mar 2023 06:13:37 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=IKjEROzV; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233117AbjCOMxR (ORCPT + 99 others); Wed, 15 Mar 2023 08:53:17 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59816 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233105AbjCOMxK (ORCPT ); Wed, 15 Mar 2023 08:53:10 -0400 Received: from mail-wm1-x32c.google.com (mail-wm1-x32c.google.com [IPv6:2a00:1450:4864:20::32c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id F1CFA99268; Wed, 15 Mar 2023 05:53:03 -0700 (PDT) Received: by mail-wm1-x32c.google.com with SMTP id c8-20020a05600c0ac800b003ed2f97a63eso1015224wmr.3; Wed, 15 Mar 2023 05:53:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; t=1678884782; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=OmyRdtH3qiSO5QWPBLWz/8v2lIwyybKSmrlaokl//yI=; b=IKjEROzVMXRIH2jmXhG0/7c8b2IOHgUL6F+2j+hWDlKOJt/JE1CMxqsmVUaTSZ/ZII dKxHynYor0Co5ohMyb3jBPd3zTrvQxj0esnBH5bKxoki7lGTb2Pkw+/vx901t66bJvUM loxVftu+i7vhzZJi4xrspTeYOoe1S1Fms95gRioUAdgmxAhGxEFQQ3t2kw3Lox6vAFSR NL9aWBYFpdcq5c/DpZbhv6wECLhIiyxvF0A1juR88OjXaQAM2DEcGIudZwH3c1pLWPCj pTpj9gmfbneGj4wRu8aPNV6Kr5SIcTiI87VPALaqWD3/U818FxvX8qsZnfVBC/hs2hAE AeEQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1678884782; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=OmyRdtH3qiSO5QWPBLWz/8v2lIwyybKSmrlaokl//yI=; b=OT9O4AoJDAw6T4/8u3SUE1c4+2zBH+nKpe+rjF1mW6akYwdaz79C0b6bnohF1Xy0IM xudEcl3F+KFroX60sGj33QPjz7xP+kL5ctLt+by+RwAnEvM0X+8bc559bbCfDua4/m7r yHnxSGwTDWv3yW1/cKVCBTuPaJjJ+4pzK5IVVyZcvtCVTAz+kqx/ly6nyuhCFinJWQDI 153S8vgbqMpDi+6Y4T8xvMfKRfj9MyMzvwPsCVI4rdCUKkfbcr8lPROYHuWGDJZcaALM +eE1BarMK+iMhjFX/6mNNF6MfOvRyaSNbsBaG3Qw0wYtVdtNi3hyPh9Bv/Gfk4DxXiCL UF+w== X-Gm-Message-State: AO0yUKUTeho28MfYGct6VJW+II/SPGr0fmo/rcN6W4z7pSO6Gtz/sxz3 Jqt72ro58rSK4v6dReSQQXY= X-Received: by 2002:a05:600c:198f:b0:3ea:f6c4:5f26 with SMTP id t15-20020a05600c198f00b003eaf6c45f26mr17240294wmq.17.1678884782269; Wed, 15 Mar 2023 05:53:02 -0700 (PDT) Received: from localhost.localdomain (host-79-35-102-94.retail.telecomitalia.it. [79.35.102.94]) by smtp.gmail.com with ESMTPSA id k26-20020a7bc31a000000b003eb596cbc54sm1936761wmj.0.2023.03.15.05.52.59 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 15 Mar 2023 05:53:01 -0700 (PDT) From: "Fabio M. De Francesco" To: Luis Chamberlain , Jason Wessel , Daniel Thompson , Douglas Anderson , Josh Poimboeuf , Jiri Kosina , Miroslav Benes , Petr Mladek , Joe Lawrence , Chris Down , Nick Terrell , Nathan Chancellor , Nick Desaulniers , Tom Rix , linux-modules@vger.kernel.org, linux-kernel@vger.kernel.org, kgdb-bugreport@lists.sourceforge.net, live-patching@vger.kernel.org, bpf@vger.kernel.org, llvm@lists.linux.dev Cc: "Fabio M. De Francesco" , Piotr Gorski , Dmitry Torokhov , Stephen Boyd , Ira Weiny Subject: [PATCH] module/decompress: Never use kunmap() for local un-mappings Date: Wed, 15 Mar 2023 13:52:56 +0100 Message-Id: <20230315125256.22772-1-fmdefrancesco@gmail.com> X-Mailer: git-send-email 2.39.2 MIME-Version: 1.0 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM, RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1760439584814647492?= X-GMAIL-MSGID: =?utf-8?q?1760439584814647492?= Use kunmap_local() to unmap pages locally mapped with kmap_local_page(). kunmap_local() must be called on the kernel virtual address returned by kmap_local_page(), differently from how we use kunmap() which instead expects the mapped page as its argument. In module_zstd_decompress() we currently map with kmap_local_page() and unmap with kunmap(). This breaks the code and so it should be fixed. Cc: Piotr Gorski Cc: Dmitry Torokhov Cc: Luis Chamberlain Cc: Stephen Boyd Cc: Ira Weiny Fixes: 169a58ad824d ("module/decompress: Support zstd in-kernel decompression") Signed-off-by: Fabio M. De Francesco Reviewed-by: Ira Weiny Reviewed-by: Stephen Boyd Reviewed-by: Piotr Gorski Reviewed-by: Piotr Gorski --- kernel/module/decompress.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/module/decompress.c b/kernel/module/decompress.c index bb79ac1a6d8f..7ddc87bee274 100644 --- a/kernel/module/decompress.c +++ b/kernel/module/decompress.c @@ -267,7 +267,7 @@ static ssize_t module_zstd_decompress(struct load_info *info, zstd_dec.size = PAGE_SIZE; ret = zstd_decompress_stream(dstream, &zstd_dec, &zstd_buf); - kunmap(page); + kunmap_local(zstd_dec.dst); retval = zstd_get_error_code(ret); if (retval) break;