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 kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id C6146C282D1 for ; Thu, 6 Mar 2025 06:06:13 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 53F8A6B0085; Thu, 6 Mar 2025 01:06:12 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 4EE956B0088; Thu, 6 Mar 2025 01:06:12 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 3B6BB6B0089; Thu, 6 Mar 2025 01:06:12 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0017.hostedemail.com [216.40.44.17]) by kanga.kvack.org (Postfix) with ESMTP id 1D40A6B0085 for ; Thu, 6 Mar 2025 01:06:12 -0500 (EST) Received: from smtpin19.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay06.hostedemail.com (Postfix) with ESMTP id 6EC3EB863A for ; Thu, 6 Mar 2025 06:06:12 +0000 (UTC) X-FDA: 83190090984.19.8498BB2 Received: from frasgout.his.huawei.com (frasgout.his.huawei.com [185.176.79.56]) by imf09.hostedemail.com (Postfix) with ESMTP id 6967A140002 for ; Thu, 6 Mar 2025 06:06:09 +0000 (UTC) Authentication-Results: imf09.hostedemail.com; dkim=none; dmarc=pass (policy=quarantine) header.from=huawei.com; spf=pass (imf09.hostedemail.com: domain of jonathan.cameron@huawei.com designates 185.176.79.56 as permitted sender) smtp.mailfrom=jonathan.cameron@huawei.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1741241170; a=rsa-sha256; cv=none; b=b/aaKsehEmKNyhQTKKV/rpYZ133U8mHIR725g2FIoUItl2pOpm1vzrRkmGw1lzTp9TusNE JUUNYwVJLBsHdcT2ceouoXFuXeaY1Q3soPGWFZV2uZ0lC0zoObmE4Rvzo5YZtn2E9Z3zDK RdWtHxicgp23niKMTivWH7/UdaLiZWM= ARC-Authentication-Results: i=1; imf09.hostedemail.com; dkim=none; dmarc=pass (policy=quarantine) header.from=huawei.com; spf=pass (imf09.hostedemail.com: domain of jonathan.cameron@huawei.com designates 185.176.79.56 as permitted sender) smtp.mailfrom=jonathan.cameron@huawei.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1741241170; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=ttIVLF7M+kmh/kHH4f3EzeGqY1MjI9g4rbVj9wi54C8=; b=B9sVtRZIDfGfKqMAVZ1oTvV4lRD9XMu7E4yr5HEsisQ8Fu/yvKOurzkgh8SREqsx48LEy/ WBU3VUBfwJhf1GHydGavkn9SvEdhE2WwGNyG9PrYMj9X4TJPzfD0qgDxbsa5uka5vpE7Dh Tn1hg6ucYqNNwrToWlTxJuBl6i9ZEa4= Received: from mail.maildlp.com (unknown [172.18.186.231]) by frasgout.his.huawei.com (SkyGuard) with ESMTP id 4Z7f4N3mY6z6K61Y; Thu, 6 Mar 2025 14:03:48 +0800 (CST) Received: from frapeml500008.china.huawei.com (unknown [7.182.85.71]) by mail.maildlp.com (Postfix) with ESMTPS id D0AF5140CB1; Thu, 6 Mar 2025 14:06:05 +0800 (CST) Received: from localhost (10.96.237.92) by frapeml500008.china.huawei.com (7.182.85.71) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.39; Thu, 6 Mar 2025 07:05:55 +0100 Date: Thu, 6 Mar 2025 14:05:50 +0800 From: Jonathan Cameron To: CC: , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , Subject: Re: [PATCH v2 1/3] ACPI: ACPI 6.5: RAS2: Shorten RAS2 table structure and variable names Message-ID: <20250306140550.00001016@huawei.com> In-Reply-To: <20250305180225.1226-2-shiju.jose@huawei.com> References: <20250305180225.1226-1-shiju.jose@huawei.com> <20250305180225.1226-2-shiju.jose@huawei.com> X-Mailer: Claws Mail 4.3.0 (GTK 3.24.42; x86_64-w64-mingw32) MIME-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.96.237.92] X-ClientProxiedBy: lhrpeml100012.china.huawei.com (7.191.174.184) To frapeml500008.china.huawei.com (7.182.85.71) X-Rspam-User: X-Rspamd-Server: rspam03 X-Rspamd-Queue-Id: 6967A140002 X-Stat-Signature: 8yzeobkf3npqyywj8n9sywkpht6icw9h X-HE-Tag: 1741241169-367316 X-HE-Meta: U2FsdGVkX18RdlsQYyhSH6TSSdqKCY3/IzhxVd8ZRSmJ7h92D2DDb7ELN/98SjjHvDq929cqLiBR+STmiwReWuI9MEZ3z8GxiunLqx7mqEctM/TftBaizJ0NhRj6LIojixOGrbyPTi3LaAy6w4Quxl/WQXPzqMO5BycIjPphprjRXjldPM+zDIICxC2db59qXozExZCE44SKIfhlL0D9kgZedlbxpCGFM5f7XmuK0xHriIQMUjs6SgBFrrh2A1S7fsZWgdJRNV/d+GxVUwVRPkZfy6ckoZjcmU5tRTk/Ni9wYvbn0g0XyH2gF91cL3xShqAg0ZE4Cr/6YbqvpQvLLDnKIqImciow6nPaDyI3VQ10ZsHUKx4OC2ie898gtPagC0zuFhbEm1DujDVK+9rey2rtjt24ChiEZWOgtjOx4xbdjXPWU5ncJ44Go/DzBCbsrZGQCbWMNaiAjNATkNwbOK6SidbQr9/fQ67dep6m7vYH11WXj/SbaxbvSAsn9go0ygSpo0QhisLAU2b5hdas7lV11pPR/x0sHUKUhkellAocMoasWrs6zoOGvQAsbK+Kfe8z7ROG2uE/bkWgLZC+epWb77cYcYWgP/qFBOg34GyZJk4PP0U4DdT6IgMqE1GmY560S4Ye4g298bzgdRc5PM3dV7miLQEvZsi035UuTe6G1T+dB0sYVnyGs07AOc8LCyVApgUIDIUwq3UUQ3OngzHO/x+P0OgoTE8Hf0CGkqzmi4/zhbSgiyt15/ys1i3/R/04hJmv9nmI87RpidyaQ6KQWm5GFXvkzxfRgloyKCXI7WrRvCZFkxhs3+j7ofv4hwkcfAUiCtusFkC32DdJrRmEdryQxC5GgLOW8U8XNbyEiCjL3OY4rAnBpB8Tj38Zkhgm9KQujWtHpS61uyiZU1q7N+kK2i1PcxKWNE4SemCVT0eJbsErIJSNT8/+B5BjieNy1O8M2E+N5ubvRWM fywAi3RC ECRK2mEUH0xczCInaj7zm2LrToRDqz8dTDOeO8KgFqj32UbgCvz+KuJ+0exKuGpgWVtYawwJB5R+qhHiTTOdaC1L9a+4aoS9fFYBpIv7TV+UEukyHkJ2/2F96M/LDOWrhPKPM4DwZSKM1BJzi4K+Y4DBJan6JzcQpC+g7mHidSYLXRtG1PInECHn+ViaPraiMe4xisWb7GLyKaWXAhS3XZHHgh+Defpydc9fjbq8cew9wVcrGJ6DqBroBU3fpjOgNDVZ/cHd2Gdb13l0= X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: On Wed, 5 Mar 2025 18:02:22 +0000 wrote: > From: Shiju Jose > > Shorten RAS2 table structure and variable names. > > Signed-off-by: Shiju Jose Hi Shiju, Generally looks reasonable to me, but I'm not sure what your decision process was for which to shorten and which to leave alone. Perhaps it is worth mentioning that in the patch description? > --- > include/acpi/actbl2.h | 38 +++++++++++++++++++------------------- > 1 file changed, 19 insertions(+), 19 deletions(-) > > diff --git a/include/acpi/actbl2.h b/include/acpi/actbl2.h > index 2e917a8f8bca..5cfc65ba6e9e 100644 > --- a/include/acpi/actbl2.h > +++ b/include/acpi/actbl2.h > @@ -2802,20 +2802,20 @@ struct acpi_ras2_pcc_desc { > > /* RAS2 Platform Communication Channel Shared Memory Region */ > > -struct acpi_ras2_shared_memory { > +struct acpi_ras2_shmem { > u32 signature; > - u16 command; > + u16 cmd; > u16 status; > u16 version; > u8 features[16]; > - u8 set_capabilities[16]; > - u16 num_parameter_blocks; > - u32 set_capabilities_status; > + u8 set_caps[16]; > + u16 num_param_blks; > + u32 set_caps_status; I assume focus was on fields that were leading to long line lengths? If it was just generally shortening things to common form sig, sts, ver, feats etc would also seem reasonable to me (all subject to Tony's question on whether we can touch this at all.) > }; > > /* RAS2 Parameter Block Structure for PATROL_SCRUB */ > > -struct acpi_ras2_parameter_block { > +struct acpi_ras2_param_blk { > u16 type; > u16 version; > u16 length; > @@ -2823,11 +2823,11 @@ struct acpi_ras2_parameter_block { > > /* RAS2 Parameter Block Structure for PATROL_SCRUB */ > > -struct acpi_ras2_patrol_scrub_parameter { > - struct acpi_ras2_parameter_block header; > - u16 patrol_scrub_command; > - u64 requested_address_range[2]; > - u64 actual_address_range[2]; > +struct acpi_ras2_patrol_scrub_param { > + struct acpi_ras2_param_blk header; > + u16 cmd; > + u64 req_addr_range[2]; > + u64 actl_addr_range[2]; > u32 flags; > u32 scrub_params_out; > u32 scrub_params_in; > @@ -2839,12 +2839,12 @@ struct acpi_ras2_patrol_scrub_parameter { > > /* RAS2 Parameter Block Structure for LA2PA_TRANSLATION */ > > -struct acpi_ras2_la2pa_translation_parameter { > - struct acpi_ras2_parameter_block header; > - u16 addr_translation_command; > +struct acpi_ras2_la2pa_transln_param { > + struct acpi_ras2_param_blk header; > + u16 cmd; > u64 sub_inst_id; > - u64 logical_address; > - u64 physical_address; > + u64 logical_addr; > + u64 phy_addr; > u32 status; > }; > > @@ -2863,7 +2863,7 @@ enum acpi_ras2_features { > > /* RAS2 Patrol Scrub Commands */ > > -enum acpi_ras2_patrol_scrub_commands { > +enum acpi_ras2_patrol_scrub_cmds { > ACPI_RAS2_GET_PATROL_PARAMETERS = 1, > ACPI_RAS2_START_PATROL_SCRUBBER = 2, > ACPI_RAS2_STOP_PATROL_SCRUBBER = 3 > @@ -2871,13 +2871,13 @@ enum acpi_ras2_patrol_scrub_commands { > > /* RAS2 LA2PA Translation Commands */ > > -enum acpi_ras2_la2_pa_translation_commands { > +enum acpi_ras2_la2_pa_transln_cmds { > ACPI_RAS2_GET_LA2PA_TRANSLATION = 1, > }; > > /* RAS2 LA2PA Translation Status values */ > > -enum acpi_ras2_la2_pa_translation_status { > +enum acpi_ras2_la2_pa_transln_status { Do we touch this in the main code? If not I'd be tempted to leave decision to shorten this or not to whowever writes code that uses it. > ACPI_RAS2_LA2PA_TRANSLATION_SUCCESS = 0, > ACPI_RAS2_LA2PA_TRANSLATION_FAIL = 1, > };