On 12/16/16 02:00, Kinney, Michael D wrote: > Laszlo, > > I agree with Jiewen that this is a compatible change. > > Reviewed-by: Michael Kinney <michael.d.kin...@intel.com>
Thank you both for the reviews, I pushed the series: 7c6075e2546d..32ea56f0a65b. Laszlo > > Mike > > From: Yao, Jiewen > Sent: Friday, December 9, 2016 2:58 PM > To: Laszlo Ersek <ler...@redhat.com>; Zeng, Star <star.z...@intel.com>; > edk2-devel-01 <edk2-de...@ml01.01.org> > Cc: Kinney, Michael D <michael.d.kin...@intel.com>; Tian, Feng > <feng.t...@intel.com>; Gao, Liming <liming....@intel.com>; Wei, David > <david....@intel.com> > Subject: RE: [edk2] [PATCH 0/3] MdePkg, MdeModulePkg, Vlv2TbltDevicePkg: > 64-bit LoopTimes in S3 MEM_POLL > > Hi Laszlo > According to your description on compatibility, I do not have concern. > So far, I do not receive any concern from any other people, so I think it is > good. > > Reviewed-by: jiewen....@intel.com<mailto:jiewen....@intel.com> > > Thank you > Yao Jiewen > > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Sunday, December 4, 2016 2:34 AM > To: Yao, Jiewen <jiewen....@intel.com<mailto:jiewen....@intel.com>>; Zeng, > Star <star.z...@intel.com<mailto:star.z...@intel.com>>; edk2-devel-01 > <edk2-de...@ml01.01.org<mailto:edk2-de...@ml01.01.org>> > Cc: Kinney, Michael D > <michael.d.kin...@intel.com<mailto:michael.d.kin...@intel.com>>; Tian, Feng > <feng.t...@intel.com<mailto:feng.t...@intel.com>>; Gao, Liming > <liming....@intel.com<mailto:liming....@intel.com>>; Wei, David > <david....@intel.com<mailto:david....@intel.com>> > Subject: Re: [edk2] [PATCH 0/3] MdePkg, MdeModulePkg, Vlv2TbltDevicePkg: > 64-bit LoopTimes in S3 MEM_POLL > > On 12/03/16 00:56, Yao, Jiewen wrote: >> HI Laszlo >> >> Would you please give me some time, so that I could discuss internal >> team to see if there is any concern on this API update? >> >> >> >> The estimation will be 1 work week at most. >> >> >> >> Usually adding a new API does not need compatibility check, but updating >> an existing API always need. > > Sure, thanks. > Laszlo > >> *From:*Laszlo Ersek [mailto:ler...@redhat.com] >> *Sent:* Friday, December 2, 2016 5:47 PM >> *To:* Zeng, Star <star.z...@intel.com<mailto:star.z...@intel.com>>; Yao, >> Jiewen >> <jiewen....@intel.com<mailto:jiewen....@intel.com>>; edk2-devel-01 >> <edk2-de...@ml01.01.org<mailto:edk2-de...@ml01.01.org>> >> *Cc:* Kinney, Michael D >> <michael.d.kin...@intel.com<mailto:michael.d.kin...@intel.com>>; Tian, Feng >> <feng.t...@intel.com<mailto:feng.t...@intel.com>>; Gao, Liming >> <liming....@intel.com<mailto:liming....@intel.com>>; Wei, David >> <david....@intel.com<mailto:david....@intel.com>> >> *Subject:* Re: [edk2] [PATCH 0/3] MdePkg, MdeModulePkg, >> Vlv2TbltDevicePkg: 64-bit LoopTimes in S3 MEM_POLL >> >> >> >> On 12/02/16 05:54, Zeng, Star wrote: >>> On 2016/12/2 9:53, Yao, Jiewen wrote: >>>> HI Laszlo >>>> Thank you to raise this long time existing issue. :-) >>>> >>>> There is historic reason that we inherit this undocumented API from >>>> EDK-I to EDKII. >>>> >>>> I do not object your update. >> >> Thanks! >> >>>> I am thinking if it is time to create a new API which can match PI >>>> specification, and suggest all consumers use the new API. >>>> >>>> RETURN_STATUS >>>> EFIAPI >>>> S3BootScriptSavePiMemPoll ( >>>> IN S3_BOOT_SCRIPT_LIB_WIDTH Width, >>>> IN UINT64 Address, >>>> IN VOID *Data, >>>> IN VOID *DataMask, >>>> IN UINT64 Delay, >>>> ); >>> >>> If the new API is introduced, the old one could be marked as deprecated >>> by DISABLE_NEW_DEPRECATED_INTERFACES. >> >> That's cool, but until then, can we please accept these patches? :) >> >> Thanks! >> Laszlo >> >>> Thanks, >>> Star >>> >>>> >>>> Thank you >>>> Yao Jiewen >>>> >>>> >>>>> -----Original Message----- >>>>> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of >>>>> Laszlo Ersek >>>>> Sent: Friday, December 2, 2016 1:56 AM >>>>> To: edk2-devel-01 <edk2-de...@ml01.01.org >>>>> <mailto:edk2-de...@ml01.01.org<mailto:edk2-de...@ml01.01.org >>>>> %3cmailto:edk2-de...@ml01.01.org>>> >>>>> Cc: Tian, Feng <feng.t...@intel.com >>>>> <mailto:feng.t...@intel.com<mailto:feng.t...@intel.com >>>>> %3cmailto:feng.t...@intel.com>>>; Gao, Liming >>>>> <liming....@intel.com >>>>> <mailto:liming....@intel.com<mailto:liming....@intel.com >>>>> %3cmailto:liming....@intel.com>>>; >>>>> Kinney, Michael D <michael.d.kin...@intel.com >>>>> <mailto:michael.d.kin...@intel.com<mailto:michael.d.kin...@intel.com >>>>> %3cmailto:michael.d.kin...@intel.com>>>; Zeng, Star >>>>> <star.z...@intel.com > <mailto:star.z...@intel.com%0b>> <mailto:star.z...@intel.com>>; Wei, David > <david....@intel.com > <mailto:david....@intel.com%0b>> <mailto:david....@intel.com>> >>>>> Subject: [edk2] [PATCH 0/3] MdePkg, MdeModulePkg, Vlv2TbltDevicePkg: >>>>> 64-bit LoopTimes in S3 MEM_POLL >>>>> >>>>> While working on S3 boot script related stuff in OVMF, I wanted to see >>>>> if infinite blocking was possible in the various POLL opcodes. While >>>>> edk2's implementation of IO_POLL, PCI_CONFIG_POLL and >>>>> PCI_CONFIG2_POLL >>>>> follows the PI spec vol5 closely, even internally (using 100ns delay >>>>> units), the MEM_POLL internals differ -- they are microseconds based. >>>>> >>>>> That's not a problem per se (it's just a different internal opcode >>>>> representation, which is fine); the problem is that the current >>>>> internals don't conform to the spec: in 32-bit builds, the UINT64 number >>>>> of 100ns units that the caller intends to wait for is silently >>>>> truncated, for no good reason. This issue is not hard to fix (we can >>>>> even keep the microseconds-based internals), so let's fix it. >>>>> >>>>> Repo: https://github.com/lersek/edk2/ >>>>> Branch: mempoll_looptimes_64bit >>>>> >>>>> Cc: David Wei <david....@intel.com >>>>> <mailto:david....@intel.com<mailto:david....@intel.com >>>>> %3cmailto:david....@intel.com>>> >>>>> Cc: Feng Tian <feng.t...@intel.com >>>>> <mailto:feng.t...@intel.com<mailto:feng.t...@intel.com >>>>> %3cmailto:feng.t...@intel.com>>> >>>>> Cc: Liming Gao <liming....@intel.com >>>>> <mailto:liming....@intel.com<mailto:liming....@intel.com >>>>> %3cmailto:liming....@intel.com>>> >>>>> Cc: Mang Guo <mang....@intel.com >>>>> <mailto:mang....@intel.com<mailto:mang....@intel.com >>>>> %3cmailto:mang....@intel.com>>> >>>>> Cc: Michael D Kinney <michael.d.kin...@intel.com >>>>> <mailto:michael.d.kin...@intel.com<mailto:michael.d.kin...@intel.com >>>>> %3cmailto:michael.d.kin...@intel.com>>> >>>>> Cc: Star Zeng <star.z...@intel.com >>>>> <mailto:star.z...@intel.com<mailto:star.z...@intel.com >>>>> %3cmailto:star.z...@intel.com>>> >>>>> >>>>> Thanks >>>>> Laszlo >>>>> >>>>> Laszlo Ersek (3): >>>>> MdePkg, MdeModulePkg: S3BootScriptSaveMemPoll(): accept 64-bit >>>>> LoopTimes >>>>> MdeModulePkg: S3SaveStateDxe, SmmS3SaveState: save 64-bit >>>>> LoopTimes >>>>> Vlv2TbltDevicePkg/BootScriptSaveDxe: save 64-bit LoopTimes >>>>> >>>>> MdeModulePkg/Library/PiDxeS3BootScriptLib/BootScriptSave.c | 2 +- >>>>> MdeModulePkg/Universal/Acpi/S3SaveStateDxe/S3SaveState.c | 8 >>>>> ++++---- >>>>> MdeModulePkg/Universal/Acpi/SmmS3SaveState/SmmS3SaveState.c | 8 >>>>> ++++---- >>>>> MdePkg/Include/Library/S3BootScriptLib.h | 2 +- >>>>> MdePkg/Library/BaseS3BootScriptLibNull/BootScriptLib.c | 2 +- >>>>> Vlv2TbltDevicePkg/BootScriptSaveDxe/ScriptSave.c | 4 ++-- >>>>> 6 files changed, 13 insertions(+), 13 deletions(-) >>>>> >>>>> -- >>>>> 2.9.2 >>>>> >>>>> _______________________________________________ >>>>> edk2-devel mailing list >>>>> edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org> >>>>> <mailto:edk2-devel@lists.01.org> >>>>> https://lists.01.org/mailman/listinfo/edk2-devel >>> >> > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.01.org > https://lists.01.org/mailman/listinfo/edk2-devel > _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel