From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by spool.mail.gandi.net (Postfix) with ESMTPS id 579C2AC103D for ; Fri, 12 Jan 2024 09:12:46 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=xxXiS0HQTm1TDklg37KOc8G5AzHZYNLDjGtB6Gzb5Fc=; c=relaxed/simple; d=groups.io; h=ARC-Seal:ARC-Message-Signature:ARC-Authentication-Results:From:To:CC:Subject:Thread-Topic:Thread-Index:Date:Message-ID:Accept-Language:MIME-Version:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Language:Content-Type; s=20140610; t=1705050765; v=1; b=NgPBWR96jBwVaI52tjoLGx0tl2VcWGddyfzo26DZI3CtiJUgNxpVb4kS++Fc5n0DFeT4sGpa DMW4EZKZVUGopslj6VlhyOoZYOkbInfk4XOklJ4nvYhy7qGBhZ1/yQTyarP1S0PKlSkomISmwoP aI+qIchGqrKmT4PAC0A21oFs= X-Received: by 127.0.0.2 with SMTP id XhnYYY7687511xVomiKsW0Sx; Fri, 12 Jan 2024 01:12:45 -0800 X-Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.11]) by mx.groups.io with SMTP id smtpd.web11.3495.1705050763931508134 for ; Fri, 12 Jan 2024 01:12:44 -0800 X-IronPort-AV: E=McAfee;i="6600,9927,10950"; a="5847585" X-IronPort-AV: E=Sophos;i="6.04,189,1695711600"; d="scan'208,217";a="5847585" X-Received: from orsmga006.jf.intel.com ([10.7.209.51]) by orvoesa103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 12 Jan 2024 01:12:44 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10950"; a="759074635" X-IronPort-AV: E=Sophos;i="6.04,189,1695711600"; d="scan'208,217";a="759074635" X-Received: from fmsmsx601.amr.corp.intel.com ([10.18.126.81]) by orsmga006.jf.intel.com with ESMTP/TLS/AES256-GCM-SHA384; 12 Jan 2024 01:12:42 -0800 X-Received: from fmsmsx610.amr.corp.intel.com (10.18.126.90) by fmsmsx601.amr.corp.intel.com (10.18.126.81) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35; Fri, 12 Jan 2024 01:12:41 -0800 X-Received: from fmsedg601.ED.cps.intel.com (10.1.192.135) by fmsmsx610.amr.corp.intel.com (10.18.126.90) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35 via Frontend Transport; Fri, 12 Jan 2024 01:12:41 -0800 X-Received: from NAM12-DM6-obe.outbound.protection.outlook.com (104.47.59.168) by edgegateway.intel.com (192.55.55.70) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.35; Fri, 12 Jan 2024 01:12:41 -0800 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=IwxF5AkCcMGweCjygP8m/kuFNkgysbMiHWG+m24qmdv9UcRPP1aYMHzyjNvTU5UUjC1XpRhtAguJXkOUhugpmW4P23LqsDXbI16hj+FrlnGNVfM+JuywUriiY2E5Kzq7n60YL+yTgYrhJ+w4ORlRxuYiLe4ASY52Isg7s75lqx5yulDDYIUTTTl+1E8QXB+McaRik8JMZSp+t4YjIw4nwvRa01pvxd8FZOGcOY5uvuXuD6rnLmH4L9q5q42kXQsWB5thDF6vNARBE05zXHQwbUfz1TQFRk2H4NZnxkzlTy7rE9ZpjWLwX18HS3AVsEGJLgGPryCXB4hIIeRA46D05g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=2CKWncH/FpfW+nqpUxqm6UF8RVJr5nHkIDja68xLTL4=; b=W3DLnxSdxUBHWUaXtRdkp4lF1M/bO0GD/F7rJqrFXn67mgPvznFaRWxtCUSOFeV3KYO97GZhwoDckSgpymLPgL0gf2Fld4pscPF/3V5+uee01vXEJ0iepi7Ya9lsKVUgkMVxkpTYXu+f548atxm+CSi+oB49ycMxB4BKDqWZnczli7/cfvh9oXSm+Lw7Tarrf5B+fdcwGZmWxYq1rgSlgZqzTmtZFERl9cfYcx8k+ZnSkoh4Edfh2mbmS+F5iudsCvz2V5uWr4J9qIV+Sh16g658UqnT53rLtGTOehNEvhJOaS41IwB3KeNYjrfQ+nFf5fmCdiNYMFGe7z6iZhF7TQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none X-Received: from MN6PR11MB8244.namprd11.prod.outlook.com (2603:10b6:208:470::14) by PH7PR11MB6793.namprd11.prod.outlook.com (2603:10b6:510:1b7::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7181.17; Fri, 12 Jan 2024 09:12:34 +0000 X-Received: from MN6PR11MB8244.namprd11.prod.outlook.com ([fe80::3fea:ca2b:2ef7:e3d4]) by MN6PR11MB8244.namprd11.prod.outlook.com ([fe80::3fea:ca2b:2ef7:e3d4%4]) with mapi id 15.20.7159.020; Fri, 12 Jan 2024 09:12:34 +0000 From: "Ni, Ray" To: "devel@edk2.groups.io" , Sunil V L CC: "Ni, Ray" , "lichao@loongson.cn" , "Kinney, Michael D" , Laszlo Ersek , Gerd Hoffmann Subject: [edk2-devel] RFC: Folder layout change in UefiCpuPkg Thread-Topic: RFC: Folder layout change in UefiCpuPkg Thread-Index: AdpFNi/wTf+No1DoTCqsWNHnTlW5cA== Date: Fri, 12 Jan 2024 09:12:34 +0000 Message-ID: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-publictraffictype: Email x-ms-traffictypediagnostic: MN6PR11MB8244:EE_|PH7PR11MB6793:EE_ x-ms-office365-filtering-correlation-id: 1370d4bf-b658-4f08-2090-08dc134e9d0d x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam-message-info: ETYpij+gWUrVUGswbXSKNECHP0cv0jG5QzcuzPZ61y8/V8Tz1NEjmrzY7CKBmOk+xgY2h0WedHEO4Pn1fYBqGPltr98jVkxvIg6+gNriQow4+Y8SUgn0vvRZLikiClUHyl0R4NiFO6uyh6SPJBttQwl7RRvDJ8lCoiOHe9hrgQzr7tGaHoRKNo9JqDUl7gCH0/WS6HrgCqcRru3USMOF/p+/8jH4HVWz1UARF0HSnOBvati2z8fQ3eB7HjyVNxYQQGIpTSPYYac2TSY0uPKteJOKukU5LxCpakVIeLOkoxguo/juoncbzUGoIx1VEjMWfhoDVVSFgwhInUPMW7+zecUT+gDEVTs7/Nqt5B6sQTwlgpku1zhT71gMemdkNzNPg8s3VPJgDzse2KzCuBt5Nr20FM8IxNCqeOjZo00+w9nK34SOiGuXlJMUXSu5lhi8fdV5uyUZbR+CbQJUs6kgQQTvuawaVYhKbgOhetM7Vpst3yhNLUWF4KcDYaOhZJYOjK3vSOkeLHPb50gG+sVQj+bI9WDlITVbJarzcjis2TtWThYaiyCUnTA+CfjvlQUG9r2rckuWmMSPQImahhZDNNNNcC4AyDDTFolJUQ1B0kRLmPSHqAa1Fu4dAGO+AC7N x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?TllJvMnbpLv0Yavz2PaOeTzfvmcz97g1XBGeI/QRQvaHMtboIWG9jHmL7Wkg?= =?us-ascii?Q?16xDRsehr07ySm2bJdHZdFe41wo00Ky1Jbf+ZJDIT7J505LDbdwnW4MZ8Ugh?= =?us-ascii?Q?XfN8CiVTGhIhl+IkJpTj6tvWeyI//8QXdFN6UNcGlBST1z2LvpUyUsikFd3s?= =?us-ascii?Q?fFCp89wsUyRfbNIco7Sq0b/wo743uqDQBKc1fAtyo2cSd4f1/BKj9AZDnMfy?= =?us-ascii?Q?WSvICi0oB1HVzoOGRDZjn1pMiTxA+E8UkncGcBb9m0juhR+GlDsW1kiBbuGq?= =?us-ascii?Q?RfPryhKVaCLFLikm9oALivX4k1TB2tkAVX82g62u4bVLAk1UaLYm5DJzjtcM?= =?us-ascii?Q?CS1UGZgPffwybX7DPAhKbUj5v3oadwBV+UM44Ig06QGFz+QXLWeuPJ3rSnG9?= =?us-ascii?Q?2kCCkyllissZNN2wjaZ8dncAGsx0H7cDgneFU1fU6S641ELDqwXjWcpRPNo+?= =?us-ascii?Q?M4f8PN6YFtJXhaE4/7J1X7ZiVCtYyY/tiv4ri3cRRHFTTycdrjglaZbvLoZP?= =?us-ascii?Q?ZZwLlGF/9/CcfvElE/XRTa7GyYvv9ws8m+fbPO9EmVW+0TjdkhK+VK6hJadX?= =?us-ascii?Q?loDZ5FlCLfYqzUV1v+b7LIwAKbFdsCucaiUYO5DK1QSGlUUCJ+xgUkxJA+eQ?= =?us-ascii?Q?F+X9ICn0Pf3ZXbFZsvOrZRWkhsFLsDHQdUX6bDY6vg+7DeKyAox9FZ3neHs/?= =?us-ascii?Q?ei5qQzTfk8gcIXHcu9Sj1hi1lnitEjkTYpTq8h7D9lTcUgF8umYQ0j09fgmp?= =?us-ascii?Q?1BIJCTQQ0QtqMBkTUSwDzGtzwppROFHAjfmnyVo66t2g6LZ9S75BliUtQzgT?= =?us-ascii?Q?pAIChubY8HTXaCPw7bdqEaTF0EGEkDm0nUA5LV7L67aYqDAPw/bgNqTTeLzS?= =?us-ascii?Q?d5jQebzMx3JzVSZFpU/ZYCjGlQneXwxBPBWTH2f+9VHvga280jVFSHFgNHHU?= =?us-ascii?Q?A8gwOu7hVUasA6kIFImvfKfyxsVF1ciPIsSiFmjNSUqHbh0jywcNxHobpCTN?= =?us-ascii?Q?puhw1Ur9ApuKgydKHy7hxHxhYJt15+2Oc9hzV5PJr1ZYvWr+RHDUbmZ3sDeh?= =?us-ascii?Q?4X9sbAov+hihPl3qzytyjlUCfQaUIzt7E+kIc/uYlPSlI67WoIPxI5iYbGpE?= =?us-ascii?Q?IQmHg93+nqVZR+VEDloW+jMf0yzmdonKGj2zEjKKS5ix0xlFYsSmqoJoW3OJ?= =?us-ascii?Q?9F1flDhCQu4m/+h/JcUb1VDbSxF0LYq9QzX3PAVXKU9InqB4kak3ziMBfpKv?= =?us-ascii?Q?45hZyqf+xQDQYVeZg/4vCQAYYeZnSAb0HPBNRKNGntT35KriSRqgUAsnUld0?= =?us-ascii?Q?RGIN0axFixgGHg0oV538b0cyDFX83f54tmzDP/ZYtLw6yTESKmrBtqpzIUCF?= =?us-ascii?Q?zeIVn+wFXxPVfJVQLt5AU8sShSKPvQZHaj8Aeb9G4QeAvOVe7TKw0gZkSt3e?= =?us-ascii?Q?2OLmz0vE7AFzQ1F/zOQaE46XIJvXnJZ5Zpq21YOAEGVaRMLW9Q+dUFcQRa72?= =?us-ascii?Q?ahjJob4jAz0RhNCnaXITaJp5uHvC/BYija3SZAAJqBC0JDo4CV6p7Be5/G6U?= =?us-ascii?Q?OF//JADdTIcH0S9oxoI=3D?= MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: MN6PR11MB8244.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 1370d4bf-b658-4f08-2090-08dc134e9d0d X-MS-Exchange-CrossTenant-originalarrivaltime: 12 Jan 2024 09:12:34.2715 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: UuUUfZX2cg9RiKgMq5rZkzXwqbrfx8wvXg2nBqahvBVurHQvv+oqloShVPbQwKrbSMWSIwbKBL6S3HCC/fm32A== X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR11MB6793 X-OriginatorOrg: intel.com Precedence: Bulk List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,ray.ni@intel.com List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: X-Gm-Message-State: FG8WATjvFQgMYXmgQhwcn43bx7686176AA= Content-Language: en-US Content-Type: multipart/alternative; boundary="_000_MN6PR11MB8244364844C5861F5518DDA38C6F2MN6PR11MB8244namp_" X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20140610 header.b=NgPBWR96; arc=reject ("signature check failed: fail, {[1] = sig:microsoft.com:reject}"); dmarc=fail reason="SPF not aligned (relaxed), DKIM not aligned (relaxed)" header.from=intel.com (policy=none); spf=pass (spool.mail.gandi.net: domain of bounce@groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce@groups.io --_000_MN6PR11MB8244364844C5861F5518DDA38C6F2MN6PR11MB8244namp_ Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Sunil, I would like to hear your feedback regarding locations of following RiscV64= components in UefiCpuPkg: * UefiCpuPkg/Library/BaseRiscV64CpuExceptionHandlerLib/ * UefiCpuPkg/Library/BaseRiscV64CpuTimerLib/ * UefiCpuPkg/CpuDxeRiscV64/ * UefiCpuPkg/CpuTimerDxeRiscV64/ I would like to move them to the following new locations accordingly: * UefiCpuPkg/Library/CpuExceptionHandlerLib/RiscV64/ * UefiCpuPkg/Library/CpuTimerLib/RiscV64/ * UefiCpuPkg/CpuDxe/RiscV64/ * UefiCpuPkg/CpuTimerDxe/RiscV64/ I want to avoid too many similar drivers in root folder, and too many libra= ries in Library folder. Movement of the first 3 ones put the RiscV components under existing folder= s. Movement of the last one creates the UefiCpuPkg/CpuTimerDxe folder, that co= uld be potentially shared by other archs as well. I raised similar comments to Chao Li who is working on LoongArch upstream. The location movement follows the 2nd pattern defined by edk2 coding standa= rd: Driver's location could be: [[]] or [/[/]] Library's location could be: [[]][] or []/[[/]] Thanks, Ray -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#113700): https://edk2.groups.io/g/devel/message/113700 Mute This Topic: https://groups.io/mt/103679850/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/leave/12367111/7686176/19134562= 12/xyzzy [rebecca@openfw.io] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- --_000_MN6PR11MB8244364844C5861F5518DDA38C6F2MN6PR11MB8244namp_ Content-Type: text/html; charset="us-ascii" Content-Transfer-Encoding: quoted-printable

Sunil,

I would like to hear your feedback regarding locatio= ns of following RiscV64 components in UefiCpuPkg:

* UefiCpuPkg/Library/BaseRiscV64CpuExceptionHandlerL= ib/

* UefiCpuPkg/Library/BaseRiscV64CpuTimerLib/

* UefiCpuPkg/CpuDxeRiscV64/

* UefiCpuPkg/CpuTimerDxeRiscV64/

 

I would like to move them to the following new locat= ions accordingly:

* UefiCpuPkg/Library/CpuExceptionHandlerLib/RiscV64/=

* UefiCpuPkg/Library/CpuTimerLib/RiscV64/=

* UefiCpuPkg/CpuDxe/RiscV64/

* UefiCpuPkg/CpuTimerDxe/RiscV64/

 

I want to avoid too many similar drivers in root fol= der, and too many libraries in Library folder.

 

Movement of the first 3 ones put the RiscV component= s under existing folders.

Movement of the last one creates the UefiCpuPkg/CpuT= imerDxe folder, that could be potentially shared by other archs as well.

 

I raised similar comments to Chao Li who is working = on LoongArch upstream.

 

The location movement follows the 2nd pattern define= d by edk2 coding standard:

Driver’s location could be:

<Feature><Phase>[<CpuArch>[= <Vendor>]]

  or

<Feature><Phase>[/<CpuArch>= [/<Vendor>]]

 

Library’s location could be:

<Phase>[<=
CpuArch>[<Vendor>]]<LibraryClassName>[<Dependency>]
  or
<Phase><L=
ibraryClassName>[<Dependency>]/[<CpuArch>[/<Vendor>]]<=
o:p>

 

 

 

Thanks,

Ray

_._,_._,_
Groups.io Links:

=20 You receive all messages sent to this group. =20 =20

View/Reply Online (#113700) | =20 | Mute= This Topic | New Topic
Your Subscriptio= n | Contact Group Owner | Unsubscribe [rebecca@openfw.io]

_._,_._,_
--_000_MN6PR11MB8244364844C5861F5518DDA38C6F2MN6PR11MB8244namp_--