> -----Original Message----- > From: Kinney, Michael D [mailto:michael.d.kin...@intel.com] > Sent: Friday, October 28, 2016 2:28 PM > To: Duran, Leo <leo.du...@amd.com>; edk2-de...@ml01.01.org; Kinney, > Michael D <michael.d.kin...@intel.com> > Cc: Fan, Jeff <jeff....@intel.com>; Gao, Liming <liming....@intel.com>; > 'Laszlo Ersek' <ler...@redhat.com> > Subject: RE: LocalApicLib: Why two separate directories? > > Leo, > > Your observation is correct, but the reason not to make this change now is > the DSC file changes required that would break platform builds. As Laszlo > points out, it is possible to do this type of change and coordinate update to > all > platforms in edk2/master. However, there are many other platforms that > use edk2 and a change like this would break them on next pull of > edk2/master. > > I believe the original X2 APIC implementation did not have as much common > code, so that was likely why it was added as a different library. > > I recommend we just leave them in their own directories for right now. > > Thanks, > > Mike > [Duran, Leo] Yes, that was my main concern. I suppose that in hindsight the two libraries could have being built from a common directory, allowing for (future) refactoring of code with breaking the .DSC's But, it is what it is... Thanks for the comments. Leo
> > > -----Original Message----- > > From: Duran, Leo [mailto:leo.du...@amd.com] > > Sent: Friday, October 28, 2016 12:04 PM > > To: edk2-de...@ml01.01.org > > Cc: Kinney, Michael D <michael.d.kin...@intel.com>; Fan, Jeff > > <jeff....@intel.com>; Gao, Liming <liming....@intel.com>; 'Laszlo > > Ersek' <ler...@redhat.com> > > Subject: LocalApicLib: Why two separate directories? > > > > All, > > Just a quick observation to request comments: > > > > Since a lot of the code in BaseXApicX2ApicLib.c and BaseXApicLib is > > the same, how about we merge the common code and build the libraries > from the same directory? > > > > UefiCpuPkg/Library/LocalApilLib/ > > - LocalApicLib.c --> common code > > - BaseXApicLib.c --> legacy APIC code > > - BaseXApicX2ApicLib.c --> X2APIC code > > - BaseXApicLib.inf -> builds from LocalApicLib.c + BaseXApicLib.c > > - BaseXApicX2ApicLib.inf -> builds from LocalApicLib.c + > > BaseXApicX2ApicLib.c > > > > Of course, doing this would require modification to existing .DSC > > files, to point to the appropriate .INF under the merged LocalApicLib > directory. > > Would that be too disruptive? > > > > Leo. > > > > > > > > _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel