[AMD Official Use Only - General] Hi Christine, Is the VFR python tool upstream to edk2-staging? May I know where can I get it? I would like to try it and see the flexibility of output some VFR information in JSON/YAML file. For edk2 Redfish, we are looking for the solution to automatically adding necessary edk2 Redfish drivers to platform DSC/FDF according to the UNI file.
Thanks Abner > -----Original Message----- > From: [email protected] <[email protected]> On Behalf Of Yuwei > Chen via groups.io > Sent: Tuesday, January 9, 2024 4:24 PM > To: Kinney, Michael D <[email protected]>; Pedro Falcato > <[email protected]>; [email protected] > Cc: Gao, Liming <[email protected]>; Rebecca Cran > <[email protected]>; Zimmer, Vincent <[email protected]>; Leif > Lindholm <[email protected]>; Andrew Fish <[email protected]>; > Feng, Bob C <[email protected]>; Yang, Yuting2 > <[email protected]>; Hartung, Stephen <[email protected]>; > Narayanan, Bimod <[email protected]> > Subject: Re: [edk2-devel] [edk2-stable202311][PATCH] BaseTools: Python > VfrCompiler implementation > > Caution: This message originated from an External Source. Use proper caution > when opening attachments, clicking links, or responding. > > > Hi Liming, > > Have you tried with the new python tool? Any comments or concerns? > > Thanks, > Christine > > > -----Original Message----- > > From: Kinney, Michael D <[email protected]> > > Sent: Saturday, December 16, 2023 1:31 AM > > To: Pedro Falcato <[email protected]>; [email protected]; Chen, > > Christine <[email protected]> > > Cc: Gao, Liming <[email protected]>; Rebecca Cran > > <[email protected]>; Zimmer, Vincent <[email protected]>; Leif > > Lindholm <[email protected]>; Andrew Fish <[email protected]>; > > Feng, Bob C <[email protected]>; Yang, Yuting2 > > <[email protected]>; Hartung, Stephen > <[email protected]>; > > Kinney, Michael D <[email protected]> > > Subject: RE: [edk2-devel] [edk2-stable202311][PATCH] BaseTools: Python > > VfrCompiler implementation > > > > There are several advantages for this direction: > > > > > > * Current VFR compiler in C has dependencies on very old libs that > > have not been updated. > > > > * The movement to python will remove the pre-build step that requires > > some of the build tools to be built using host C compiler before > > running edk2 build command. > > > > * The other element is moving all the python code into edk2-basetools > > repo with a published pip package. This enables the use of > > pip-requirements.txt to provide developers all the content needed > > to build. > > > > I agree that we should not have both VFR compilers. We need to make sure > > the new one in Python is 100% compatible with the C version and make the > > decision to simultaneously add Python one and delete the C one and commit > > to the Python one. I provided this feedback to the VFR developers in the > > TianoCore Tools/CI meeting earlier this year. > > > > The perf question is very good. It would be good for the VFR developers to > > provide some perf comparisons. I do not expect any significant different > > that > > would impact overall platform build times. > > > > Mike > > > > > -----Original Message----- > > > From: Pedro Falcato <[email protected]> > > > Sent: Friday, December 15, 2023 9:04 AM > > > To: [email protected]; Chen, Christine <[email protected]> > > > Cc: Gao, Liming <[email protected]>; Rebecca Cran > > > <[email protected]>; Zimmer, Vincent <[email protected]>; > > > Kinney, Michael D <[email protected]>; Leif Lindholm > > > <[email protected]>; Andrew Fish <[email protected]>; Feng, Bob > > > C <[email protected]>; Yang, Yuting2 <[email protected]>; > > > Hartung, Stephen <[email protected]> > > > Subject: Re: [edk2-devel] [edk2-stable202311][PATCH] BaseTools: Python > > > VfrCompiler implementation > > > > > > On Thu, Dec 7, 2023 at 9:08 AM Yuwei Chen <[email protected]> > > wrote: > > > > > > > > Hi Liming, > > > > > > > > > > > > > > > > Is this feature been tested and reviewed these two weeks? 😊 > > > > > > Two questions: > > > > > > 1) What testing strategy do you have to test for regressions in such a > > > huge rewrite? > > > 2) What's the point in shipping this to upstream if you're not aiming > > > for the replacement of the original VfrCompiler? > > > 3) What's the value of rewriting this in Python? If the existing > > > VfrCompiler is already working fine (AFAIK?), a python version will > > > likely just be slower (unless the original C version is super badly > > > written). > > > I *seriously* struggle to understand what this Python movement is > > > supposed to do, except gratuitously rewrite large bits of BaseTools > > > for a net loss (performance) > > > > > > -- > > > Pedro > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#116449): https://edk2.groups.io/g/devel/message/116449 Mute This Topic: https://groups.io/mt/102486097/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
