* [edk2-devel] [PATCH] BaseTools: Remove Duplicate sets of SkuName and SkuId from allskuset @ 2023-12-25 15:27 Ashraf Ali S 2024-01-08 0:29 ` Yuwei Chen 0 siblings, 1 reply; 11+ messages in thread From: Ashraf Ali S @ 2023-12-25 15:27 UTC (permalink / raw) To: devel Cc: Ashraf Ali S, Yuwei Chen, Rebecca Cran, Liming Gao, Bob Feng, Amy Chan, Sai Chaganty Currently when the platform has many SKUs then allskuset will be having so many duplicate. and while parsing the allskuset will take longer time while assing Pcd.SkuInfoList. This patch is to eliminate those duplicate entires to reduce the build time Cc: Yuwei Chen <yuwei.chen@intel.com> Cc: Rebecca Cran <rebecca@bsdio.com> Cc: Liming Gao <gaoliming@byosoft.com.cn> Cc: Bob Feng <bob.c.feng@intel.com> Cc: Amy Chan <amy.chan@intel.com> Cc: Sai Chaganty <rangasai.v.chaganty@intel.com> Signed-off-by: Ashraf Ali S <ashraf.ali.s@intel.com> --- BaseTools/Source/Python/AutoGen/PlatformAutoGen.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py b/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py index 592d4824a4..dac81454a9 100644 --- a/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py +++ b/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py @@ -707,6 +707,8 @@ class PlatformAutoGen(AutoGen): self._DynamicPcdList.extend(list(OtherPcdArray)) self._DynamicPcdList.sort() allskuset = [(SkuName, Sku.SkuId) for pcd in self._DynamicPcdList for (SkuName, Sku) in pcd.SkuInfoList.items()] + # Remove duplicate sets in the list + allskuset = list(set(allskuset)) for pcd in self._DynamicPcdList: if len(pcd.SkuInfoList) == 1: for (SkuName, SkuId) in allskuset: -- 2.39.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#112884): https://edk2.groups.io/g/devel/message/112884 Mute This Topic: https://groups.io/mt/103360241/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [edk2-devel] [PATCH] BaseTools: Remove Duplicate sets of SkuName and SkuId from allskuset 2023-12-25 15:27 [edk2-devel] [PATCH] BaseTools: Remove Duplicate sets of SkuName and SkuId from allskuset Ashraf Ali S @ 2024-01-08 0:29 ` Yuwei Chen 2024-01-08 3:54 ` Chan, Amy 2024-01-24 0:28 ` Michael D Kinney 0 siblings, 2 replies; 11+ messages in thread From: Yuwei Chen @ 2024-01-08 0:29 UTC (permalink / raw) To: S, Ashraf Ali, devel@edk2.groups.io Cc: Rebecca Cran, Gao, Liming, Feng, Bob C, Chan, Amy, Chaganty, Rangasai V Reviewed-by: Yuwei Chen <yuwei.chen@intel.com> > -----Original Message----- > From: S, Ashraf Ali <ashraf.ali.s@intel.com> > Sent: Monday, December 25, 2023 11:28 PM > To: devel@edk2.groups.io > Cc: S, Ashraf Ali <ashraf.ali.s@intel.com>; Chen, Christine > <yuwei.chen@intel.com>; Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > <gaoliming@byosoft.com.cn>; Feng, Bob C <bob.c.feng@intel.com>; Chan, > Amy <amy.chan@intel.com>; Chaganty, Rangasai V > <rangasai.v.chaganty@intel.com> > Subject: [PATCH] BaseTools: Remove Duplicate sets of SkuName and SkuId > from allskuset > > Currently when the platform has many SKUs then allskuset will be having so > many duplicate. and while parsing the allskuset will take longer time while > assing Pcd.SkuInfoList. > This patch is to eliminate those duplicate entires to reduce the build time > > Cc: Yuwei Chen <yuwei.chen@intel.com> > Cc: Rebecca Cran <rebecca@bsdio.com> > Cc: Liming Gao <gaoliming@byosoft.com.cn> > Cc: Bob Feng <bob.c.feng@intel.com> > Cc: Amy Chan <amy.chan@intel.com> > Cc: Sai Chaganty <rangasai.v.chaganty@intel.com> > Signed-off-by: Ashraf Ali S <ashraf.ali.s@intel.com> > --- > BaseTools/Source/Python/AutoGen/PlatformAutoGen.py | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > b/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > index 592d4824a4..dac81454a9 100644 > --- a/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > +++ b/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > @@ -707,6 +707,8 @@ class PlatformAutoGen(AutoGen): > self._DynamicPcdList.extend(list(OtherPcdArray)) > self._DynamicPcdList.sort() > allskuset = [(SkuName, Sku.SkuId) for pcd in self._DynamicPcdList for > (SkuName, Sku) in pcd.SkuInfoList.items()] > + # Remove duplicate sets in the list > + allskuset = list(set(allskuset)) > for pcd in self._DynamicPcdList: > if len(pcd.SkuInfoList) == 1: > for (SkuName, SkuId) in allskuset: > -- > 2.39.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#113334): https://edk2.groups.io/g/devel/message/113334 Mute This Topic: https://groups.io/mt/103360241/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [edk2-devel] [PATCH] BaseTools: Remove Duplicate sets of SkuName and SkuId from allskuset 2024-01-08 0:29 ` Yuwei Chen @ 2024-01-08 3:54 ` Chan, Amy 2024-01-30 7:48 ` Feng, Bob C 2024-01-24 0:28 ` Michael D Kinney 1 sibling, 1 reply; 11+ messages in thread From: Chan, Amy @ 2024-01-08 3:54 UTC (permalink / raw) To: Chen, Christine, S, Ashraf Ali, devel@edk2.groups.io Cc: Rebecca Cran, Gao, Liming, Feng, Bob C, Chaganty, Rangasai V Reviewed-by: Amy Chan <amy.chan@intel.com> > -----Original Message----- > From: Chen, Christine <yuwei.chen@intel.com> > Sent: Monday, January 8, 2024 8:29 AM > To: S, Ashraf Ali <ashraf.ali.s@intel.com>; devel@edk2.groups.io > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > <gaoliming@byosoft.com.cn>; Feng, Bob C <bob.c.feng@intel.com>; Chan, > Amy <amy.chan@intel.com>; Chaganty, Rangasai V > <rangasai.v.chaganty@intel.com> > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName and SkuId > from allskuset > > Reviewed-by: Yuwei Chen <yuwei.chen@intel.com> > > > -----Original Message----- > > From: S, Ashraf Ali <ashraf.ali.s@intel.com> > > Sent: Monday, December 25, 2023 11:28 PM > > To: devel@edk2.groups.io > > Cc: S, Ashraf Ali <ashraf.ali.s@intel.com>; Chen, Christine > > <yuwei.chen@intel.com>; Rebecca Cran <rebecca@bsdio.com>; Gao, > Liming > > <gaoliming@byosoft.com.cn>; Feng, Bob C <bob.c.feng@intel.com>; Chan, > > Amy <amy.chan@intel.com>; Chaganty, Rangasai V > > <rangasai.v.chaganty@intel.com> > > Subject: [PATCH] BaseTools: Remove Duplicate sets of SkuName and SkuId > > from allskuset > > > > Currently when the platform has many SKUs then allskuset will be having so > > many duplicate. and while parsing the allskuset will take longer time while > > assing Pcd.SkuInfoList. > > This patch is to eliminate those duplicate entires to reduce the build time > > > > Cc: Yuwei Chen <yuwei.chen@intel.com> > > Cc: Rebecca Cran <rebecca@bsdio.com> > > Cc: Liming Gao <gaoliming@byosoft.com.cn> > > Cc: Bob Feng <bob.c.feng@intel.com> > > Cc: Amy Chan <amy.chan@intel.com> > > Cc: Sai Chaganty <rangasai.v.chaganty@intel.com> > > Signed-off-by: Ashraf Ali S <ashraf.ali.s@intel.com> > > --- > > BaseTools/Source/Python/AutoGen/PlatformAutoGen.py | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > b/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > index 592d4824a4..dac81454a9 100644 > > --- a/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > +++ b/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > @@ -707,6 +707,8 @@ class PlatformAutoGen(AutoGen): > > self._DynamicPcdList.extend(list(OtherPcdArray)) > > self._DynamicPcdList.sort() > > allskuset = [(SkuName, Sku.SkuId) for pcd in self._DynamicPcdList for > > (SkuName, Sku) in pcd.SkuInfoList.items()] > > + # Remove duplicate sets in the list > > + allskuset = list(set(allskuset)) > > for pcd in self._DynamicPcdList: > > if len(pcd.SkuInfoList) == 1: > > for (SkuName, SkuId) in allskuset: > > -- > > 2.39.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#113451): https://edk2.groups.io/g/devel/message/113451 Mute This Topic: https://groups.io/mt/103360241/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [edk2-devel] [PATCH] BaseTools: Remove Duplicate sets of SkuName and SkuId from allskuset 2024-01-08 3:54 ` Chan, Amy @ 2024-01-30 7:48 ` Feng, Bob C 2024-02-08 5:49 ` Ashraf Ali S 0 siblings, 1 reply; 11+ messages in thread From: Feng, Bob C @ 2024-01-30 7:48 UTC (permalink / raw) To: Chan, Amy, Chen, Christine, S, Ashraf Ali, devel@edk2.groups.io Cc: Rebecca Cran, Gao, Liming, Chaganty, Rangasai V Reviewed-by: Bob Feng <bob.c.feng@intel.com> -----Original Message----- From: Chan, Amy <amy.chan@intel.com> Sent: Monday, January 8, 2024 11:54 AM To: Chen, Christine <yuwei.chen@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>; devel@edk2.groups.io Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming <gaoliming@byosoft.com.cn>; Feng, Bob C <bob.c.feng@intel.com>; Chaganty, Rangasai V <rangasai.v.chaganty@intel.com> Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName and SkuId from allskuset Reviewed-by: Amy Chan <amy.chan@intel.com> > -----Original Message----- > From: Chen, Christine <yuwei.chen@intel.com> > Sent: Monday, January 8, 2024 8:29 AM > To: S, Ashraf Ali <ashraf.ali.s@intel.com>; devel@edk2.groups.io > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > <gaoliming@byosoft.com.cn>; Feng, Bob C <bob.c.feng@intel.com>; Chan, > Amy <amy.chan@intel.com>; Chaganty, Rangasai V > <rangasai.v.chaganty@intel.com> > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName and > SkuId from allskuset > > Reviewed-by: Yuwei Chen <yuwei.chen@intel.com> > > > -----Original Message----- > > From: S, Ashraf Ali <ashraf.ali.s@intel.com> > > Sent: Monday, December 25, 2023 11:28 PM > > To: devel@edk2.groups.io > > Cc: S, Ashraf Ali <ashraf.ali.s@intel.com>; Chen, Christine > > <yuwei.chen@intel.com>; Rebecca Cran <rebecca@bsdio.com>; Gao, > Liming > > <gaoliming@byosoft.com.cn>; Feng, Bob C <bob.c.feng@intel.com>; > > Chan, Amy <amy.chan@intel.com>; Chaganty, Rangasai V > > <rangasai.v.chaganty@intel.com> > > Subject: [PATCH] BaseTools: Remove Duplicate sets of SkuName and > > SkuId from allskuset > > > > Currently when the platform has many SKUs then allskuset will be > > having so many duplicate. and while parsing the allskuset will take > > longer time while assing Pcd.SkuInfoList. > > This patch is to eliminate those duplicate entires to reduce the > > build time > > > > Cc: Yuwei Chen <yuwei.chen@intel.com> > > Cc: Rebecca Cran <rebecca@bsdio.com> > > Cc: Liming Gao <gaoliming@byosoft.com.cn> > > Cc: Bob Feng <bob.c.feng@intel.com> > > Cc: Amy Chan <amy.chan@intel.com> > > Cc: Sai Chaganty <rangasai.v.chaganty@intel.com> > > Signed-off-by: Ashraf Ali S <ashraf.ali.s@intel.com> > > --- > > BaseTools/Source/Python/AutoGen/PlatformAutoGen.py | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > b/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > index 592d4824a4..dac81454a9 100644 > > --- a/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > +++ b/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > @@ -707,6 +707,8 @@ class PlatformAutoGen(AutoGen): > > self._DynamicPcdList.extend(list(OtherPcdArray)) > > self._DynamicPcdList.sort() > > allskuset = [(SkuName, Sku.SkuId) for pcd in > > self._DynamicPcdList for (SkuName, Sku) in pcd.SkuInfoList.items()] > > + # Remove duplicate sets in the list > > + allskuset = list(set(allskuset)) > > for pcd in self._DynamicPcdList: > > if len(pcd.SkuInfoList) == 1: > > for (SkuName, SkuId) in allskuset: > > -- > > 2.39.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#114799): https://edk2.groups.io/g/devel/message/114799 Mute This Topic: https://groups.io/mt/103360241/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [edk2-devel] [PATCH] BaseTools: Remove Duplicate sets of SkuName and SkuId from allskuset 2024-01-30 7:48 ` Feng, Bob C @ 2024-02-08 5:49 ` Ashraf Ali S 2024-02-08 5:53 ` Michael D Kinney 0 siblings, 1 reply; 11+ messages in thread From: Ashraf Ali S @ 2024-02-08 5:49 UTC (permalink / raw) To: Feng, Bob C, Chan, Amy, Chen, Christine, devel@edk2.groups.io Cc: Rebecca Cran, Gao, Liming, Chaganty, Rangasai V, Kinney, Michael D Hi Is this patch merged! PR under base-tools repo : https://github.com/tianocore/edk2-basetools/pull/112 Builds are failing, looks like it's not due to these changes. Thanks., S, Ashraf Ali -----Original Message----- From: Feng, Bob C <bob.c.feng@intel.com> Sent: Tuesday, January 30, 2024 1:19 PM To: Chan, Amy <amy.chan@intel.com>; Chen, Christine <yuwei.chen@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>; devel@edk2.groups.io Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming <gaoliming@byosoft.com.cn>; Chaganty, Rangasai V <rangasai.v.chaganty@intel.com> Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName and SkuId from allskuset Reviewed-by: Bob Feng <bob.c.feng@intel.com> -----Original Message----- From: Chan, Amy <amy.chan@intel.com> Sent: Monday, January 8, 2024 11:54 AM To: Chen, Christine <yuwei.chen@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>; devel@edk2.groups.io Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming <gaoliming@byosoft.com.cn>; Feng, Bob C <bob.c.feng@intel.com>; Chaganty, Rangasai V <rangasai.v.chaganty@intel.com> Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName and SkuId from allskuset Reviewed-by: Amy Chan <amy.chan@intel.com> > -----Original Message----- > From: Chen, Christine <yuwei.chen@intel.com> > Sent: Monday, January 8, 2024 8:29 AM > To: S, Ashraf Ali <ashraf.ali.s@intel.com>; devel@edk2.groups.io > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > <gaoliming@byosoft.com.cn>; Feng, Bob C <bob.c.feng@intel.com>; Chan, > Amy <amy.chan@intel.com>; Chaganty, Rangasai V > <rangasai.v.chaganty@intel.com> > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName and > SkuId from allskuset > > Reviewed-by: Yuwei Chen <yuwei.chen@intel.com> > > > -----Original Message----- > > From: S, Ashraf Ali <ashraf.ali.s@intel.com> > > Sent: Monday, December 25, 2023 11:28 PM > > To: devel@edk2.groups.io > > Cc: S, Ashraf Ali <ashraf.ali.s@intel.com>; Chen, Christine > > <yuwei.chen@intel.com>; Rebecca Cran <rebecca@bsdio.com>; Gao, > Liming > > <gaoliming@byosoft.com.cn>; Feng, Bob C <bob.c.feng@intel.com>; > > Chan, Amy <amy.chan@intel.com>; Chaganty, Rangasai V > > <rangasai.v.chaganty@intel.com> > > Subject: [PATCH] BaseTools: Remove Duplicate sets of SkuName and > > SkuId from allskuset > > > > Currently when the platform has many SKUs then allskuset will be > > having so many duplicate. and while parsing the allskuset will take > > longer time while assing Pcd.SkuInfoList. > > This patch is to eliminate those duplicate entires to reduce the > > build time > > > > Cc: Yuwei Chen <yuwei.chen@intel.com> > > Cc: Rebecca Cran <rebecca@bsdio.com> > > Cc: Liming Gao <gaoliming@byosoft.com.cn> > > Cc: Bob Feng <bob.c.feng@intel.com> > > Cc: Amy Chan <amy.chan@intel.com> > > Cc: Sai Chaganty <rangasai.v.chaganty@intel.com> > > Signed-off-by: Ashraf Ali S <ashraf.ali.s@intel.com> > > --- > > BaseTools/Source/Python/AutoGen/PlatformAutoGen.py | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > b/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > index 592d4824a4..dac81454a9 100644 > > --- a/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > +++ b/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > @@ -707,6 +707,8 @@ class PlatformAutoGen(AutoGen): > > self._DynamicPcdList.extend(list(OtherPcdArray)) > > self._DynamicPcdList.sort() > > allskuset = [(SkuName, Sku.SkuId) for pcd in > > self._DynamicPcdList for (SkuName, Sku) in pcd.SkuInfoList.items()] > > + # Remove duplicate sets in the list > > + allskuset = list(set(allskuset)) > > for pcd in self._DynamicPcdList: > > if len(pcd.SkuInfoList) == 1: > > for (SkuName, SkuId) in allskuset: > > -- > > 2.39.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#115264): https://edk2.groups.io/g/devel/message/115264 Mute This Topic: https://groups.io/mt/103360241/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [edk2-devel] [PATCH] BaseTools: Remove Duplicate sets of SkuName and SkuId from allskuset 2024-02-08 5:49 ` Ashraf Ali S @ 2024-02-08 5:53 ` Michael D Kinney 2024-02-08 6:38 ` Yuwei Chen 0 siblings, 1 reply; 11+ messages in thread From: Michael D Kinney @ 2024-02-08 5:53 UTC (permalink / raw) To: S, Ashraf Ali, Feng, Bob C, Chan, Amy, Chen, Christine, devel@edk2.groups.io Cc: Rebecca Cran, Gao, Liming, Chaganty, Rangasai V, Kinney, Michael D I will merge it soon. Please work with Christine to get edk2-basetools CI working correctly. Mike > -----Original Message----- > From: S, Ashraf Ali <ashraf.ali.s@intel.com> > Sent: Wednesday, February 7, 2024 9:50 PM > To: Feng, Bob C <bob.c.feng@intel.com>; Chan, Amy <amy.chan@intel.com>; > Chen, Christine <yuwei.chen@intel.com>; devel@edk2.groups.io > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > <gaoliming@byosoft.com.cn>; Chaganty, Rangasai V > <rangasai.v.chaganty@intel.com>; Kinney, Michael D > <michael.d.kinney@intel.com> > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName and > SkuId from allskuset > > Hi > > Is this patch merged! > PR under base-tools repo : https://github.com/tianocore/edk2- > basetools/pull/112 > > Builds are failing, looks like it's not due to these changes. > > Thanks., > S, Ashraf Ali > > -----Original Message----- > From: Feng, Bob C <bob.c.feng@intel.com> > Sent: Tuesday, January 30, 2024 1:19 PM > To: Chan, Amy <amy.chan@intel.com>; Chen, Christine > <yuwei.chen@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>; > devel@edk2.groups.io > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > <gaoliming@byosoft.com.cn>; Chaganty, Rangasai V > <rangasai.v.chaganty@intel.com> > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName and > SkuId from allskuset > > Reviewed-by: Bob Feng <bob.c.feng@intel.com> > > -----Original Message----- > From: Chan, Amy <amy.chan@intel.com> > Sent: Monday, January 8, 2024 11:54 AM > To: Chen, Christine <yuwei.chen@intel.com>; S, Ashraf Ali > <ashraf.ali.s@intel.com>; devel@edk2.groups.io > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > <gaoliming@byosoft.com.cn>; Feng, Bob C <bob.c.feng@intel.com>; > Chaganty, Rangasai V <rangasai.v.chaganty@intel.com> > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName and > SkuId from allskuset > > Reviewed-by: Amy Chan <amy.chan@intel.com> > > > -----Original Message----- > > From: Chen, Christine <yuwei.chen@intel.com> > > Sent: Monday, January 8, 2024 8:29 AM > > To: S, Ashraf Ali <ashraf.ali.s@intel.com>; devel@edk2.groups.io > > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > > <gaoliming@byosoft.com.cn>; Feng, Bob C <bob.c.feng@intel.com>; Chan, > > Amy <amy.chan@intel.com>; Chaganty, Rangasai V > > <rangasai.v.chaganty@intel.com> > > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName and > > SkuId from allskuset > > > > Reviewed-by: Yuwei Chen <yuwei.chen@intel.com> > > > > > -----Original Message----- > > > From: S, Ashraf Ali <ashraf.ali.s@intel.com> > > > Sent: Monday, December 25, 2023 11:28 PM > > > To: devel@edk2.groups.io > > > Cc: S, Ashraf Ali <ashraf.ali.s@intel.com>; Chen, Christine > > > <yuwei.chen@intel.com>; Rebecca Cran <rebecca@bsdio.com>; Gao, > > Liming > > > <gaoliming@byosoft.com.cn>; Feng, Bob C <bob.c.feng@intel.com>; > > > Chan, Amy <amy.chan@intel.com>; Chaganty, Rangasai V > > > <rangasai.v.chaganty@intel.com> > > > Subject: [PATCH] BaseTools: Remove Duplicate sets of SkuName and > > > SkuId from allskuset > > > > > > Currently when the platform has many SKUs then allskuset will be > > > having so many duplicate. and while parsing the allskuset will take > > > longer time while assing Pcd.SkuInfoList. > > > This patch is to eliminate those duplicate entires to reduce the > > > build time > > > > > > Cc: Yuwei Chen <yuwei.chen@intel.com> > > > Cc: Rebecca Cran <rebecca@bsdio.com> > > > Cc: Liming Gao <gaoliming@byosoft.com.cn> > > > Cc: Bob Feng <bob.c.feng@intel.com> > > > Cc: Amy Chan <amy.chan@intel.com> > > > Cc: Sai Chaganty <rangasai.v.chaganty@intel.com> > > > Signed-off-by: Ashraf Ali S <ashraf.ali.s@intel.com> > > > --- > > > BaseTools/Source/Python/AutoGen/PlatformAutoGen.py | 2 ++ > > > 1 file changed, 2 insertions(+) > > > > > > diff --git a/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > > b/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > > index 592d4824a4..dac81454a9 100644 > > > --- a/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > > +++ b/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > > @@ -707,6 +707,8 @@ class PlatformAutoGen(AutoGen): > > > self._DynamicPcdList.extend(list(OtherPcdArray)) > > > self._DynamicPcdList.sort() > > > allskuset = [(SkuName, Sku.SkuId) for pcd in > > > self._DynamicPcdList for (SkuName, Sku) in pcd.SkuInfoList.items()] > > > + # Remove duplicate sets in the list > > > + allskuset = list(set(allskuset)) > > > for pcd in self._DynamicPcdList: > > > if len(pcd.SkuInfoList) == 1: > > > for (SkuName, SkuId) in allskuset: > > > -- > > > 2.39.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#115265): https://edk2.groups.io/g/devel/message/115265 Mute This Topic: https://groups.io/mt/103360241/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [edk2-devel] [PATCH] BaseTools: Remove Duplicate sets of SkuName and SkuId from allskuset 2024-02-08 5:53 ` Michael D Kinney @ 2024-02-08 6:38 ` Yuwei Chen 2024-02-08 16:30 ` Michael D Kinney 0 siblings, 1 reply; 11+ messages in thread From: Yuwei Chen @ 2024-02-08 6:38 UTC (permalink / raw) To: Kinney, Michael D, S, Ashraf Ali, Feng, Bob C, Chan, Amy, devel@edk2.groups.io Cc: Rebecca Cran, Gao, Liming, Chaganty, Rangasai V Hi Mike, We checked the PR Gate -Ubuntu / Windows log, and seems the error is not raised by patch itself. In order to prove it, we submit a new PR with exactly no changes(only and empty lines), and the tests are still failure: https://github.com/tianocore/edk2-basetools/pull/114 Could you help confirm that? If it is not the code issue, can we merge the code directly or fix the pipeline issue first? Thanks, Christine > -----Original Message----- > From: Kinney, Michael D <michael.d.kinney@intel.com> > Sent: Thursday, February 8, 2024 1:54 PM > To: S, Ashraf Ali <ashraf.ali.s@intel.com>; Feng, Bob C > <bob.c.feng@intel.com>; Chan, Amy <amy.chan@intel.com>; Chen, Christine > <yuwei.chen@intel.com>; devel@edk2.groups.io > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > <gaoliming@byosoft.com.cn>; Chaganty, Rangasai V > <rangasai.v.chaganty@intel.com>; Kinney, Michael D > <michael.d.kinney@intel.com> > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName and SkuId > from allskuset > > I will merge it soon. > > Please work with Christine to get edk2-basetools CI working correctly. > > Mike > > > -----Original Message----- > > From: S, Ashraf Ali <ashraf.ali.s@intel.com> > > Sent: Wednesday, February 7, 2024 9:50 PM > > To: Feng, Bob C <bob.c.feng@intel.com>; Chan, Amy > > <amy.chan@intel.com>; Chen, Christine <yuwei.chen@intel.com>; > > devel@edk2.groups.io > > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > > <gaoliming@byosoft.com.cn>; Chaganty, Rangasai V > > <rangasai.v.chaganty@intel.com>; Kinney, Michael D > > <michael.d.kinney@intel.com> > > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName and > > SkuId from allskuset > > > > Hi > > > > Is this patch merged! > > PR under base-tools repo : https://github.com/tianocore/edk2- > > basetools/pull/112 > > > > Builds are failing, looks like it's not due to these changes. > > > > Thanks., > > S, Ashraf Ali > > > > -----Original Message----- > > From: Feng, Bob C <bob.c.feng@intel.com> > > Sent: Tuesday, January 30, 2024 1:19 PM > > To: Chan, Amy <amy.chan@intel.com>; Chen, Christine > > <yuwei.chen@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>; > > devel@edk2.groups.io > > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > > <gaoliming@byosoft.com.cn>; Chaganty, Rangasai V > > <rangasai.v.chaganty@intel.com> > > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName and > > SkuId from allskuset > > > > Reviewed-by: Bob Feng <bob.c.feng@intel.com> > > > > -----Original Message----- > > From: Chan, Amy <amy.chan@intel.com> > > Sent: Monday, January 8, 2024 11:54 AM > > To: Chen, Christine <yuwei.chen@intel.com>; S, Ashraf Ali > > <ashraf.ali.s@intel.com>; devel@edk2.groups.io > > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > > <gaoliming@byosoft.com.cn>; Feng, Bob C <bob.c.feng@intel.com>; > > Chaganty, Rangasai V <rangasai.v.chaganty@intel.com> > > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName and > > SkuId from allskuset > > > > Reviewed-by: Amy Chan <amy.chan@intel.com> > > > > > -----Original Message----- > > > From: Chen, Christine <yuwei.chen@intel.com> > > > Sent: Monday, January 8, 2024 8:29 AM > > > To: S, Ashraf Ali <ashraf.ali.s@intel.com>; devel@edk2.groups.io > > > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > > > <gaoliming@byosoft.com.cn>; Feng, Bob C <bob.c.feng@intel.com>; > > > Chan, Amy <amy.chan@intel.com>; Chaganty, Rangasai V > > > <rangasai.v.chaganty@intel.com> > > > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName and > > > SkuId from allskuset > > > > > > Reviewed-by: Yuwei Chen <yuwei.chen@intel.com> > > > > > > > -----Original Message----- > > > > From: S, Ashraf Ali <ashraf.ali.s@intel.com> > > > > Sent: Monday, December 25, 2023 11:28 PM > > > > To: devel@edk2.groups.io > > > > Cc: S, Ashraf Ali <ashraf.ali.s@intel.com>; Chen, Christine > > > > <yuwei.chen@intel.com>; Rebecca Cran <rebecca@bsdio.com>; Gao, > > > Liming > > > > <gaoliming@byosoft.com.cn>; Feng, Bob C <bob.c.feng@intel.com>; > > > > Chan, Amy <amy.chan@intel.com>; Chaganty, Rangasai V > > > > <rangasai.v.chaganty@intel.com> > > > > Subject: [PATCH] BaseTools: Remove Duplicate sets of SkuName and > > > > SkuId from allskuset > > > > > > > > Currently when the platform has many SKUs then allskuset will be > > > > having so many duplicate. and while parsing the allskuset will > > > > take longer time while assing Pcd.SkuInfoList. > > > > This patch is to eliminate those duplicate entires to reduce the > > > > build time > > > > > > > > Cc: Yuwei Chen <yuwei.chen@intel.com> > > > > Cc: Rebecca Cran <rebecca@bsdio.com> > > > > Cc: Liming Gao <gaoliming@byosoft.com.cn> > > > > Cc: Bob Feng <bob.c.feng@intel.com> > > > > Cc: Amy Chan <amy.chan@intel.com> > > > > Cc: Sai Chaganty <rangasai.v.chaganty@intel.com> > > > > Signed-off-by: Ashraf Ali S <ashraf.ali.s@intel.com> > > > > --- > > > > BaseTools/Source/Python/AutoGen/PlatformAutoGen.py | 2 ++ > > > > 1 file changed, 2 insertions(+) > > > > > > > > diff --git a/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > > > b/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > > > index 592d4824a4..dac81454a9 100644 > > > > --- a/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > > > +++ b/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > > > @@ -707,6 +707,8 @@ class PlatformAutoGen(AutoGen): > > > > self._DynamicPcdList.extend(list(OtherPcdArray)) > > > > self._DynamicPcdList.sort() > > > > allskuset = [(SkuName, Sku.SkuId) for pcd in > > > > self._DynamicPcdList for (SkuName, Sku) in > > > > pcd.SkuInfoList.items()] > > > > + # Remove duplicate sets in the list > > > > + allskuset = list(set(allskuset)) > > > > for pcd in self._DynamicPcdList: > > > > if len(pcd.SkuInfoList) == 1: > > > > for (SkuName, SkuId) in allskuset: > > > > -- > > > > 2.39.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#115266): https://edk2.groups.io/g/devel/message/115266 Mute This Topic: https://groups.io/mt/103360241/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [edk2-devel] [PATCH] BaseTools: Remove Duplicate sets of SkuName and SkuId from allskuset 2024-02-08 6:38 ` Yuwei Chen @ 2024-02-08 16:30 ` Michael D Kinney 2024-02-08 19:27 ` Michael D Kinney 0 siblings, 1 reply; 11+ messages in thread From: Michael D Kinney @ 2024-02-08 16:30 UTC (permalink / raw) To: Chen, Christine, S, Ashraf Ali, Feng, Bob C, Chan, Amy, devel@edk2.groups.io Cc: Rebecca Cran, Gao, Liming, Chaganty, Rangasai V, Kinney, Michael D Hi Christine, I will merge this change into edk2/BaseTools. We need to work in why the edk2-basetools CI is not working and fix that And get the set of BaseTools changes completed in edk2-basetools before the edk2-stable202402 release is completed, so edk2/BaseTools and edk2-basetools is aligned for that release and can stay aligned until edk2/BaseTools can be retired. Mike > -----Original Message----- > From: Chen, Christine <yuwei.chen@intel.com> > Sent: Wednesday, February 7, 2024 10:39 PM > To: Kinney, Michael D <michael.d.kinney@intel.com>; S, Ashraf Ali > <ashraf.ali.s@intel.com>; Feng, Bob C <bob.c.feng@intel.com>; Chan, Amy > <amy.chan@intel.com>; devel@edk2.groups.io > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > <gaoliming@byosoft.com.cn>; Chaganty, Rangasai V > <rangasai.v.chaganty@intel.com> > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName and > SkuId from allskuset > > Hi Mike, > > We checked the PR Gate -Ubuntu / Windows log, and seems the error is > not raised by patch itself. In order to prove it, we submit a new PR > with exactly no changes(only and empty lines), and the tests are still > failure: https://github.com/tianocore/edk2-basetools/pull/114 > > Could you help confirm that? > If it is not the code issue, can we merge the code directly or fix the > pipeline issue first? > > Thanks, > Christine > > > -----Original Message----- > > From: Kinney, Michael D <michael.d.kinney@intel.com> > > Sent: Thursday, February 8, 2024 1:54 PM > > To: S, Ashraf Ali <ashraf.ali.s@intel.com>; Feng, Bob C > > <bob.c.feng@intel.com>; Chan, Amy <amy.chan@intel.com>; Chen, > Christine > > <yuwei.chen@intel.com>; devel@edk2.groups.io > > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > > <gaoliming@byosoft.com.cn>; Chaganty, Rangasai V > > <rangasai.v.chaganty@intel.com>; Kinney, Michael D > > <michael.d.kinney@intel.com> > > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName and > SkuId > > from allskuset > > > > I will merge it soon. > > > > Please work with Christine to get edk2-basetools CI working > correctly. > > > > Mike > > > > > -----Original Message----- > > > From: S, Ashraf Ali <ashraf.ali.s@intel.com> > > > Sent: Wednesday, February 7, 2024 9:50 PM > > > To: Feng, Bob C <bob.c.feng@intel.com>; Chan, Amy > > > <amy.chan@intel.com>; Chen, Christine <yuwei.chen@intel.com>; > > > devel@edk2.groups.io > > > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > > > <gaoliming@byosoft.com.cn>; Chaganty, Rangasai V > > > <rangasai.v.chaganty@intel.com>; Kinney, Michael D > > > <michael.d.kinney@intel.com> > > > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName > and > > > SkuId from allskuset > > > > > > Hi > > > > > > Is this patch merged! > > > PR under base-tools repo : https://github.com/tianocore/edk2- > > > basetools/pull/112 > > > > > > Builds are failing, looks like it's not due to these changes. > > > > > > Thanks., > > > S, Ashraf Ali > > > > > > -----Original Message----- > > > From: Feng, Bob C <bob.c.feng@intel.com> > > > Sent: Tuesday, January 30, 2024 1:19 PM > > > To: Chan, Amy <amy.chan@intel.com>; Chen, Christine > > > <yuwei.chen@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>; > > > devel@edk2.groups.io > > > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > > > <gaoliming@byosoft.com.cn>; Chaganty, Rangasai V > > > <rangasai.v.chaganty@intel.com> > > > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName > and > > > SkuId from allskuset > > > > > > Reviewed-by: Bob Feng <bob.c.feng@intel.com> > > > > > > -----Original Message----- > > > From: Chan, Amy <amy.chan@intel.com> > > > Sent: Monday, January 8, 2024 11:54 AM > > > To: Chen, Christine <yuwei.chen@intel.com>; S, Ashraf Ali > > > <ashraf.ali.s@intel.com>; devel@edk2.groups.io > > > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > > > <gaoliming@byosoft.com.cn>; Feng, Bob C <bob.c.feng@intel.com>; > > > Chaganty, Rangasai V <rangasai.v.chaganty@intel.com> > > > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName > and > > > SkuId from allskuset > > > > > > Reviewed-by: Amy Chan <amy.chan@intel.com> > > > > > > > -----Original Message----- > > > > From: Chen, Christine <yuwei.chen@intel.com> > > > > Sent: Monday, January 8, 2024 8:29 AM > > > > To: S, Ashraf Ali <ashraf.ali.s@intel.com>; devel@edk2.groups.io > > > > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > > > > <gaoliming@byosoft.com.cn>; Feng, Bob C <bob.c.feng@intel.com>; > > > > Chan, Amy <amy.chan@intel.com>; Chaganty, Rangasai V > > > > <rangasai.v.chaganty@intel.com> > > > > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName > and > > > > SkuId from allskuset > > > > > > > > Reviewed-by: Yuwei Chen <yuwei.chen@intel.com> > > > > > > > > > -----Original Message----- > > > > > From: S, Ashraf Ali <ashraf.ali.s@intel.com> > > > > > Sent: Monday, December 25, 2023 11:28 PM > > > > > To: devel@edk2.groups.io > > > > > Cc: S, Ashraf Ali <ashraf.ali.s@intel.com>; Chen, Christine > > > > > <yuwei.chen@intel.com>; Rebecca Cran <rebecca@bsdio.com>; Gao, > > > > Liming > > > > > <gaoliming@byosoft.com.cn>; Feng, Bob C <bob.c.feng@intel.com>; > > > > > Chan, Amy <amy.chan@intel.com>; Chaganty, Rangasai V > > > > > <rangasai.v.chaganty@intel.com> > > > > > Subject: [PATCH] BaseTools: Remove Duplicate sets of SkuName > and > > > > > SkuId from allskuset > > > > > > > > > > Currently when the platform has many SKUs then allskuset will > be > > > > > having so many duplicate. and while parsing the allskuset will > > > > > take longer time while assing Pcd.SkuInfoList. > > > > > This patch is to eliminate those duplicate entires to reduce > the > > > > > build time > > > > > > > > > > Cc: Yuwei Chen <yuwei.chen@intel.com> > > > > > Cc: Rebecca Cran <rebecca@bsdio.com> > > > > > Cc: Liming Gao <gaoliming@byosoft.com.cn> > > > > > Cc: Bob Feng <bob.c.feng@intel.com> > > > > > Cc: Amy Chan <amy.chan@intel.com> > > > > > Cc: Sai Chaganty <rangasai.v.chaganty@intel.com> > > > > > Signed-off-by: Ashraf Ali S <ashraf.ali.s@intel.com> > > > > > --- > > > > > BaseTools/Source/Python/AutoGen/PlatformAutoGen.py | 2 ++ > > > > > 1 file changed, 2 insertions(+) > > > > > > > > > > diff --git a/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > > > > b/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > > > > index 592d4824a4..dac81454a9 100644 > > > > > --- a/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > > > > +++ b/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > > > > @@ -707,6 +707,8 @@ class PlatformAutoGen(AutoGen): > > > > > self._DynamicPcdList.extend(list(OtherPcdArray)) > > > > > self._DynamicPcdList.sort() > > > > > allskuset = [(SkuName, Sku.SkuId) for pcd in > > > > > self._DynamicPcdList for (SkuName, Sku) in > > > > > pcd.SkuInfoList.items()] > > > > > + # Remove duplicate sets in the list > > > > > + allskuset = list(set(allskuset)) > > > > > for pcd in self._DynamicPcdList: > > > > > if len(pcd.SkuInfoList) == 1: > > > > > for (SkuName, SkuId) in allskuset: > > > > > -- > > > > > 2.39.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#115268): https://edk2.groups.io/g/devel/message/115268 Mute This Topic: https://groups.io/mt/103360241/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [edk2-devel] [PATCH] BaseTools: Remove Duplicate sets of SkuName and SkuId from allskuset 2024-02-08 16:30 ` Michael D Kinney @ 2024-02-08 19:27 ` Michael D Kinney 0 siblings, 0 replies; 11+ messages in thread From: Michael D Kinney @ 2024-02-08 19:27 UTC (permalink / raw) To: Chen, Christine, S, Ashraf Ali, Feng, Bob C, Chan, Amy, devel@edk2.groups.io Cc: Rebecca Cran, Gao, Liming, Chaganty, Rangasai V, Kinney, Michael D Merged: https://github.com/tianocore/edk2/pull/5359 > -----Original Message----- > From: Kinney, Michael D <michael.d.kinney@intel.com> > Sent: Thursday, February 8, 2024 8:31 AM > To: Chen, Christine <yuwei.chen@intel.com>; S, Ashraf Ali > <ashraf.ali.s@intel.com>; Feng, Bob C <bob.c.feng@intel.com>; Chan, Amy > <amy.chan@intel.com>; devel@edk2.groups.io > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > <gaoliming@byosoft.com.cn>; Chaganty, Rangasai V > <rangasai.v.chaganty@intel.com>; Kinney, Michael D > <michael.d.kinney@intel.com> > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName and > SkuId from allskuset > > Hi Christine, > > I will merge this change into edk2/BaseTools. > > We need to work in why the edk2-basetools CI is not working and fix > that > And get the set of BaseTools changes completed in edk2-basetools before > the edk2-stable202402 release is completed, so edk2/BaseTools and > edk2-basetools is aligned for that release and can stay aligned until > edk2/BaseTools can be retired. > > Mike > > > > > -----Original Message----- > > From: Chen, Christine <yuwei.chen@intel.com> > > Sent: Wednesday, February 7, 2024 10:39 PM > > To: Kinney, Michael D <michael.d.kinney@intel.com>; S, Ashraf Ali > > <ashraf.ali.s@intel.com>; Feng, Bob C <bob.c.feng@intel.com>; Chan, > Amy > > <amy.chan@intel.com>; devel@edk2.groups.io > > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > > <gaoliming@byosoft.com.cn>; Chaganty, Rangasai V > > <rangasai.v.chaganty@intel.com> > > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName and > > SkuId from allskuset > > > > Hi Mike, > > > > We checked the PR Gate -Ubuntu / Windows log, and seems the error is > > not raised by patch itself. In order to prove it, we submit a new PR > > with exactly no changes(only and empty lines), and the tests are > still > > failure: https://github.com/tianocore/edk2-basetools/pull/114 > > > > Could you help confirm that? > > If it is not the code issue, can we merge the code directly or fix > the > > pipeline issue first? > > > > Thanks, > > Christine > > > > > -----Original Message----- > > > From: Kinney, Michael D <michael.d.kinney@intel.com> > > > Sent: Thursday, February 8, 2024 1:54 PM > > > To: S, Ashraf Ali <ashraf.ali.s@intel.com>; Feng, Bob C > > > <bob.c.feng@intel.com>; Chan, Amy <amy.chan@intel.com>; Chen, > > Christine > > > <yuwei.chen@intel.com>; devel@edk2.groups.io > > > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > > > <gaoliming@byosoft.com.cn>; Chaganty, Rangasai V > > > <rangasai.v.chaganty@intel.com>; Kinney, Michael D > > > <michael.d.kinney@intel.com> > > > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName > and > > SkuId > > > from allskuset > > > > > > I will merge it soon. > > > > > > Please work with Christine to get edk2-basetools CI working > > correctly. > > > > > > Mike > > > > > > > -----Original Message----- > > > > From: S, Ashraf Ali <ashraf.ali.s@intel.com> > > > > Sent: Wednesday, February 7, 2024 9:50 PM > > > > To: Feng, Bob C <bob.c.feng@intel.com>; Chan, Amy > > > > <amy.chan@intel.com>; Chen, Christine <yuwei.chen@intel.com>; > > > > devel@edk2.groups.io > > > > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > > > > <gaoliming@byosoft.com.cn>; Chaganty, Rangasai V > > > > <rangasai.v.chaganty@intel.com>; Kinney, Michael D > > > > <michael.d.kinney@intel.com> > > > > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName > > and > > > > SkuId from allskuset > > > > > > > > Hi > > > > > > > > Is this patch merged! > > > > PR under base-tools repo : https://github.com/tianocore/edk2- > > > > basetools/pull/112 > > > > > > > > Builds are failing, looks like it's not due to these changes. > > > > > > > > Thanks., > > > > S, Ashraf Ali > > > > > > > > -----Original Message----- > > > > From: Feng, Bob C <bob.c.feng@intel.com> > > > > Sent: Tuesday, January 30, 2024 1:19 PM > > > > To: Chan, Amy <amy.chan@intel.com>; Chen, Christine > > > > <yuwei.chen@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>; > > > > devel@edk2.groups.io > > > > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > > > > <gaoliming@byosoft.com.cn>; Chaganty, Rangasai V > > > > <rangasai.v.chaganty@intel.com> > > > > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName > > and > > > > SkuId from allskuset > > > > > > > > Reviewed-by: Bob Feng <bob.c.feng@intel.com> > > > > > > > > -----Original Message----- > > > > From: Chan, Amy <amy.chan@intel.com> > > > > Sent: Monday, January 8, 2024 11:54 AM > > > > To: Chen, Christine <yuwei.chen@intel.com>; S, Ashraf Ali > > > > <ashraf.ali.s@intel.com>; devel@edk2.groups.io > > > > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > > > > <gaoliming@byosoft.com.cn>; Feng, Bob C <bob.c.feng@intel.com>; > > > > Chaganty, Rangasai V <rangasai.v.chaganty@intel.com> > > > > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of SkuName > > and > > > > SkuId from allskuset > > > > > > > > Reviewed-by: Amy Chan <amy.chan@intel.com> > > > > > > > > > -----Original Message----- > > > > > From: Chen, Christine <yuwei.chen@intel.com> > > > > > Sent: Monday, January 8, 2024 8:29 AM > > > > > To: S, Ashraf Ali <ashraf.ali.s@intel.com>; > devel@edk2.groups.io > > > > > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > > > > > <gaoliming@byosoft.com.cn>; Feng, Bob C <bob.c.feng@intel.com>; > > > > > Chan, Amy <amy.chan@intel.com>; Chaganty, Rangasai V > > > > > <rangasai.v.chaganty@intel.com> > > > > > Subject: RE: [PATCH] BaseTools: Remove Duplicate sets of > SkuName > > and > > > > > SkuId from allskuset > > > > > > > > > > Reviewed-by: Yuwei Chen <yuwei.chen@intel.com> > > > > > > > > > > > -----Original Message----- > > > > > > From: S, Ashraf Ali <ashraf.ali.s@intel.com> > > > > > > Sent: Monday, December 25, 2023 11:28 PM > > > > > > To: devel@edk2.groups.io > > > > > > Cc: S, Ashraf Ali <ashraf.ali.s@intel.com>; Chen, Christine > > > > > > <yuwei.chen@intel.com>; Rebecca Cran <rebecca@bsdio.com>; > Gao, > > > > > Liming > > > > > > <gaoliming@byosoft.com.cn>; Feng, Bob C > <bob.c.feng@intel.com>; > > > > > > Chan, Amy <amy.chan@intel.com>; Chaganty, Rangasai V > > > > > > <rangasai.v.chaganty@intel.com> > > > > > > Subject: [PATCH] BaseTools: Remove Duplicate sets of SkuName > > and > > > > > > SkuId from allskuset > > > > > > > > > > > > Currently when the platform has many SKUs then allskuset will > > be > > > > > > having so many duplicate. and while parsing the allskuset > will > > > > > > take longer time while assing Pcd.SkuInfoList. > > > > > > This patch is to eliminate those duplicate entires to reduce > > the > > > > > > build time > > > > > > > > > > > > Cc: Yuwei Chen <yuwei.chen@intel.com> > > > > > > Cc: Rebecca Cran <rebecca@bsdio.com> > > > > > > Cc: Liming Gao <gaoliming@byosoft.com.cn> > > > > > > Cc: Bob Feng <bob.c.feng@intel.com> > > > > > > Cc: Amy Chan <amy.chan@intel.com> > > > > > > Cc: Sai Chaganty <rangasai.v.chaganty@intel.com> > > > > > > Signed-off-by: Ashraf Ali S <ashraf.ali.s@intel.com> > > > > > > --- > > > > > > BaseTools/Source/Python/AutoGen/PlatformAutoGen.py | 2 ++ > > > > > > 1 file changed, 2 insertions(+) > > > > > > > > > > > > diff --git > a/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > > > > > b/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > > > > > index 592d4824a4..dac81454a9 100644 > > > > > > --- a/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > > > > > +++ b/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > > > > > @@ -707,6 +707,8 @@ class PlatformAutoGen(AutoGen): > > > > > > self._DynamicPcdList.extend(list(OtherPcdArray)) > > > > > > self._DynamicPcdList.sort() > > > > > > allskuset = [(SkuName, Sku.SkuId) for pcd in > > > > > > self._DynamicPcdList for (SkuName, Sku) in > > > > > > pcd.SkuInfoList.items()] > > > > > > + # Remove duplicate sets in the list > > > > > > + allskuset = list(set(allskuset)) > > > > > > for pcd in self._DynamicPcdList: > > > > > > if len(pcd.SkuInfoList) == 1: > > > > > > for (SkuName, SkuId) in allskuset: > > > > > > -- > > > > > > 2.39.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#115274): https://edk2.groups.io/g/devel/message/115274 Mute This Topic: https://groups.io/mt/103360241/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [edk2-devel] [PATCH] BaseTools: Remove Duplicate sets of SkuName and SkuId from allskuset 2024-01-08 0:29 ` Yuwei Chen 2024-01-08 3:54 ` Chan, Amy @ 2024-01-24 0:28 ` Michael D Kinney 2024-01-30 7:07 ` Ashraf Ali S 1 sibling, 1 reply; 11+ messages in thread From: Michael D Kinney @ 2024-01-24 0:28 UTC (permalink / raw) To: devel@edk2.groups.io, Chen, Christine, S, Ashraf Ali Cc: Rebecca Cran, Gao, Liming, Feng, Bob C, Chan, Amy, Chaganty, Rangasai V, Kinney, Michael D Hi Christine, I did not see this merged yet. Is this change also in edk2-basetools repo? I do not see it there. I thought the process was to do changed in edk2-basetools repo first where there are a large set of CI checks and then make the same change in the edk2 repo. Mike > -----Original Message----- > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Yuwei > Chen > Sent: Sunday, January 7, 2024 4:29 PM > To: S, Ashraf Ali <ashraf.ali.s@intel.com>; devel@edk2.groups.io > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > <gaoliming@byosoft.com.cn>; Feng, Bob C <bob.c.feng@intel.com>; Chan, > Amy <amy.chan@intel.com>; Chaganty, Rangasai V > <rangasai.v.chaganty@intel.com> > Subject: Re: [edk2-devel] [PATCH] BaseTools: Remove Duplicate sets of > SkuName and SkuId from allskuset > > Reviewed-by: Yuwei Chen <yuwei.chen@intel.com> > > > -----Original Message----- > > From: S, Ashraf Ali <ashraf.ali.s@intel.com> > > Sent: Monday, December 25, 2023 11:28 PM > > To: devel@edk2.groups.io > > Cc: S, Ashraf Ali <ashraf.ali.s@intel.com>; Chen, Christine > > <yuwei.chen@intel.com>; Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > > <gaoliming@byosoft.com.cn>; Feng, Bob C <bob.c.feng@intel.com>; Chan, > > Amy <amy.chan@intel.com>; Chaganty, Rangasai V > > <rangasai.v.chaganty@intel.com> > > Subject: [PATCH] BaseTools: Remove Duplicate sets of SkuName and SkuId > > from allskuset > > > > Currently when the platform has many SKUs then allskuset will be > having so > > many duplicate. and while parsing the allskuset will take longer time > while > > assing Pcd.SkuInfoList. > > This patch is to eliminate those duplicate entires to reduce the build > time > > > > Cc: Yuwei Chen <yuwei.chen@intel.com> > > Cc: Rebecca Cran <rebecca@bsdio.com> > > Cc: Liming Gao <gaoliming@byosoft.com.cn> > > Cc: Bob Feng <bob.c.feng@intel.com> > > Cc: Amy Chan <amy.chan@intel.com> > > Cc: Sai Chaganty <rangasai.v.chaganty@intel.com> > > Signed-off-by: Ashraf Ali S <ashraf.ali.s@intel.com> > > --- > > BaseTools/Source/Python/AutoGen/PlatformAutoGen.py | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > b/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > index 592d4824a4..dac81454a9 100644 > > --- a/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > +++ b/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > @@ -707,6 +707,8 @@ class PlatformAutoGen(AutoGen): > > self._DynamicPcdList.extend(list(OtherPcdArray)) > > self._DynamicPcdList.sort() > > allskuset = [(SkuName, Sku.SkuId) for pcd in > self._DynamicPcdList for > > (SkuName, Sku) in pcd.SkuInfoList.items()] > > + # Remove duplicate sets in the list > > + allskuset = list(set(allskuset)) > > for pcd in self._DynamicPcdList: > > if len(pcd.SkuInfoList) == 1: > > for (SkuName, SkuId) in allskuset: > > -- > > 2.39.1.windows.1 > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#114245): https://edk2.groups.io/g/devel/message/114245 Mute This Topic: https://groups.io/mt/103360241/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] 11+ messages in thread
* Re: [edk2-devel] [PATCH] BaseTools: Remove Duplicate sets of SkuName and SkuId from allskuset 2024-01-24 0:28 ` Michael D Kinney @ 2024-01-30 7:07 ` Ashraf Ali S 0 siblings, 0 replies; 11+ messages in thread From: Ashraf Ali S @ 2024-01-30 7:07 UTC (permalink / raw) To: Kinney, Michael D, devel@edk2.groups.io, Chen, Christine Cc: Rebecca Cran, Gao, Liming, Feng, Bob C, Chan, Amy, Chaganty, Rangasai V Hi., @Kinney, Michael D Thanks for the update. I have triggered the PR in edk2-basetools repo 😊 https://github.com/tianocore/edk2-basetools/pull/112 Thanks., S, Ashraf Ali -----Original Message----- From: Kinney, Michael D <michael.d.kinney@intel.com> Sent: Wednesday, January 24, 2024 5:59 AM To: devel@edk2.groups.io; Chen, Christine <yuwei.chen@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com> Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming <gaoliming@byosoft.com.cn>; Feng, Bob C <bob.c.feng@intel.com>; Chan, Amy <amy.chan@intel.com>; Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com> Subject: RE: [edk2-devel] [PATCH] BaseTools: Remove Duplicate sets of SkuName and SkuId from allskuset Hi Christine, I did not see this merged yet. Is this change also in edk2-basetools repo? I do not see it there. I thought the process was to do changed in edk2-basetools repo first where there are a large set of CI checks and then make the same change in the edk2 repo. Mike > -----Original Message----- > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Yuwei > Chen > Sent: Sunday, January 7, 2024 4:29 PM > To: S, Ashraf Ali <ashraf.ali.s@intel.com>; devel@edk2.groups.io > Cc: Rebecca Cran <rebecca@bsdio.com>; Gao, Liming > <gaoliming@byosoft.com.cn>; Feng, Bob C <bob.c.feng@intel.com>; Chan, > Amy <amy.chan@intel.com>; Chaganty, Rangasai V > <rangasai.v.chaganty@intel.com> > Subject: Re: [edk2-devel] [PATCH] BaseTools: Remove Duplicate sets of > SkuName and SkuId from allskuset > > Reviewed-by: Yuwei Chen <yuwei.chen@intel.com> > > > -----Original Message----- > > From: S, Ashraf Ali <ashraf.ali.s@intel.com> > > Sent: Monday, December 25, 2023 11:28 PM > > To: devel@edk2.groups.io > > Cc: S, Ashraf Ali <ashraf.ali.s@intel.com>; Chen, Christine > > <yuwei.chen@intel.com>; Rebecca Cran <rebecca@bsdio.com>; Gao, > > Liming <gaoliming@byosoft.com.cn>; Feng, Bob C > > <bob.c.feng@intel.com>; Chan, Amy <amy.chan@intel.com>; Chaganty, > > Rangasai V <rangasai.v.chaganty@intel.com> > > Subject: [PATCH] BaseTools: Remove Duplicate sets of SkuName and > > SkuId from allskuset > > > > Currently when the platform has many SKUs then allskuset will be > having so > > many duplicate. and while parsing the allskuset will take longer > > time > while > > assing Pcd.SkuInfoList. > > This patch is to eliminate those duplicate entires to reduce the > > build > time > > > > Cc: Yuwei Chen <yuwei.chen@intel.com> > > Cc: Rebecca Cran <rebecca@bsdio.com> > > Cc: Liming Gao <gaoliming@byosoft.com.cn> > > Cc: Bob Feng <bob.c.feng@intel.com> > > Cc: Amy Chan <amy.chan@intel.com> > > Cc: Sai Chaganty <rangasai.v.chaganty@intel.com> > > Signed-off-by: Ashraf Ali S <ashraf.ali.s@intel.com> > > --- > > BaseTools/Source/Python/AutoGen/PlatformAutoGen.py | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > b/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > index 592d4824a4..dac81454a9 100644 > > --- a/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > +++ b/BaseTools/Source/Python/AutoGen/PlatformAutoGen.py > > @@ -707,6 +707,8 @@ class PlatformAutoGen(AutoGen): > > self._DynamicPcdList.extend(list(OtherPcdArray)) > > self._DynamicPcdList.sort() > > allskuset = [(SkuName, Sku.SkuId) for pcd in > self._DynamicPcdList for > > (SkuName, Sku) in pcd.SkuInfoList.items()] > > + # Remove duplicate sets in the list > > + allskuset = list(set(allskuset)) > > for pcd in self._DynamicPcdList: > > if len(pcd.SkuInfoList) == 1: > > for (SkuName, SkuId) in allskuset: > > -- > > 2.39.1.windows.1 > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#114762): https://edk2.groups.io/g/devel/message/114762 Mute This Topic: https://groups.io/mt/103360241/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2024-02-08 19:27 UTC | newest] Thread overview: 11+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2023-12-25 15:27 [edk2-devel] [PATCH] BaseTools: Remove Duplicate sets of SkuName and SkuId from allskuset Ashraf Ali S 2024-01-08 0:29 ` Yuwei Chen 2024-01-08 3:54 ` Chan, Amy 2024-01-30 7:48 ` Feng, Bob C 2024-02-08 5:49 ` Ashraf Ali S 2024-02-08 5:53 ` Michael D Kinney 2024-02-08 6:38 ` Yuwei Chen 2024-02-08 16:30 ` Michael D Kinney 2024-02-08 19:27 ` Michael D Kinney 2024-01-24 0:28 ` Michael D Kinney 2024-01-30 7:07 ` Ashraf Ali S
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox