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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id DE0B4C77B61 for ; Mon, 10 Apr 2023 16:47:20 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230113AbjDJQrT (ORCPT ); Mon, 10 Apr 2023 12:47:19 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52280 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229685AbjDJQrR (ORCPT ); Mon, 10 Apr 2023 12:47:17 -0400 Received: from mail-ed1-x531.google.com (mail-ed1-x531.google.com [IPv6:2a00:1450:4864:20::531]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 021B81BC0 for ; Mon, 10 Apr 2023 09:47:16 -0700 (PDT) Received: by mail-ed1-x531.google.com with SMTP id 4fb4d7f45d1cf-50480ce88dbso1693216a12.3 for ; Mon, 10 Apr 2023 09:47:15 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1681145234; 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=s+gxGMIiSz5uZWgSiDLv5wb5ekq1gKmiqT8Bd8yz/ds=; b=GQ8L1696PKy/4oEanTiqpv5S049NPWjf9AapfHHOvaUOFomKL9hdA5inRia5cFnph1 XeEiOh+fraVbenfuv7oTB2FnUInydzkOsg+Dx3W4KcwD+d/zN8bz56WwJiXMEPu6qKXD obB4rw03UABGtvIVsTcPSDXB6UiuIQJjJ3QNvMDqlaMEgz3OYDyMaBJgd1AcZWrpQyML ERb+AnBmEVJYIDHiyE9az4hd8bLf6zHGp05RbdJLcYJeBOWTSuA7vvt8IGn0IUAIvXBS dbAhIelo+DWXhLr3d5tb4SdgHOPQ0KADe6wwD+s3B7qdDD1U5JtSSAZVncqDYjgpQsdS N2vg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1681145234; 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=s+gxGMIiSz5uZWgSiDLv5wb5ekq1gKmiqT8Bd8yz/ds=; b=NdDjxzV9mYzVL7URTrr5ra/uv6RLjxEZps00eYRNbD7ruH2paPOs2Xgw5sG/6UsjtL S0erutClMDBsYAm6FpeLXpH3IIteSpduHqx/PEiDsHAJ5hvk/LtqTfW/IseC8u21wtZ5 C+ZvyqT9vGfj0/YrPyAn1H/lAKVFC1a9BvX2oPcoL2Z9X9WAzINBbYVEey0y1KhfQmra NAGtII0kYWH3geT0OIb5PuZGYsrhVOYw18Se+AqWznngYcKOl9q3KAy6Qk1Jd5YQgGod kfndm8gn0OyPrYfxJDO55zNSSyMpcSPuFE1hgEzZmvbZ5d9NR6+3i0JThWKPoEFBngpV 1HKw== X-Gm-Message-State: AAQBX9dY5WY4QCkxUTVi1PqeK1aAkY9R04ZOvu95Yq83tfRKECV8FVlP 6d53jOmdBRQblunYWqYTJkrLMA== X-Google-Smtp-Source: AKy350aF6ODPvsCZJZHKkPkWmgpgg3SFf2Yb1hUKMl+fKRqs3xDD5O9oVflnyKPtaL7Hn3xY4dCd4A== X-Received: by 2002:a50:ed05:0:b0:504:9a52:ea52 with SMTP id j5-20020a50ed05000000b005049a52ea52mr5182455eds.35.1681145234446; Mon, 10 Apr 2023 09:47:14 -0700 (PDT) Received: from ?IPV6:2a02:810d:15c0:828:1a6c:6968:e633:48df? ([2a02:810d:15c0:828:1a6c:6968:e633:48df]) by smtp.gmail.com with ESMTPSA id v6-20020a50c406000000b0050432d2b443sm5026002edf.48.2023.04.10.09.47.13 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 10 Apr 2023 09:47:13 -0700 (PDT) Message-ID: Date: Mon, 10 Apr 2023 18:47:12 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.9.0 Subject: Re: [PATCH] ARM: exynos_defconfig: Enable zram as loadable modules To: Javier Martinez Canillas , linux-kernel@vger.kernel.org Cc: Alim Akhtar , Arnd Bergmann , Gregory CLEMENT , Jernej Skrabec , Robert Elliott , Russell King , Stefan Hansson , Sudeep Holla , linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org References: <20230410124907.3293869-1-javierm@redhat.com> Content-Language: en-US From: Krzysztof Kozlowski In-Reply-To: <20230410124907.3293869-1-javierm@redhat.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 10/04/2023 14:49, Javier Martinez Canillas wrote: > Enable zram and dependencies as loadable modules. This allows to use the > /dev/zramX virtual block devices as swap stored in system memory. > > Options were taken from commit 5c824e8be64d ("ARM: omap2plus_defconfig: > Enable zram as loadable modules") that does the same for OMAP2+ platforms. Neither this commit msg nor above omap commit explain why do we want it. I know what zram is, I know its purpose, but the commit should answer why we want it in defconfig. To me it is not a matching defconfig at all: 1. Not a feature related to Exynos hardware, 2. Not needed for basic boot (systemd) or debug. > > Signed-off-by: Javier Martinez Canillas > --- > > arch/arm/configs/exynos_defconfig | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/arch/arm/configs/exynos_defconfig b/arch/arm/configs/exynos_defconfig > index b0f0baa3a6c4..2d5be864b4ca 100644 > --- a/arch/arm/configs/exynos_defconfig > +++ b/arch/arm/configs/exynos_defconfig > @@ -37,6 +37,8 @@ CONFIG_MODULES=y > CONFIG_MODULE_UNLOAD=y > CONFIG_PARTITION_ADVANCED=y > CONFIG_CMA=y > +CONFIG_ZSMALLOC=m > +CONFIG_PGTABLE_MAPPING=y > CONFIG_NET=y > CONFIG_PACKET=y > CONFIG_UNIX=y > @@ -84,6 +86,8 @@ CONFIG_NFC_SHDLC=y > CONFIG_NFC_S3FWRN5_I2C=y > CONFIG_DEVTMPFS=y > CONFIG_DEVTMPFS_MOUNT=y > +CONFIG_ZRAM=m > +CONFIG_ZRAM_WRITEBACK=y > CONFIG_BLK_DEV_LOOP=y > CONFIG_BLK_DEV_CRYPTOLOOP=y > CONFIG_BLK_DEV_RAM=y > > base-commit: e3adc46da349d4a4cda1c58d8186c5bce0b011fd BTW, I think the base commit makes sense if it is "well-known commit". I have latest master and next and commit is unknown. Best regards, Krzysztof 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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (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 7C47DC76196 for ; Mon, 10 Apr 2023 16:48:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To: Subject:MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=+86pCZQ9LyGs7ShTIQlB0vctz1hZMF4jJ1503SOWwNQ=; b=ibpUB4tGMdZ8+T BjODO7sB54uuK+cujXg69GkoaDA1ruiuo7RZhPbDWIo1SWrrF2w6BGGPNUa8TDVJa1R3RpD1TVyz7 Lw1GNvWFPH/aiciNWzgnEiBU/uJWH+VxNEAaduK0+NOMYFTm75K5nnJf79YdkEoU8NYwxoA0H62v/ o21ycTp3gFqyZdZN0ekWrUloAmNod68p/SS83k9ZoGmBQTYhu1wsMNXFupTJBAlLM4KC2V0StWkSD 2ocWQ+Tz9UWgAAxVwLaoUf4KC+4nJNcAolEk0R8hKKFf+ol9c5h4tLgLp49r0Yc+TaJAjJ+cpvwxe qg9n8rpLCjRVoYne4Gpg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1plufo-00FZzM-2B; Mon, 10 Apr 2023 16:47:20 +0000 Received: from mail-ed1-x52d.google.com ([2a00:1450:4864:20::52d]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1plufl-00FZys-1D for linux-arm-kernel@lists.infradead.org; Mon, 10 Apr 2023 16:47:18 +0000 Received: by mail-ed1-x52d.google.com with SMTP id 4fb4d7f45d1cf-50489acccdfso1333449a12.1 for ; Mon, 10 Apr 2023 09:47:15 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1681145234; 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=s+gxGMIiSz5uZWgSiDLv5wb5ekq1gKmiqT8Bd8yz/ds=; b=GQ8L1696PKy/4oEanTiqpv5S049NPWjf9AapfHHOvaUOFomKL9hdA5inRia5cFnph1 XeEiOh+fraVbenfuv7oTB2FnUInydzkOsg+Dx3W4KcwD+d/zN8bz56WwJiXMEPu6qKXD obB4rw03UABGtvIVsTcPSDXB6UiuIQJjJ3QNvMDqlaMEgz3OYDyMaBJgd1AcZWrpQyML ERb+AnBmEVJYIDHiyE9az4hd8bLf6zHGp05RbdJLcYJeBOWTSuA7vvt8IGn0IUAIvXBS dbAhIelo+DWXhLr3d5tb4SdgHOPQ0KADe6wwD+s3B7qdDD1U5JtSSAZVncqDYjgpQsdS N2vg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1681145234; 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=s+gxGMIiSz5uZWgSiDLv5wb5ekq1gKmiqT8Bd8yz/ds=; b=lxdRr/e704U4m/lBJ37NmZKiGy9yvG5PlzlAuJuwPb3/WGxweWb+urh0WX3LkeI+0c AOieSZynEZhJz3gafqa0+q57rTI8Wmd/+4fXX63Bss63ZDT/SDSmpI+QYIMxN3mF3CPq uJnW6EIAaz8RfY+CcfITH4ybCK0YpNVwr2kRzbUSP7/e72UWeFTceBLT/uCmm/KunoW+ Claa1BFRHLbfXYJnevxEnSexoVrhaA4t7U/ixcvLxE/NgManSK3jVzzfUgrJcHEsrGTd x7g16z5ekaRhs7h9enkx2mj+nzLDVKQ2KjOE3MAXKszuXb3hGBhjjbcdRAHODiA3HZKC 0TAg== X-Gm-Message-State: AAQBX9dxtwL7vWqCL9L7mVYXlkI0QJk7yLRpSXVNXxn9YmoQV1qCFgRO rzudOraS0RP7bbrcO3iRMWiB5A== X-Google-Smtp-Source: AKy350aF6ODPvsCZJZHKkPkWmgpgg3SFf2Yb1hUKMl+fKRqs3xDD5O9oVflnyKPtaL7Hn3xY4dCd4A== X-Received: by 2002:a50:ed05:0:b0:504:9a52:ea52 with SMTP id j5-20020a50ed05000000b005049a52ea52mr5182455eds.35.1681145234446; Mon, 10 Apr 2023 09:47:14 -0700 (PDT) Received: from ?IPV6:2a02:810d:15c0:828:1a6c:6968:e633:48df? ([2a02:810d:15c0:828:1a6c:6968:e633:48df]) by smtp.gmail.com with ESMTPSA id v6-20020a50c406000000b0050432d2b443sm5026002edf.48.2023.04.10.09.47.13 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 10 Apr 2023 09:47:13 -0700 (PDT) Message-ID: Date: Mon, 10 Apr 2023 18:47:12 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.9.0 Subject: Re: [PATCH] ARM: exynos_defconfig: Enable zram as loadable modules To: Javier Martinez Canillas , linux-kernel@vger.kernel.org Cc: Alim Akhtar , Arnd Bergmann , Gregory CLEMENT , Jernej Skrabec , Robert Elliott , Russell King , Stefan Hansson , Sudeep Holla , linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org References: <20230410124907.3293869-1-javierm@redhat.com> Content-Language: en-US From: Krzysztof Kozlowski In-Reply-To: <20230410124907.3293869-1-javierm@redhat.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230410_094717_484842_6D98B524 X-CRM114-Status: GOOD ( 18.91 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 10/04/2023 14:49, Javier Martinez Canillas wrote: > Enable zram and dependencies as loadable modules. This allows to use the > /dev/zramX virtual block devices as swap stored in system memory. > > Options were taken from commit 5c824e8be64d ("ARM: omap2plus_defconfig: > Enable zram as loadable modules") that does the same for OMAP2+ platforms. Neither this commit msg nor above omap commit explain why do we want it. I know what zram is, I know its purpose, but the commit should answer why we want it in defconfig. To me it is not a matching defconfig at all: 1. Not a feature related to Exynos hardware, 2. Not needed for basic boot (systemd) or debug. > > Signed-off-by: Javier Martinez Canillas > --- > > arch/arm/configs/exynos_defconfig | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/arch/arm/configs/exynos_defconfig b/arch/arm/configs/exynos_defconfig > index b0f0baa3a6c4..2d5be864b4ca 100644 > --- a/arch/arm/configs/exynos_defconfig > +++ b/arch/arm/configs/exynos_defconfig > @@ -37,6 +37,8 @@ CONFIG_MODULES=y > CONFIG_MODULE_UNLOAD=y > CONFIG_PARTITION_ADVANCED=y > CONFIG_CMA=y > +CONFIG_ZSMALLOC=m > +CONFIG_PGTABLE_MAPPING=y > CONFIG_NET=y > CONFIG_PACKET=y > CONFIG_UNIX=y > @@ -84,6 +86,8 @@ CONFIG_NFC_SHDLC=y > CONFIG_NFC_S3FWRN5_I2C=y > CONFIG_DEVTMPFS=y > CONFIG_DEVTMPFS_MOUNT=y > +CONFIG_ZRAM=m > +CONFIG_ZRAM_WRITEBACK=y > CONFIG_BLK_DEV_LOOP=y > CONFIG_BLK_DEV_CRYPTOLOOP=y > CONFIG_BLK_DEV_RAM=y > > base-commit: e3adc46da349d4a4cda1c58d8186c5bce0b011fd BTW, I think the base commit makes sense if it is "well-known commit". I have latest master and next and commit is unknown. Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel