Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
IronPort-SDR: N90IWYwHtka/3jr+sRCmL/gdNbzkS8rCVSWNO0wKCpjO8cqiYzv//LWN++g9ovfIDpTXIHWYcJ
 WGNvS1E/W4YUjRqpbvRaxf9MlsZUV0B98Jxj8iqYZiZLLkKwsndpj0XV5XvpNZSh0geOVlPs0d
 iBsVhGMCWUltiREaKNyCMF2ZRrHjmPZ5JOkvTI2Gh4ai6r9JBR1fvuU1XMpa8tD7EujW3LVbkf
 W8gE2a114eJptNpkuWUpLe7ddrXRYTjcJdZX/WOgiUO/iar9dTlY7sOKP+cq+iJGz1j2kBO9Ye
 QYw=
From:   Anup Patel <Anup.Patel@wdc.com>
To:     Palmer Dabbelt <palmer@dabbelt.com>,
        Paul Walmsley <paul.walmsley@sifive.com>,
        Albert Ou <aou@eecs.berkeley.edu>
CC:     Atish Patra <Atish.Patra@wdc.com>,
        Alistair Francis <Alistair.Francis@wdc.com>,
        Christoph Hellwig <hch@lst.de>,
        Anup Patel <anup@brainfault.org>,
        "linux-riscv@lists.infradead.org" <linux-riscv@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Anup Patel <Anup.Patel@wdc.com>
Subject: [PATCH v2] RISC-V: Add fragmented config for debug options
Thread-Topic: [PATCH v2] RISC-V: Add fragmented config for debug options
Thread-Index: AQHVq5RRWtXKoOVpQ06lEm6d7kwovw==
Date:   Thu, 5 Dec 2019 17:49:18 +0000
Message-ID: <20191205174902.4935-1-anup.patel@wdc.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-messagesentrepresentingtype: 1
x-mailer: git-send-email 2.17.1
x-originating-ip: [199.255.44.170]
x-ms-publictraffictype: Email
x-ms-office365-filtering-ht: Tenant
x-ms-office365-filtering-correlation-id: ab38c3c0-d51d-4b07-8185-08d779ab738b
x-ms-traffictypediagnostic: MN2PR04MB6349:
x-ms-exchange-transport-forked: True
x-microsoft-antispam-prvs: <MN2PR04MB6349BF287A6FAEE0467582908D5C0@MN2PR04MB6349.namprd04.prod.outlook.com>
wdcipoutbound: EOP-TRUE
x-ms-oob-tlc-oobclassifiers: OLM:5797;
x-forefront-prvs: 02426D11FE
x-forefront-antispam-report: SFV:NSPM;SFS:(10019020)(4636009)(366004)(346002)(39860400002)(136003)(396003)(376002)(189003)(199004)(2616005)(25786009)(99286004)(52116002)(186003)(54906003)(44832011)(316002)(4326008)(478600001)(14454004)(71200400001)(50226002)(36756003)(71190400001)(6506007)(102836004)(110136005)(81156014)(66556008)(66476007)(66446008)(66946007)(64756008)(8936002)(6486002)(6512007)(5660300002)(305945005)(1076003)(2906002)(8676002)(86362001)(81166006)(26005);DIR:OUT;SFP:1102;SCL:1;SRVR:MN2PR04MB6349;H:MN2PR04MB6061.namprd04.prod.outlook.com;FPR:;SPF:None;LANG:en;PTR:InfoNoRecords;MX:1;A:1;
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: M2i3qUBh4bXx2bgGlVy4L5xXQ/SIsT1xP6Ek04kuLfYtITtwCo0JEXmugFE2kH4cmQD5XI0cAoyR9P9BOFy5tOVDwF/gdaXGjt9aXc/k6Kl5FDRee8bSih9HL1kck57pAUe60OWZezPldQz5WRFSyxZzkoeyGbP1zOEZn1Rjv1X6b0Kn7t2IWn9TnrZ6vf8bydxLO1M2MuqO2eEU6eeQSmfbjauA76xxR42v6Im2MjcKRRWSPm7HASllJYde/hf8nynECSHBVvokCZeVeslcAER7ar5uN7xw1s32Ji6JxKYZH7LWK7LgnMmFmk8xVX//k+AMJiRNyY8jR1+JWRbBWz2ac1YUbqgIZ1itQnqlQg6sXSar6RQi5lLBbFpDR6WmOGauOsTgMUtVxtonPexZCwFZW0iI2uLHI6ndgtxycFubxgPsqPKmarfCmZ3Tyx+y
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: wdc.com
X-MS-Exchange-CrossTenant-Network-Message-Id: ab38c3c0-d51d-4b07-8185-08d779ab738b
X-MS-Exchange-CrossTenant-originalarrivaltime: 05 Dec 2019 17:49:18.6422
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: b61c8803-16f3-4c35-9b17-6f65f441df86
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: d4e0Y7wTaL23SMEMWzuVSCF+rsv0mHVp0D17zgqsIPFh+MMa49xEge7pOFs3d3GevomeaFxARvRrJJpTuAldWA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR04MB6349
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Various Linux kernel DEBUG options have big performance impact so
these should not be enabled in RISC-V normal defconfigs.

Instead we should have separate RISC-V fragmented config for enabling
these DEBUG options. This way Linux RISC-V kernel can be built for
both non-debug and debug purposes using same defconfig.

This patch moves additional DEBUG options to extra_debug.config.

To configure a non-debug RV64 kernel, we use our normal defconfig:
   $ make O=3D<linux_build_directory> defconfig
Wherease to configure a debug RV64 kernel, we use extra_debug.config:
   $ make O=3D<linux_build_directory> defconfig extra_debug.config

Signed-off-by: Anup Patel <anup.patel@wdc.com>
---
Changes since v1:
 - Use fragmented .config instead of separate debug defconfigs.
---
 arch/riscv/configs/defconfig          | 23 -----------------------
 arch/riscv/configs/extra_debug.config | 21 +++++++++++++++++++++
 arch/riscv/configs/rv32_defconfig     | 23 -----------------------
 3 files changed, 21 insertions(+), 46 deletions(-)
 create mode 100644 arch/riscv/configs/extra_debug.config

diff --git a/arch/riscv/configs/defconfig b/arch/riscv/configs/defconfig
index e2ff95cb3390..f0710d8f50cc 100644
--- a/arch/riscv/configs/defconfig
+++ b/arch/riscv/configs/defconfig
@@ -101,27 +101,4 @@ CONFIG_CRYPTO_USER_API_HASH=3Dy
 CONFIG_CRYPTO_DEV_VIRTIO=3Dy
 CONFIG_PRINTK_TIME=3Dy
 CONFIG_DEBUG_FS=3Dy
-CONFIG_DEBUG_PAGEALLOC=3Dy
-CONFIG_DEBUG_VM=3Dy
-CONFIG_DEBUG_VM_PGFLAGS=3Dy
-CONFIG_DEBUG_MEMORY_INIT=3Dy
-CONFIG_DEBUG_PER_CPU_MAPS=3Dy
-CONFIG_SOFTLOCKUP_DETECTOR=3Dy
-CONFIG_WQ_WATCHDOG=3Dy
-CONFIG_SCHED_STACK_END_CHECK=3Dy
-CONFIG_DEBUG_TIMEKEEPING=3Dy
-CONFIG_DEBUG_RT_MUTEXES=3Dy
-CONFIG_DEBUG_SPINLOCK=3Dy
-CONFIG_DEBUG_MUTEXES=3Dy
-CONFIG_DEBUG_RWSEMS=3Dy
-CONFIG_DEBUG_ATOMIC_SLEEP=3Dy
-CONFIG_STACKTRACE=3Dy
-CONFIG_DEBUG_LIST=3Dy
-CONFIG_DEBUG_PLIST=3Dy
-CONFIG_DEBUG_SG=3Dy
 # CONFIG_RCU_TRACE is not set
-CONFIG_RCU_EQS_DEBUG=3Dy
-CONFIG_DEBUG_BLOCK_EXT_DEVT=3Dy
-# CONFIG_FTRACE is not set
-# CONFIG_RUNTIME_TESTING_MENU is not set
-CONFIG_MEMTEST=3Dy
diff --git a/arch/riscv/configs/extra_debug.config b/arch/riscv/configs/ext=
ra_debug.config
new file mode 100644
index 000000000000..66c58bb645a4
--- /dev/null
+++ b/arch/riscv/configs/extra_debug.config
@@ -0,0 +1,21 @@
+CONFIG_DEBUG_PAGEALLOC=3Dy
+CONFIG_DEBUG_VM=3Dy
+CONFIG_DEBUG_VM_PGFLAGS=3Dy
+CONFIG_DEBUG_MEMORY_INIT=3Dy
+CONFIG_DEBUG_PER_CPU_MAPS=3Dy
+CONFIG_SOFTLOCKUP_DETECTOR=3Dy
+CONFIG_WQ_WATCHDOG=3Dy
+CONFIG_SCHED_STACK_END_CHECK=3Dy
+CONFIG_DEBUG_TIMEKEEPING=3Dy
+CONFIG_DEBUG_RT_MUTEXES=3Dy
+CONFIG_DEBUG_SPINLOCK=3Dy
+CONFIG_DEBUG_MUTEXES=3Dy
+CONFIG_DEBUG_RWSEMS=3Dy
+CONFIG_DEBUG_ATOMIC_SLEEP=3Dy
+CONFIG_STACKTRACE=3Dy
+CONFIG_DEBUG_LIST=3Dy
+CONFIG_DEBUG_PLIST=3Dy
+CONFIG_DEBUG_SG=3Dy
+CONFIG_RCU_EQS_DEBUG=3Dy
+CONFIG_DEBUG_BLOCK_EXT_DEVT=3Dy
+CONFIG_MEMTEST=3Dy
diff --git a/arch/riscv/configs/rv32_defconfig b/arch/riscv/configs/rv32_de=
fconfig
index eb519407c841..bdec58e6c5f7 100644
--- a/arch/riscv/configs/rv32_defconfig
+++ b/arch/riscv/configs/rv32_defconfig
@@ -98,27 +98,4 @@ CONFIG_CRYPTO_USER_API_HASH=3Dy
 CONFIG_CRYPTO_DEV_VIRTIO=3Dy
 CONFIG_PRINTK_TIME=3Dy
 CONFIG_DEBUG_FS=3Dy
-CONFIG_DEBUG_PAGEALLOC=3Dy
-CONFIG_DEBUG_VM=3Dy
-CONFIG_DEBUG_VM_PGFLAGS=3Dy
-CONFIG_DEBUG_MEMORY_INIT=3Dy
-CONFIG_DEBUG_PER_CPU_MAPS=3Dy
-CONFIG_SOFTLOCKUP_DETECTOR=3Dy
-CONFIG_WQ_WATCHDOG=3Dy
-CONFIG_SCHED_STACK_END_CHECK=3Dy
-CONFIG_DEBUG_TIMEKEEPING=3Dy
-CONFIG_DEBUG_RT_MUTEXES=3Dy
-CONFIG_DEBUG_SPINLOCK=3Dy
-CONFIG_DEBUG_MUTEXES=3Dy
-CONFIG_DEBUG_RWSEMS=3Dy
-CONFIG_DEBUG_ATOMIC_SLEEP=3Dy
-CONFIG_STACKTRACE=3Dy
-CONFIG_DEBUG_LIST=3Dy
-CONFIG_DEBUG_PLIST=3Dy
-CONFIG_DEBUG_SG=3Dy
 # CONFIG_RCU_TRACE is not set
-CONFIG_RCU_EQS_DEBUG=3Dy
-CONFIG_DEBUG_BLOCK_EXT_DEVT=3Dy
-# CONFIG_FTRACE is not set
-# CONFIG_RUNTIME_TESTING_MENU is not set
-CONFIG_MEMTEST=3Dy
--=20
2.17.1