From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.ozlabs.org (lists.ozlabs.org [112.213.38.117]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 7C551C4345F for ; Tue, 30 Apr 2024 12:42:18 +0000 (UTC) Authentication-Results: lists.ozlabs.org; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=linaro.org header.i=@linaro.org header.a=rsa-sha256 header.s=google header.b=DQ1YH6rd; dkim-atps=neutral Received: from boromir.ozlabs.org (localhost [IPv6:::1]) by lists.ozlabs.org (Postfix) with ESMTP id 4VTKbF0w5Zz3cXQ for ; Tue, 30 Apr 2024 22:42:17 +1000 (AEST) Authentication-Results: lists.ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=linaro.org header.i=@linaro.org header.a=rsa-sha256 header.s=google header.b=DQ1YH6rd; dkim-atps=neutral Authentication-Results: lists.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=linaro.org (client-ip=2a00:1450:4864:20::134; helo=mail-lf1-x134.google.com; envelope-from=philmd@linaro.org; receiver=lists.ozlabs.org) Received: from mail-lf1-x134.google.com (mail-lf1-x134.google.com [IPv6:2a00:1450:4864:20::134]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 4VTKZP4vMTz3c6n for ; Tue, 30 Apr 2024 22:41:31 +1000 (AEST) Received: by mail-lf1-x134.google.com with SMTP id 2adb3069b0e04-51d62b0ecb7so4351919e87.3 for ; Tue, 30 Apr 2024 05:41:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1714480887; x=1715085687; darn=lists.ozlabs.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=Z1zoRl9JNX91wxTFZ95WC2aOVdcvQqQ9ebO1YgbX9Wg=; b=DQ1YH6rdr19aIZwd4mUPf+pGQ9hkSsOdJiH6XScEn3ZqfrJeDCm93cuPA2OJl19zf3 X/V40YCTawRNLAGtUW8TMfLikjpBdCqZc+7NRpCdEQUMX9hDLztYll4iIbcKXftQjoqv FqsnQ7j4mXt/FUBlEsCuM+T8fB7Cq+fmwKCNvDc3K8Xscg60XZOTmgtKY+eM/m1sktQx RV+R9T+19GfPlilx/JEe1gUteoEHKldRfno4Jiz1ItJCzEdhbhbRcggx784eU/O40UZG sGNWM5wqDu/zrQxlQT/Kj5V028vaTv1nm3kesd+07R3YtxCxqVpaVjaiv9Adknhy7hDT XLLA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1714480887; x=1715085687; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=Z1zoRl9JNX91wxTFZ95WC2aOVdcvQqQ9ebO1YgbX9Wg=; b=kfnxFBeR8sriSTpcfyyAKIHYo/azSzOtUOajoRWWruZL6m5lHDPUa+nGjWFAeZgPM6 3J5J4uRD1T2/17oK8x1dhC+yOw01DcBSlsVn7YdJ5+UTe57QlDHgxJjQ4XAp57xCQQYS MRhhUPtxWe1S2IvGvs7zrIuIkmBqV28zFU45IMsl0lh9foqA13qweTL/EIchQ9idpjlp AU2XQZM5Mx4C6IiCLECEH6MW++V2KBTe6imlhj9naroWW14amOBS9tj16X9A45+EzN5N YoMgVfE2sov1DpjLN3C2ZctcGwRQLlx/6DBB1Yjxbuv22EI3vwDhEo0hinGv4/BT9yB3 bMbA== X-Forwarded-Encrypted: i=1; AJvYcCUh2yppGUG/5o8FNKyEYCXx/LjeYII1o8BX0XzPbvgFh5hpY9vbHv3zy/18Wsh7UijdiryELMdiJhNwrkHulm54QGhxccXqoDIx7VL6pA== X-Gm-Message-State: AOJu0Yz6V+60XfDuMj0YGq0WIACSX+ptXiXL7Qx/plqGyF+0S+/uHmvT 62QhjrJwI9/dBEggNlobR9dSPivIOBi9Dw/ogH4i7zSyrr7iSK5vN1cX/cdLOQM= X-Google-Smtp-Source: AGHT+IE/W9/dkxBso0irV5VrVX3LkNO5o5h+kLfYM+38G90SJ9h02blz/dj31DJFXSXcVMXms6NQew== X-Received: by 2002:a05:6512:45c:b0:51c:2c7e:ac92 with SMTP id y28-20020a056512045c00b0051c2c7eac92mr11527071lfk.23.1714480887229; Tue, 30 Apr 2024 05:41:27 -0700 (PDT) Received: from [192.168.69.100] (mab78-h01-176-184-55-179.dsl.sta.abo.bbox.fr. [176.184.55.179]) by smtp.gmail.com with ESMTPSA id i27-20020a17090639db00b00a58eab2bf0fsm3928326eje.179.2024.04.30.05.41.23 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 30 Apr 2024 05:41:26 -0700 (PDT) Message-ID: Date: Tue, 30 Apr 2024 14:41:22 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v7 05/16] module: make module_memory_{alloc,free} more self-contained To: Mike Rapoport , linux-kernel@vger.kernel.org References: <20240429121620.1186447-1-rppt@kernel.org> <20240429121620.1186447-6-rppt@kernel.org> Content-Language: en-US From: =?UTF-8?Q?Philippe_Mathieu-Daud=C3=A9?= In-Reply-To: <20240429121620.1186447-6-rppt@kernel.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-BeenThere: linuxppc-dev@lists.ozlabs.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Mark Rutland , x86@kernel.org, Sam Ravnborg , Peter Zijlstra , Catalin Marinas , Song Liu , Donald Dutile , sparclinux@vger.kernel.org, linux-riscv@lists.infradead.org, Nadav Amit , linux-arch@vger.kernel.org, linux-s390@vger.kernel.org, Helge Deller , Huacai Chen , Russell King , linux-trace-kernel@vger.kernel.org, Alexandre Ghiti , Will Deacon , Heiko Carstens , Steven Rostedt , loongarch@lists.linux.dev, =?UTF-8?B?QmrDtnJuIFTDtnBlbA==?= , Thomas Gleixner , bpf@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Thomas Bogendoerfer , linux-parisc@vger.kernel.org, linux-mm@kvack.org, netdev@vger.kernel.org, Kent Overstreet , linux-mips@vger.kernel.org, Dinh Nguyen , Luis Chamberlain , Eric Chanudet , Palmer Dabbelt , Masami Hiramatsu , Andrew Morton , Rick Edgecombe , linuxppc-dev@lists.ozlabs.org, "David S. Miller" , linux-modules@vger.kernel.org Errors-To: linuxppc-dev-bounces+linuxppc-dev=archiver.kernel.org@lists.ozlabs.org Sender: "Linuxppc-dev" On 29/4/24 14:16, Mike Rapoport wrote: > From: "Mike Rapoport (IBM)" > > Move the logic related to the memory allocation and freeing into > module_memory_alloc() and module_memory_free(). > > Signed-off-by: Mike Rapoport (IBM) > --- > kernel/module/main.c | 64 +++++++++++++++++++++++++++----------------- > 1 file changed, 39 insertions(+), 25 deletions(-) Nice code simplification. Reviewed-by: Philippe Mathieu-Daudé