public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [edk2-devel] RFC: Folder layout change in UefiCpuPkg
@ 2024-01-12  9:12 Ni, Ray
  2024-01-12  9:57 ` Chao Li
  2024-01-12 10:19 ` Sunil V L
  0 siblings, 2 replies; 17+ messages in thread
From: Ni, Ray @ 2024-01-12  9:12 UTC (permalink / raw)
  To: devel@edk2.groups.io, Sunil V L
  Cc: Ni, Ray, lichao@loongson.cn, Kinney, Michael D, Laszlo Ersek,
	Gerd Hoffmann

[-- Attachment #1: Type: text/plain, Size: 1688 bytes --]

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 libraries in Library folder.

Movement of the first 3 ones put the RiscV components under existing folders.
Movement of the last one creates the UefiCpuPkg/CpuTimerDxe 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 defined 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><LibraryClassName>[<Dependency>]/[<CpuArch>[/<Vendor>]]



Thanks,
Ray


-=-=-=-=-=-=-=-=-=-=-=-
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/1913456212/xyzzy [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



[-- Attachment #2: Type: text/html, Size: 6888 bytes --]

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
  2024-01-12  9:12 [edk2-devel] RFC: Folder layout change in UefiCpuPkg Ni, Ray
@ 2024-01-12  9:57 ` Chao Li
  2024-01-12 10:19 ` Sunil V L
  1 sibling, 0 replies; 17+ messages in thread
From: Chao Li @ 2024-01-12  9:57 UTC (permalink / raw)
  To: devel, ray.ni, Sunil V L; +Cc: Kinney, Michael D, Laszlo Ersek, Gerd Hoffmann

[-- Attachment #1: Type: text/plain, Size: 2405 bytes --]

Hi Ray,

OK, here I come.

[Ray] INF is a good question.

[Ray] There are two options:

[Ray]    Re-use current INF file.

[Ray]    Create a new INF under LoongArch64 folder.

[Ray] If the INF content can be shared a lot between different archs. 
Single INF is better.

[Ray] I am just afraid sometimes we just mixed INF contents for 
different archs together into one INF, which makes the INF hard to read.

[Ray] So, it depends on how many contents can be shared.

I think it is almost difficult to share, because currently common files 
only work with IA32 and X86, other ARCHs cannot share files with IA32 
and X64.

Thanks,
Chao
On 2024/1/12 17:12, Ni, Ray wrote:
>
> 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 
> libraries in Library folder.
>
> Movement of the first 3 ones put the RiscV components under existing 
> folders.
>
> Movement of the last one creates the UefiCpuPkg/CpuTimerDxe 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 defined 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><LibraryClassName>[<Dependency>]/[<CpuArch>[/<Vendor>]]|
>
> Thanks,
>
> Ray
>
> 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#113706): https://edk2.groups.io/g/devel/message/113706
Mute This Topic: https://groups.io/mt/103679850/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



[-- Attachment #2: Type: text/html, Size: 8306 bytes --]

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
  2024-01-12  9:12 [edk2-devel] RFC: Folder layout change in UefiCpuPkg Ni, Ray
  2024-01-12  9:57 ` Chao Li
@ 2024-01-12 10:19 ` Sunil V L
  2024-01-15  1:13   ` Ni, Ray
  2024-01-15  8:48   ` Laszlo Ersek
  1 sibling, 2 replies; 17+ messages in thread
From: Sunil V L @ 2024-01-12 10:19 UTC (permalink / raw)
  To: Ni, Ray
  Cc: devel@edk2.groups.io, lichao@loongson.cn, Kinney, Michael D,
	Laszlo Ersek, Gerd Hoffmann

Hi Ray,

On Fri, Jan 12, 2024 at 09:12:34AM +0000, Ni, Ray wrote:
> 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 libraries in Library folder.
> 
> Movement of the first 3 ones put the RiscV components under existing folders.
> Movement of the last one creates the UefiCpuPkg/CpuTimerDxe 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 defined 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><LibraryClassName>[<Dependency>]/[<CpuArch>[/<Vendor>]]
> 
> 
Your proposal looks good to me except better to keep directory name as
RiscV as in other packages.

Thanks,
Sunil


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#113707): https://edk2.groups.io/g/devel/message/113707
Mute This Topic: https://groups.io/mt/103679850/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
  2024-01-12 10:19 ` Sunil V L
@ 2024-01-15  1:13   ` Ni, Ray
  2024-01-15  4:44     ` Sunil V L
  2024-01-15  8:48   ` Laszlo Ersek
  1 sibling, 1 reply; 17+ messages in thread
From: Ni, Ray @ 2024-01-15  1:13 UTC (permalink / raw)
  To: Sunil V L
  Cc: devel@edk2.groups.io, lichao@loongson.cn, Kinney, Michael D,
	Laszlo Ersek, Gerd Hoffmann

Sunil,
I am glad that you don't have concern about the direction.

"RiscV" is good to me.


Thanks,
Ray
> -----Original Message-----
> From: Sunil V L <sunilvl@ventanamicro.com>
> Sent: Friday, January 12, 2024 6:19 PM
> To: Ni, Ray <ray.ni@intel.com>
> Cc: devel@edk2.groups.io; lichao@loongson.cn; Kinney, Michael D
> <michael.d.kinney@intel.com>; Laszlo Ersek <lersek@redhat.com>; Gerd
> Hoffmann <kraxel@redhat.com>
> Subject: Re: RFC: Folder layout change in UefiCpuPkg
> 
> Hi Ray,
> 
> On Fri, Jan 12, 2024 at 09:12:34AM +0000, Ni, Ray wrote:
> > 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 libraries
> in Library folder.
> >
> > Movement of the first 3 ones put the RiscV components under existing
> folders.
> > Movement of the last one creates the UefiCpuPkg/CpuTimerDxe 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 defined 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><LibraryClassName>[<Dependency>]/[<CpuArch>[/<Vendor>]]
> >
> >
> Your proposal looks good to me except better to keep directory name as
> RiscV as in other packages.
> 
> Thanks,
> Sunil


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#113783): https://edk2.groups.io/g/devel/message/113783
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/1913456212/xyzzy [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
  2024-01-15  1:13   ` Ni, Ray
@ 2024-01-15  4:44     ` Sunil V L
  2024-01-15  5:44       ` Ni, Ray
  0 siblings, 1 reply; 17+ messages in thread
From: Sunil V L @ 2024-01-15  4:44 UTC (permalink / raw)
  To: Ni, Ray
  Cc: devel@edk2.groups.io, lichao@loongson.cn, Kinney, Michael D,
	Laszlo Ersek, Gerd Hoffmann

On Mon, Jan 15, 2024 at 01:13:03AM +0000, Ni, Ray wrote:
> Sunil,
> I am glad that you don't have concern about the direction.
> 
> "RiscV" is good to me.
> 
> 
Thanks!, Ray. So, shall I send patches to reorganize this?

Thanks,
Sunil


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#113791): https://edk2.groups.io/g/devel/message/113791
Mute This Topic: https://groups.io/mt/103679850/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
  2024-01-15  4:44     ` Sunil V L
@ 2024-01-15  5:44       ` Ni, Ray
  2024-01-15  6:17         ` Chao Li
  0 siblings, 1 reply; 17+ messages in thread
From: Ni, Ray @ 2024-01-15  5:44 UTC (permalink / raw)
  To: Sunil V L
  Cc: devel@edk2.groups.io, lichao@loongson.cn, Kinney, Michael D,
	Laszlo Ersek, Gerd Hoffmann

That will be great!

Thanks,
Ray
> -----Original Message-----
> From: Sunil V L <sunilvl@ventanamicro.com>
> Sent: Monday, January 15, 2024 12:44 PM
> To: Ni, Ray <ray.ni@intel.com>
> Cc: devel@edk2.groups.io; lichao@loongson.cn; Kinney, Michael D
> <michael.d.kinney@intel.com>; Laszlo Ersek <lersek@redhat.com>; Gerd
> Hoffmann <kraxel@redhat.com>
> Subject: Re: RFC: Folder layout change in UefiCpuPkg
> 
> On Mon, Jan 15, 2024 at 01:13:03AM +0000, Ni, Ray wrote:
> > Sunil,
> > I am glad that you don't have concern about the direction.
> >
> > "RiscV" is good to me.
> >
> >
> Thanks!, Ray. So, shall I send patches to reorganize this?
> 
> Thanks,
> Sunil


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#113795): https://edk2.groups.io/g/devel/message/113795
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/1913456212/xyzzy [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
  2024-01-15  5:44       ` Ni, Ray
@ 2024-01-15  6:17         ` Chao Li
  2024-01-15  6:46           ` Sunil V L
  0 siblings, 1 reply; 17+ messages in thread
From: Chao Li @ 2024-01-15  6:17 UTC (permalink / raw)
  To: devel, ray.ni, Sunil V L; +Cc: Kinney, Michael D, Laszlo Ersek, Gerd Hoffmann

[-- Attachment #1: Type: text/plain, Size: 1370 bytes --]

Ray and Sunil,

I plan send two example PRs to github to  show how to deal the INFs, one 
to adjust current INF, and another PR to move the libraris to the 
current folders. I hope after these two PRs are sent we can decide what 
to do with the INF. What do you think of this plan?


Thanks,
Chao
On 2024/1/15 13:44, Ni, Ray wrote:
> That will be great!
>
> Thanks,
> Ray
>> -----Original Message-----
>> From: Sunil V L<sunilvl@ventanamicro.com>
>> Sent: Monday, January 15, 2024 12:44 PM
>> To: Ni, Ray<ray.ni@intel.com>
>> Cc:devel@edk2.groups.io;lichao@loongson.cn; Kinney, Michael D
>> <michael.d.kinney@intel.com>; Laszlo Ersek<lersek@redhat.com>; Gerd
>> Hoffmann<kraxel@redhat.com>
>> Subject: Re: RFC: Folder layout change in UefiCpuPkg
>>
>> On Mon, Jan 15, 2024 at 01:13:03AM +0000, Ni, Ray wrote:
>>> Sunil,
>>> I am glad that you don't have concern about the direction.
>>>
>>> "RiscV" is good to me.
>>>
>>>
>> Thanks!, Ray. So, shall I send patches to reorganize this?
>>
>> Thanks,
>> Sunil
>
> 
>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#113796): https://edk2.groups.io/g/devel/message/113796
Mute This Topic: https://groups.io/mt/103679850/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



[-- Attachment #2: Type: text/html, Size: 3285 bytes --]

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
  2024-01-15  6:17         ` Chao Li
@ 2024-01-15  6:46           ` Sunil V L
  2024-01-15  8:11             ` Ni, Ray
  0 siblings, 1 reply; 17+ messages in thread
From: Sunil V L @ 2024-01-15  6:46 UTC (permalink / raw)
  To: Chao Li; +Cc: devel, ray.ni, Kinney, Michael D, Laszlo Ersek, Gerd Hoffmann

On Mon, Jan 15, 2024 at 02:17:09PM +0800, Chao Li wrote:
> Ray and Sunil,
> 
> I plan send two example PRs to github to  show how to deal the INFs, one to
> adjust current INF, and another PR to move the libraris to the current
> folders. I hope after these two PRs are sent we can decide what to do with
> the INF. What do you think of this plan?
> 
> 
Hi Chao, That's fine with me.

Thanks,
Sunil
> Thanks,
> Chao
> On 2024/1/15 13:44, Ni, Ray wrote:
> > That will be great!
> > 
> > Thanks,
> > Ray
> > > -----Original Message-----
> > > From: Sunil V L<sunilvl@ventanamicro.com>
> > > Sent: Monday, January 15, 2024 12:44 PM
> > > To: Ni, Ray<ray.ni@intel.com>
> > > Cc:devel@edk2.groups.io;lichao@loongson.cn; Kinney, Michael D
> > > <michael.d.kinney@intel.com>; Laszlo Ersek<lersek@redhat.com>; Gerd
> > > Hoffmann<kraxel@redhat.com>
> > > Subject: Re: RFC: Folder layout change in UefiCpuPkg
> > > 
> > > On Mon, Jan 15, 2024 at 01:13:03AM +0000, Ni, Ray wrote:
> > > > Sunil,
> > > > I am glad that you don't have concern about the direction.
> > > > 
> > > > "RiscV" is good to me.
> > > > 
> > > > 
> > > Thanks!, Ray. So, shall I send patches to reorganize this?
> > > 
> > > Thanks,
> > > Sunil
> > 
> > 
> > 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#113797): https://edk2.groups.io/g/devel/message/113797
Mute This Topic: https://groups.io/mt/103679850/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
  2024-01-15  6:46           ` Sunil V L
@ 2024-01-15  8:11             ` Ni, Ray
  2024-01-16  0:59               ` Chao Li
       [not found]               ` <17AAAE0FD29564BB.7072@groups.io>
  0 siblings, 2 replies; 17+ messages in thread
From: Ni, Ray @ 2024-01-15  8:11 UTC (permalink / raw)
  To: Sunil V L, Chao Li
  Cc: devel@edk2.groups.io, Kinney, Michael D, Laszlo Ersek,
	Gerd Hoffmann

Yes. Fine to me. Thanks!

Thanks,
Ray
> -----Original Message-----
> From: Sunil V L <sunilvl@ventanamicro.com>
> Sent: Monday, January 15, 2024 2:46 PM
> To: Chao Li <lichao@loongson.cn>
> Cc: devel@edk2.groups.io; Ni, Ray <ray.ni@intel.com>; Kinney, Michael D
> <michael.d.kinney@intel.com>; Laszlo Ersek <lersek@redhat.com>; Gerd
> Hoffmann <kraxel@redhat.com>
> Subject: Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
> 
> On Mon, Jan 15, 2024 at 02:17:09PM +0800, Chao Li wrote:
> > Ray and Sunil,
> >
> > I plan send two example PRs to github to  show how to deal the INFs, one to
> > adjust current INF, and another PR to move the libraris to the current
> > folders. I hope after these two PRs are sent we can decide what to do with
> > the INF. What do you think of this plan?
> >
> >
> Hi Chao, That's fine with me.
> 
> Thanks,
> Sunil
> > Thanks,
> > Chao
> > On 2024/1/15 13:44, Ni, Ray wrote:
> > > That will be great!
> > >
> > > Thanks,
> > > Ray
> > > > -----Original Message-----
> > > > From: Sunil V L<sunilvl@ventanamicro.com>
> > > > Sent: Monday, January 15, 2024 12:44 PM
> > > > To: Ni, Ray<ray.ni@intel.com>
> > > > Cc:devel@edk2.groups.io;lichao@loongson.cn; Kinney, Michael D
> > > > <michael.d.kinney@intel.com>; Laszlo Ersek<lersek@redhat.com>; Gerd
> > > > Hoffmann<kraxel@redhat.com>
> > > > Subject: Re: RFC: Folder layout change in UefiCpuPkg
> > > >
> > > > On Mon, Jan 15, 2024 at 01:13:03AM +0000, Ni, Ray wrote:
> > > > > Sunil,
> > > > > I am glad that you don't have concern about the direction.
> > > > >
> > > > > "RiscV" is good to me.
> > > > >
> > > > >
> > > > Thanks!, Ray. So, shall I send patches to reorganize this?
> > > >
> > > > Thanks,
> > > > Sunil
> > >
> > > 
> > >


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#113806): https://edk2.groups.io/g/devel/message/113806
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/1913456212/xyzzy [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
  2024-01-12 10:19 ` Sunil V L
  2024-01-15  1:13   ` Ni, Ray
@ 2024-01-15  8:48   ` Laszlo Ersek
  1 sibling, 0 replies; 17+ messages in thread
From: Laszlo Ersek @ 2024-01-15  8:48 UTC (permalink / raw)
  To: Sunil V L, Ni, Ray
  Cc: devel@edk2.groups.io, lichao@loongson.cn, Kinney, Michael D,
	Gerd Hoffmann

On 1/12/24 11:19, Sunil V L wrote:
> Hi Ray,
> 
> On Fri, Jan 12, 2024 at 09:12:34AM +0000, Ni, Ray wrote:
>> 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 libraries in Library folder.
>>
>> Movement of the first 3 ones put the RiscV components under existing folders.
>> Movement of the last one creates the UefiCpuPkg/CpuTimerDxe 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 defined 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><LibraryClassName>[<Dependency>]/[<CpuArch>[/<Vendor>]]
>>
>>
> Your proposal looks good to me except better to keep directory name as
> RiscV as in other packages.

no objections from me



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#113810): https://edk2.groups.io/g/devel/message/113810
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/1913456212/xyzzy [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
  2024-01-15  8:11             ` Ni, Ray
@ 2024-01-16  0:59               ` Chao Li
       [not found]               ` <17AAAE0FD29564BB.7072@groups.io>
  1 sibling, 0 replies; 17+ messages in thread
From: Chao Li @ 2024-01-16  0:59 UTC (permalink / raw)
  To: Ni, Ray, Sunil V L
  Cc: devel@edk2.groups.io, Kinney, Michael D, Laszlo Ersek,
	Gerd Hoffmann

[-- Attachment #1: Type: text/plain, Size: 2157 bytes --]

OK, let me finish this work tomorrow, I'm a little busy today.


Thanks,
Chao
On 2024/1/15 16:11, Ni, Ray wrote:
> Yes. Fine to me. Thanks!
>
> Thanks,
> Ray
>> -----Original Message-----
>> From: Sunil V L<sunilvl@ventanamicro.com>
>> Sent: Monday, January 15, 2024 2:46 PM
>> To: Chao Li<lichao@loongson.cn>
>> Cc:devel@edk2.groups.io; Ni, Ray<ray.ni@intel.com>; Kinney, Michael D
>> <michael.d.kinney@intel.com>; Laszlo Ersek<lersek@redhat.com>; Gerd
>> Hoffmann<kraxel@redhat.com>
>> Subject: Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
>>
>> On Mon, Jan 15, 2024 at 02:17:09PM +0800, Chao Li wrote:
>>> Ray and Sunil,
>>>
>>> I plan send two example PRs to github to  show how to deal the INFs, one to
>>> adjust current INF, and another PR to move the libraris to the current
>>> folders. I hope after these two PRs are sent we can decide what to do with
>>> the INF. What do you think of this plan?
>>>
>>>
>> Hi Chao, That's fine with me.
>>
>> Thanks,
>> Sunil
>>> Thanks,
>>> Chao
>>> On 2024/1/15 13:44, Ni, Ray wrote:
>>>> That will be great!
>>>>
>>>> Thanks,
>>>> Ray
>>>>> -----Original Message-----
>>>>> From: Sunil V L<sunilvl@ventanamicro.com>
>>>>> Sent: Monday, January 15, 2024 12:44 PM
>>>>> To: Ni, Ray<ray.ni@intel.com>
>>>>> Cc:devel@edk2.groups.io;lichao@loongson.cn; Kinney, Michael D
>>>>> <michael.d.kinney@intel.com>; Laszlo Ersek<lersek@redhat.com>; Gerd
>>>>> Hoffmann<kraxel@redhat.com>
>>>>> Subject: Re: RFC: Folder layout change in UefiCpuPkg
>>>>>
>>>>> On Mon, Jan 15, 2024 at 01:13:03AM +0000, Ni, Ray wrote:
>>>>>> Sunil,
>>>>>> I am glad that you don't have concern about the direction.
>>>>>>
>>>>>> "RiscV" is good to me.
>>>>>>
>>>>>>
>>>>> Thanks!, Ray. So, shall I send patches to reorganize this?
>>>>>
>>>>> Thanks,
>>>>> Sunil
>>>> 
>>>>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#113858): https://edk2.groups.io/g/devel/message/113858
Mute This Topic: https://groups.io/mt/103679850/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



[-- Attachment #2: Type: text/html, Size: 4868 bytes --]

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
       [not found]               ` <17AAAE0FD29564BB.7072@groups.io>
@ 2024-01-18  8:27                 ` Chao Li
  2024-01-19 10:17                   ` Ni, Ray
  0 siblings, 1 reply; 17+ messages in thread
From: Chao Li @ 2024-01-18  8:27 UTC (permalink / raw)
  To: Ni, Ray, Sunil V L
  Cc: devel@edk2.groups.io, Kinney, Michael D, Laszlo Ersek,
	Gerd Hoffmann, Abner Chang

[-- Attachment #1: Type: text/plain, Size: 2967 bytes --]

Hi Ray and Sunil,

Sorry, I'm late, I have very busy these days.

I created two PRs in my private repo.

*Plan A:*

Moved most of LoongArch libraries and drivers to current matching 
folders, removed LoongArch private INF, adjusted common INF.

URL: https://github.com/kilaterlee/edk2/pull/8


*Plan B:*

Moved most of LoongArch libraries and drivers to current matching 
folders, keeping to use LoongArch private INF.

URL: https://github.com/kilaterlee/edk2/pull/9


After you reading these two PRs, let's discuss how to deal with INF.

I prefer plan A, it is better to use the same INF for all ARCHs and have 
only one INF per modules. I was discussed this plan with Ray and Abner, 
but it was not allowed at the time.


Thanks,
Chao
On 2024/1/16 08:59, Chao Li wrote:
>
> OK, let me finish this work tomorrow, I'm a little busy today.
>
> On 2024/1/15 16:11, Ni, Ray wrote:
>> Yes. Fine to me. Thanks!
>>
>> Thanks,
>> Ray
>>> -----Original Message-----
>>> From: Sunil V L<sunilvl@ventanamicro.com>
>>> Sent: Monday, January 15, 2024 2:46 PM
>>> To: Chao Li<lichao@loongson.cn>
>>> Cc:devel@edk2.groups.io; Ni, Ray<ray.ni@intel.com>; Kinney, Michael D
>>> <michael.d.kinney@intel.com>; Laszlo Ersek<lersek@redhat.com>; Gerd
>>> Hoffmann<kraxel@redhat.com>
>>> Subject: Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
>>>
>>> On Mon, Jan 15, 2024 at 02:17:09PM +0800, Chao Li wrote:
>>>> Ray and Sunil,
>>>>
>>>> I plan send two example PRs to github to  show how to deal the INFs, one to
>>>> adjust current INF, and another PR to move the libraris to the current
>>>> folders. I hope after these two PRs are sent we can decide what to do with
>>>> the INF. What do you think of this plan?
>>>>
>>>>
>>> Hi Chao, That's fine with me.
>>>
>>> Thanks,
>>> Sunil
>>>> Thanks,
>>>> Chao
>>>> On 2024/1/15 13:44, Ni, Ray wrote:
>>>>> That will be great!
>>>>>
>>>>> Thanks,
>>>>> Ray
>>>>>> -----Original Message-----
>>>>>> From: Sunil V L<sunilvl@ventanamicro.com>
>>>>>> Sent: Monday, January 15, 2024 12:44 PM
>>>>>> To: Ni, Ray<ray.ni@intel.com>
>>>>>> Cc:devel@edk2.groups.io;lichao@loongson.cn; Kinney, Michael D
>>>>>> <michael.d.kinney@intel.com>; Laszlo Ersek<lersek@redhat.com>; Gerd
>>>>>> Hoffmann<kraxel@redhat.com>
>>>>>> Subject: Re: RFC: Folder layout change in UefiCpuPkg
>>>>>>
>>>>>> On Mon, Jan 15, 2024 at 01:13:03AM +0000, Ni, Ray wrote:
>>>>>>> Sunil,
>>>>>>> I am glad that you don't have concern about the direction.
>>>>>>>
>>>>>>> "RiscV" is good to me.
>>>>>>>
>>>>>>>
>>>>>> Thanks!, Ray. So, shall I send patches to reorganize this?
>>>>>>
>>>>>> Thanks,
>>>>>> Sunil
> 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#113987): https://edk2.groups.io/g/devel/message/113987
Mute This Topic: https://groups.io/mt/103679850/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



[-- Attachment #2: Type: text/html, Size: 7154 bytes --]

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
  2024-01-18  8:27                 ` Chao Li
@ 2024-01-19 10:17                   ` Ni, Ray
  2024-01-19 13:47                     ` Laszlo Ersek
  0 siblings, 1 reply; 17+ messages in thread
From: Ni, Ray @ 2024-01-19 10:17 UTC (permalink / raw)
  To: devel@edk2.groups.io, lichao@loongson.cn, Sunil V L
  Cc: Kinney, Michael D, Laszlo Ersek, Gerd Hoffmann, Abner Chang

[-- Attachment #1: Type: text/plain, Size: 4246 bytes --]

Chao,

In the plan A, CpuDxe.inf contains reference to gUefiCpuPkgTokenSpaceGuid.PcdCpuExceptionVectorBaseAddress in [Pcd] section. But I guess it’s only needed by LoongArch64. That’s why I didn’t like the common-inf idea.

But after looking at the other INF changes, I changed my mind. I think the common-inf looks good as the other INF files don’t have so much difference between LoonArch and x86.

So, I am ok with plan A. Thanks for preparing both changes for review.

Mike, Laszlo, Gerd, Abner, any comments?


Thanks,
Ray
From: Chao Li <lichao@loongson.cn>
Sent: Thursday, January 18, 2024 4:27 PM
To: Ni, Ray <ray.ni@intel.com>; Sunil V L <sunilvl@ventanamicro.com>
Cc: devel@edk2.groups.io; Kinney, Michael D <michael.d.kinney@intel.com>; Laszlo Ersek <lersek@redhat.com>; Gerd Hoffmann <kraxel@redhat.com>; Abner Chang <abner.chang@amd.com>
Subject: Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg


Hi Ray and Sunil,

Sorry, I'm late, I have very busy these days.

I created two PRs in my private repo.

Plan A:

Moved most of LoongArch libraries and drivers to current matching folders, removed LoongArch private INF, adjusted common INF.

URL: https://github.com/kilaterlee/edk2/pull/8



Plan B:

Moved most of LoongArch libraries and drivers to current matching folders, keeping to use LoongArch private INF.

URL: https://github.com/kilaterlee/edk2/pull/9



After you reading these two PRs, let's discuss how to deal with INF.

I prefer plan A, it is better to use the same INF for all ARCHs and have only one INF per modules. I was discussed this plan with Ray and Abner, but it was not allowed at the time.

Thanks,
Chao
On 2024/1/16 08:59, Chao Li wrote:

OK, let me finish this work tomorrow, I'm a little busy today.
On 2024/1/15 16:11, Ni, Ray wrote:

Yes. Fine to me. Thanks!



Thanks,

Ray

-----Original Message-----

From: Sunil V L <sunilvl@ventanamicro.com><mailto:sunilvl@ventanamicro.com>

Sent: Monday, January 15, 2024 2:46 PM

To: Chao Li <lichao@loongson.cn><mailto:lichao@loongson.cn>

Cc: devel@edk2.groups.io<mailto:devel@edk2.groups.io>; Ni, Ray <ray.ni@intel.com><mailto:ray.ni@intel.com>; Kinney, Michael D

<michael.d.kinney@intel.com><mailto:michael.d.kinney@intel.com>; Laszlo Ersek <lersek@redhat.com><mailto:lersek@redhat.com>; Gerd

Hoffmann <kraxel@redhat.com><mailto:kraxel@redhat.com>

Subject: Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg



On Mon, Jan 15, 2024 at 02:17:09PM +0800, Chao Li wrote:

Ray and Sunil,



I plan send two example PRs to github to  show how to deal the INFs, one to

adjust current INF, and another PR to move the libraris to the current

folders. I hope after these two PRs are sent we can decide what to do with

the INF. What do you think of this plan?





Hi Chao, That's fine with me.



Thanks,

Sunil

Thanks,

Chao

On 2024/1/15 13:44, Ni, Ray wrote:

That will be great!



Thanks,

Ray

-----Original Message-----

From: Sunil V L<sunilvl@ventanamicro.com><mailto:sunilvl@ventanamicro.com>

Sent: Monday, January 15, 2024 12:44 PM

To: Ni, Ray<ray.ni@intel.com><mailto:ray.ni@intel.com>

Cc:devel@edk2.groups.io;lichao@loongson.cn<mailto:Cc:devel@edk2.groups.io;lichao@loongson.cn>; Kinney, Michael D

<michael.d.kinney@intel.com><mailto:michael.d.kinney@intel.com>; Laszlo Ersek<lersek@redhat.com><mailto:lersek@redhat.com>; Gerd

Hoffmann<kraxel@redhat.com><mailto:kraxel@redhat.com>

Subject: Re: RFC: Folder layout change in UefiCpuPkg



On Mon, Jan 15, 2024 at 01:13:03AM +0000, Ni, Ray wrote:

Sunil,

I am glad that you don't have concern about the direction.



"RiscV" is good to me.





Thanks!, Ray. So, shall I send patches to reorganize this?



Thanks,

Sunil



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#114040): https://edk2.groups.io/g/devel/message/114040
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/1913456212/xyzzy [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



[-- Attachment #2: Type: text/html, Size: 11014 bytes --]

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
  2024-01-19 10:17                   ` Ni, Ray
@ 2024-01-19 13:47                     ` Laszlo Ersek
  2024-01-23  3:10                       ` Chang, Abner via groups.io
  0 siblings, 1 reply; 17+ messages in thread
From: Laszlo Ersek @ 2024-01-19 13:47 UTC (permalink / raw)
  To: Ni, Ray, devel@edk2.groups.io, lichao@loongson.cn, Sunil V L
  Cc: Kinney, Michael D, Gerd Hoffmann, Abner Chang

On 1/19/24 11:17, Ni, Ray wrote:
> Chao,
> 
>  
> 
> In the plan A, CpuDxe.inf contains reference to
> gUefiCpuPkgTokenSpaceGuid.PcdCpuExceptionVectorBaseAddress in [Pcd]
> section. But I guess it’s only needed by LoongArch64. That’s why I
> didn’t like the common-inf idea.
> 
>  
> 
> But after looking at the other INF changes, I changed my mind. I think
> the common-inf looks good as the other INF files don’t have so much
> difference between LoonArch and x86.
> 
>  
> 
> So, I am ok with plan A. Thanks for preparing both changes for review.
> 
>  
> 
> Mike, Laszlo, Gerd, Abner, any comments?

I'm also OK with plan A.

Thanks
Laszlo

> 
>  
> 
>  
> 
> Thanks,
> 
> Ray
> 
> *From:* Chao Li <lichao@loongson.cn>
> *Sent:* Thursday, January 18, 2024 4:27 PM
> *To:* Ni, Ray <ray.ni@intel.com>; Sunil V L <sunilvl@ventanamicro.com>
> *Cc:* devel@edk2.groups.io; Kinney, Michael D
> <michael.d.kinney@intel.com>; Laszlo Ersek <lersek@redhat.com>; Gerd
> Hoffmann <kraxel@redhat.com>; Abner Chang <abner.chang@amd.com>
> *Subject:* Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
> 
>  
> 
> Hi Ray and Sunil,
> 
> Sorry, I'm late, I have very busy these days.
> 
> I created two PRs in my private repo.
> 
> *Plan A:*
> 
> Moved most of LoongArch libraries and drivers to current matching
> folders, removed LoongArch private INF, adjusted common INF.
> 
> URL: https://github.com/kilaterlee/edk2/pull/8
> <https://github.com/kilaterlee/edk2/pull/8>
> 
>  
> 
> *Plan B:*
> 
> Moved most of LoongArch libraries and drivers to current matching
> folders, keeping to use LoongArch private INF.
> 
> URL: https://github.com/kilaterlee/edk2/pull/9
> <https://github.com/kilaterlee/edk2/pull/9>
> 
>  
> 
> After you reading these two PRs, let's discuss how to deal with INF.
> 
> I prefer plan A, it is better to use the same INF for all ARCHs and have
> only one INF per modules. I was discussed this plan with Ray and Abner,
> but it was not allowed at the time.
> 
>  
> 
> Thanks,
> Chao
> 
> On 2024/1/16 08:59, Chao Li wrote:
> 
>     OK, let me finish this work tomorrow, I'm a little busy today.
> 
>     On 2024/1/15 16:11, Ni, Ray wrote:
> 
>         Yes. Fine to me. Thanks!
> 
>          
> 
>         Thanks,
> 
>         Ray
> 
>             -----Original Message-----
> 
>             From: Sunil V L <sunilvl@ventanamicro.com> <mailto:sunilvl@ventanamicro.com>
> 
>             Sent: Monday, January 15, 2024 2:46 PM
> 
>             To: Chao Li <lichao@loongson.cn> <mailto:lichao@loongson.cn>
> 
>             Cc: devel@edk2.groups.io <mailto:devel@edk2.groups.io>; Ni, Ray <ray.ni@intel.com> <mailto:ray.ni@intel.com>; Kinney, Michael D
> 
>             <michael.d.kinney@intel.com> <mailto:michael.d.kinney@intel.com>; Laszlo Ersek <lersek@redhat.com> <mailto:lersek@redhat.com>; Gerd
> 
>             Hoffmann <kraxel@redhat.com> <mailto:kraxel@redhat.com>
> 
>             Subject: Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
> 
>              
> 
>             On Mon, Jan 15, 2024 at 02:17:09PM +0800, Chao Li wrote:
> 
>                 Ray and Sunil,
> 
>                  
> 
>                 I plan send two example PRs to github to  show how to deal the INFs, one to
> 
>                 adjust current INF, and another PR to move the libraris to the current
> 
>                 folders. I hope after these two PRs are sent we can decide what to do with
> 
>                 the INF. What do you think of this plan?
> 
>                  
> 
>                  
> 
>             Hi Chao, That's fine with me.
> 
>              
> 
>             Thanks,
> 
>             Sunil
> 
>                 Thanks,
> 
>                 Chao
> 
>                 On 2024/1/15 13:44, Ni, Ray wrote:
> 
>                     That will be great!
> 
>                      
> 
>                     Thanks,
> 
>                     Ray
> 
>                         -----Original Message-----
> 
>                         From: Sunil V L<sunilvl@ventanamicro.com> <mailto:sunilvl@ventanamicro.com>
> 
>                         Sent: Monday, January 15, 2024 12:44 PM
> 
>                         To: Ni, Ray<ray.ni@intel.com> <mailto:ray.ni@intel.com>
> 
>                         Cc:devel@edk2.groups.io;lichao@loongson.cn <mailto:Cc:devel@edk2.groups.io;lichao@loongson.cn>; Kinney, Michael D
> 
>                         <michael.d.kinney@intel.com> <mailto:michael.d.kinney@intel.com>; Laszlo Ersek<lersek@redhat.com> <mailto:lersek@redhat.com>; Gerd
> 
>                         Hoffmann<kraxel@redhat.com> <mailto:kraxel@redhat.com>
> 
>                         Subject: Re: RFC: Folder layout change in UefiCpuPkg
> 
>                          
> 
>                         On Mon, Jan 15, 2024 at 01:13:03AM +0000, Ni, Ray wrote:
> 
>                             Sunil,
> 
>                             I am glad that you don't have concern about the direction.
> 
>                              
> 
>                             "RiscV" is good to me.
> 
>                              
> 
>                              
> 
>                         Thanks!, Ray. So, shall I send patches to reorganize this?
> 
>                          
> 
>                         Thanks,
> 
>                         Sunil
> 
>     
> 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#114046): https://edk2.groups.io/g/devel/message/114046
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/1913456212/xyzzy [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
  2024-01-19 13:47                     ` Laszlo Ersek
@ 2024-01-23  3:10                       ` Chang, Abner via groups.io
  2024-01-23  4:39                         ` Chao Li
  0 siblings, 1 reply; 17+ messages in thread
From: Chang, Abner via groups.io @ 2024-01-23  3:10 UTC (permalink / raw)
  To: Laszlo Ersek, Ni, Ray, devel@edk2.groups.io, lichao@loongson.cn,
	Sunil V L
  Cc: Kinney, Michael D, Gerd Hoffmann

[AMD Official Use Only - General]

HI all,
I have no problem with the plan A, except the folder structure under CpuExecptionHandlerLib.
It has a LoongArch folder that contains the common source files for LoongArch and LoongArch64 folder under LoongArch for 64-buite architecture. This folder structure is different from other lib such as MpInitLib and CpuTimerLib.
Do you have 32-bit LoongArch? If not for now, then you probably can move the files under LoongArch to LoongArch64. Len you can a consistent folder structure for LoongArch.

Thanks
Abner

> -----Original Message-----
> From: Laszlo Ersek <lersek@redhat.com>
> Sent: Friday, January 19, 2024 9:48 PM
> To: Ni, Ray <ray.ni@intel.com>; devel@edk2.groups.io; lichao@loongson.cn;
> Sunil V L <sunilvl@ventanamicro.com>
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gerd Hoffmann
> <kraxel@redhat.com>; Chang, Abner <Abner.Chang@amd.com>
> Subject: Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
>
> Caution: This message originated from an External Source. Use proper caution
> when opening attachments, clicking links, or responding.
>
>
> On 1/19/24 11:17, Ni, Ray wrote:
> > Chao,
> >
> >
> >
> > In the plan A, CpuDxe.inf contains reference to
> > gUefiCpuPkgTokenSpaceGuid.PcdCpuExceptionVectorBaseAddress in [Pcd]
> > section. But I guess it’s only needed by LoongArch64. That’s why I
> > didn’t like the common-inf idea.
> >
> >
> >
> > But after looking at the other INF changes, I changed my mind. I think
> > the common-inf looks good as the other INF files don’t have so much
> > difference between LoonArch and x86.
> >
> >
> >
> > So, I am ok with plan A. Thanks for preparing both changes for review.
> >
> >
> >
> > Mike, Laszlo, Gerd, Abner, any comments?
>
> I'm also OK with plan A.
>
> Thanks
> Laszlo
>
> >
> >
> >
> >
> >
> > Thanks,
> >
> > Ray
> >
> > *From:* Chao Li <lichao@loongson.cn>
> > *Sent:* Thursday, January 18, 2024 4:27 PM
> > *To:* Ni, Ray <ray.ni@intel.com>; Sunil V L <sunilvl@ventanamicro.com>
> > *Cc:* devel@edk2.groups.io; Kinney, Michael D
> > <michael.d.kinney@intel.com>; Laszlo Ersek <lersek@redhat.com>; Gerd
> > Hoffmann <kraxel@redhat.com>; Abner Chang <abner.chang@amd.com>
> > *Subject:* Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
> >
> >
> >
> > Hi Ray and Sunil,
> >
> > Sorry, I'm late, I have very busy these days.
> >
> > I created two PRs in my private repo.
> >
> > *Plan A:*
> >
> > Moved most of LoongArch libraries and drivers to current matching
> > folders, removed LoongArch private INF, adjusted common INF.
> >
> > URL: https://github.com/kilaterlee/edk2/pull/8
> > <https://github.com/kilaterlee/edk2/pull/8>
> >
> >
> >
> > *Plan B:*
> >
> > Moved most of LoongArch libraries and drivers to current matching
> > folders, keeping to use LoongArch private INF.
> >
> > URL: https://github.com/kilaterlee/edk2/pull/9
> > <https://github.com/kilaterlee/edk2/pull/9>
> >
> >
> >
> > After you reading these two PRs, let's discuss how to deal with INF.
> >
> > I prefer plan A, it is better to use the same INF for all ARCHs and have
> > only one INF per modules. I was discussed this plan with Ray and Abner,
> > but it was not allowed at the time.
> >
> >
> >
> > Thanks,
> > Chao
> >
> > On 2024/1/16 08:59, Chao Li wrote:
> >
> >     OK, let me finish this work tomorrow, I'm a little busy today.
> >
> >     On 2024/1/15 16:11, Ni, Ray wrote:
> >
> >         Yes. Fine to me. Thanks!
> >
> >
> >
> >         Thanks,
> >
> >         Ray
> >
> >             -----Original Message-----
> >
> >             From: Sunil V L <sunilvl@ventanamicro.com>
> <mailto:sunilvl@ventanamicro.com>
> >
> >             Sent: Monday, January 15, 2024 2:46 PM
> >
> >             To: Chao Li <lichao@loongson.cn> <mailto:lichao@loongson.cn>
> >
> >             Cc: devel@edk2.groups.io <mailto:devel@edk2.groups.io>; Ni, Ray
> <ray.ni@intel.com> <mailto:ray.ni@intel.com>; Kinney, Michael D
> >
> >             <michael.d.kinney@intel.com> <mailto:michael.d.kinney@intel.com>;
> Laszlo Ersek <lersek@redhat.com> <mailto:lersek@redhat.com>; Gerd
> >
> >             Hoffmann <kraxel@redhat.com> <mailto:kraxel@redhat.com>
> >
> >             Subject: Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
> >
> >
> >
> >             On Mon, Jan 15, 2024 at 02:17:09PM +0800, Chao Li wrote:
> >
> >                 Ray and Sunil,
> >
> >
> >
> >                 I plan send two example PRs to github to  show how to deal the
> INFs, one to
> >
> >                 adjust current INF, and another PR to move the libraris to the current
> >
> >                 folders. I hope after these two PRs are sent we can decide what to do
> with
> >
> >                 the INF. What do you think of this plan?
> >
> >
> >
> >
> >
> >             Hi Chao, That's fine with me.
> >
> >
> >
> >             Thanks,
> >
> >             Sunil
> >
> >                 Thanks,
> >
> >                 Chao
> >
> >                 On 2024/1/15 13:44, Ni, Ray wrote:
> >
> >                     That will be great!
> >
> >
> >
> >                     Thanks,
> >
> >                     Ray
> >
> >                         -----Original Message-----
> >
> >                         From: Sunil V L<sunilvl@ventanamicro.com>
> <mailto:sunilvl@ventanamicro.com>
> >
> >                         Sent: Monday, January 15, 2024 12:44 PM
> >
> >                         To: Ni, Ray<ray.ni@intel.com> <mailto:ray.ni@intel.com>
> >
> >                         Cc:devel@edk2.groups.io;lichao@loongson.cn
> <mailto:Cc:devel@edk2.groups.io;lichao@loongson.cn>; Kinney, Michael D
> >
> >                         <michael.d.kinney@intel.com>
> <mailto:michael.d.kinney@intel.com>; Laszlo Ersek<lersek@redhat.com>
> <mailto:lersek@redhat.com>; Gerd
> >
> >                         Hoffmann<kraxel@redhat.com> <mailto:kraxel@redhat.com>
> >
> >                         Subject: Re: RFC: Folder layout change in UefiCpuPkg
> >
> >
> >
> >                         On Mon, Jan 15, 2024 at 01:13:03AM +0000, Ni, Ray wrote:
> >
> >                             Sunil,
> >
> >                             I am glad that you don't have concern about the direction.
> >
> >
> >
> >                             "RiscV" is good to me.
> >
> >
> >
> >
> >
> >                         Thanks!, Ray. So, shall I send patches to reorganize this?
> >
> >
> >
> >                         Thanks,
> >
> >                         Sunil
> >
> >     
> >



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#114166): https://edk2.groups.io/g/devel/message/114166
Mute This Topic: https://groups.io/mt/103679850/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
  2024-01-23  3:10                       ` Chang, Abner via groups.io
@ 2024-01-23  4:39                         ` Chao Li
  2024-01-23  5:02                           ` Chang, Abner via groups.io
  0 siblings, 1 reply; 17+ messages in thread
From: Chao Li @ 2024-01-23  4:39 UTC (permalink / raw)
  To: Chang, Abner, Laszlo Ersek, Ni, Ray, devel@edk2.groups.io,
	Sunil V L
  Cc: Kinney, Michael D, Gerd Hoffmann

[-- Attachment #1: Type: text/plain, Size: 7229 bytes --]

Hi Abner,

The ExceptionLib is different from other libs such as Mp and Timer. 
Since ExceptionLib can provide for 32-bit service for LoongArch32 in the 
future, 64-bit private files are located in LoongArch/LoongArch64/ and 
the 32-bit will be located in LoongArch/LoongArch32, although the 32-bit 
LoongArch is not upstream yet.


Thanks,
Chao
On 2024/1/23 11:10, Chang, Abner wrote:
> [AMD Official Use Only - General]
>
> HI all,
> I have no problem with the plan A, except the folder structure under CpuExecptionHandlerLib.
> It has a LoongArch folder that contains the common source files for LoongArch and LoongArch64 folder under LoongArch for 64-buite architecture. This folder structure is different from other lib such as MpInitLib and CpuTimerLib.
> Do you have 32-bit LoongArch? If not for now, then you probably can move the files under LoongArch to LoongArch64. Len you can a consistent folder structure for LoongArch.
>
> Thanks
> Abner
>
>> -----Original Message-----
>> From: Laszlo Ersek<lersek@redhat.com>
>> Sent: Friday, January 19, 2024 9:48 PM
>> To: Ni, Ray<ray.ni@intel.com>;devel@edk2.groups.io;lichao@loongson.cn;
>> Sunil V L<sunilvl@ventanamicro.com>
>> Cc: Kinney, Michael D<michael.d.kinney@intel.com>; Gerd Hoffmann
>> <kraxel@redhat.com>; Chang, Abner<Abner.Chang@amd.com>
>> Subject: Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
>>
>> Caution: This message originated from an External Source. Use proper caution
>> when opening attachments, clicking links, or responding.
>>
>>
>> On 1/19/24 11:17, Ni, Ray wrote:
>>> Chao,
>>>
>>>
>>>
>>> In the plan A, CpuDxe.inf contains reference to
>>> gUefiCpuPkgTokenSpaceGuid.PcdCpuExceptionVectorBaseAddress in [Pcd]
>>> section. But I guess it’s only needed by LoongArch64. That’s why I
>>> didn’t like the common-inf idea.
>>>
>>>
>>>
>>> But after looking at the other INF changes, I changed my mind. I think
>>> the common-inf looks good as the other INF files don’t have so much
>>> difference between LoonArch and x86.
>>>
>>>
>>>
>>> So, I am ok with plan A. Thanks for preparing both changes for review.
>>>
>>>
>>>
>>> Mike, Laszlo, Gerd, Abner, any comments?
>> I'm also OK with plan A.
>>
>> Thanks
>> Laszlo
>>
>>>
>>>
>>>
>>>
>>> Thanks,
>>>
>>> Ray
>>>
>>> *From:* Chao Li<lichao@loongson.cn>
>>> *Sent:* Thursday, January 18, 2024 4:27 PM
>>> *To:* Ni, Ray<ray.ni@intel.com>; Sunil V L<sunilvl@ventanamicro.com>
>>> *Cc:*devel@edk2.groups.io; Kinney, Michael D
>>> <michael.d.kinney@intel.com>; Laszlo Ersek<lersek@redhat.com>; Gerd
>>> Hoffmann<kraxel@redhat.com>; Abner Chang<abner.chang@amd.com>
>>> *Subject:* Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
>>>
>>>
>>>
>>> Hi Ray and Sunil,
>>>
>>> Sorry, I'm late, I have very busy these days.
>>>
>>> I created two PRs in my private repo.
>>>
>>> *Plan A:*
>>>
>>> Moved most of LoongArch libraries and drivers to current matching
>>> folders, removed LoongArch private INF, adjusted common INF.
>>>
>>> URL:https://github.com/kilaterlee/edk2/pull/8
>>> <https://github.com/kilaterlee/edk2/pull/8>
>>>
>>>
>>>
>>> *Plan B:*
>>>
>>> Moved most of LoongArch libraries and drivers to current matching
>>> folders, keeping to use LoongArch private INF.
>>>
>>> URL:https://github.com/kilaterlee/edk2/pull/9
>>> <https://github.com/kilaterlee/edk2/pull/9>
>>>
>>>
>>>
>>> After you reading these two PRs, let's discuss how to deal with INF.
>>>
>>> I prefer plan A, it is better to use the same INF for all ARCHs and have
>>> only one INF per modules. I was discussed this plan with Ray and Abner,
>>> but it was not allowed at the time.
>>>
>>>
>>>
>>> Thanks,
>>> Chao
>>>
>>> On 2024/1/16 08:59, Chao Li wrote:
>>>
>>>      OK, let me finish this work tomorrow, I'm a little busy today.
>>>
>>>      On 2024/1/15 16:11, Ni, Ray wrote:
>>>
>>>          Yes. Fine to me. Thanks!
>>>
>>>
>>>
>>>          Thanks,
>>>
>>>          Ray
>>>
>>>              -----Original Message-----
>>>
>>>              From: Sunil V L<sunilvl@ventanamicro.com>
>> <mailto:sunilvl@ventanamicro.com>
>>>              Sent: Monday, January 15, 2024 2:46 PM
>>>
>>>              To: Chao Li<lichao@loongson.cn>  <mailto:lichao@loongson.cn>
>>>
>>>              Cc:devel@edk2.groups.io  <mailto:devel@edk2.groups.io>; Ni, Ray
>> <ray.ni@intel.com>  <mailto:ray.ni@intel.com>; Kinney, Michael D
>>>              <michael.d.kinney@intel.com>  <mailto:michael.d.kinney@intel.com>;
>> Laszlo Ersek<lersek@redhat.com>  <mailto:lersek@redhat.com>; Gerd
>>>              Hoffmann<kraxel@redhat.com>  <mailto:kraxel@redhat.com>
>>>
>>>              Subject: Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
>>>
>>>
>>>
>>>              On Mon, Jan 15, 2024 at 02:17:09PM +0800, Chao Li wrote:
>>>
>>>                  Ray and Sunil,
>>>
>>>
>>>
>>>                  I plan send two example PRs to github to  show how to deal the
>> INFs, one to
>>>                  adjust current INF, and another PR to move the libraris to the current
>>>
>>>                  folders. I hope after these two PRs are sent we can decide what to do
>> with
>>>                  the INF. What do you think of this plan?
>>>
>>>
>>>
>>>
>>>
>>>              Hi Chao, That's fine with me.
>>>
>>>
>>>
>>>              Thanks,
>>>
>>>              Sunil
>>>
>>>                  Thanks,
>>>
>>>                  Chao
>>>
>>>                  On 2024/1/15 13:44, Ni, Ray wrote:
>>>
>>>                      That will be great!
>>>
>>>
>>>
>>>                      Thanks,
>>>
>>>                      Ray
>>>
>>>                          -----Original Message-----
>>>
>>>                          From: Sunil V L<sunilvl@ventanamicro.com>
>> <mailto:sunilvl@ventanamicro.com>
>>>                          Sent: Monday, January 15, 2024 12:44 PM
>>>
>>>                          To: Ni, Ray<ray.ni@intel.com>  <mailto:ray.ni@intel.com>
>>>
>>>                          Cc:devel@edk2.groups.io;lichao@loongson.cn
>> <mailto:Cc:devel@edk2.groups.io;lichao@loongson.cn>; Kinney, Michael D
>>>                          <michael.d.kinney@intel.com>
>> <mailto:michael.d.kinney@intel.com>; Laszlo Ersek<lersek@redhat.com>
>> <mailto:lersek@redhat.com>; Gerd
>>>                          Hoffmann<kraxel@redhat.com>  <mailto:kraxel@redhat.com>
>>>
>>>                          Subject: Re: RFC: Folder layout change in UefiCpuPkg
>>>
>>>
>>>
>>>                          On Mon, Jan 15, 2024 at 01:13:03AM +0000, Ni, Ray wrote:
>>>
>>>                              Sunil,
>>>
>>>                              I am glad that you don't have concern about the direction.
>>>
>>>
>>>
>>>                              "RiscV" is good to me.
>>>
>>>
>>>
>>>
>>>
>>>                          Thanks!, Ray. So, shall I send patches to reorganize this?
>>>
>>>
>>>
>>>                          Thanks,
>>>
>>>                          Sunil
>>>
>>>      
>>>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#114168): https://edk2.groups.io/g/devel/message/114168
Mute This Topic: https://groups.io/mt/103679850/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



[-- Attachment #2: Type: text/html, Size: 12432 bytes --]

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [edk2-devel] RFC: Folder layout change in UefiCpuPkg
  2024-01-23  4:39                         ` Chao Li
@ 2024-01-23  5:02                           ` Chang, Abner via groups.io
  0 siblings, 0 replies; 17+ messages in thread
From: Chang, Abner via groups.io @ 2024-01-23  5:02 UTC (permalink / raw)
  To: Chao Li, devel

[-- Attachment #1: Type: text/plain, Size: 443 bytes --]

ok, I got it. I don't have further questions about plan A.

Thanks
Abner


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#114174): https://edk2.groups.io/g/devel/message/114174
Mute This Topic: https://groups.io/mt/103679850/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



[-- Attachment #2: Type: text/html, Size: 867 bytes --]

^ permalink raw reply	[flat|nested] 17+ messages in thread

end of thread, other threads:[~2024-01-23  5:03 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-01-12  9:12 [edk2-devel] RFC: Folder layout change in UefiCpuPkg Ni, Ray
2024-01-12  9:57 ` Chao Li
2024-01-12 10:19 ` Sunil V L
2024-01-15  1:13   ` Ni, Ray
2024-01-15  4:44     ` Sunil V L
2024-01-15  5:44       ` Ni, Ray
2024-01-15  6:17         ` Chao Li
2024-01-15  6:46           ` Sunil V L
2024-01-15  8:11             ` Ni, Ray
2024-01-16  0:59               ` Chao Li
     [not found]               ` <17AAAE0FD29564BB.7072@groups.io>
2024-01-18  8:27                 ` Chao Li
2024-01-19 10:17                   ` Ni, Ray
2024-01-19 13:47                     ` Laszlo Ersek
2024-01-23  3:10                       ` Chang, Abner via groups.io
2024-01-23  4:39                         ` Chao Li
2024-01-23  5:02                           ` Chang, Abner via groups.io
2024-01-15  8:48   ` Laszlo Ersek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox