Re: [edk2-devel] [Bug 3187] FaultTolerantWriteDxe defect will cause NVRAM not recovered after number of WorkSpaceRefresh().

["Keysound Chang" <keysound_chang@phoenix.com>]

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

After checking Bugzilla 3187, NVRAM already corrupted in this case. Sorry that I didn’t aware.

From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Keysound Chang via groups.io
Sent: Friday, February 5, 2021 11:03 AM
To: devel@edk2.groups.io; marlboro.chuang@dell.com; gaoliming <gaoliming@byosoft.com.cn>
Subject: Re: [edk2-devel] [Bug 3187] FaultTolerantWriteDxe defect will cause NVRAM not recovered after number of WorkSpaceRefresh().

Hi Marlboro,

How about use non-volatile EFI variable instead of CMOS? Not sure all platforms support CMOS.

Regards,

From: devel@edk2.groups.io<mailto:devel@edk2.groups.io> <devel@edk2.groups.io<mailto:devel@edk2.groups.io>> On Behalf Of Chuang, Marlboro via groups.io
Sent: Thursday, February 4, 2021 3:08 PM
To: gaoliming <gaoliming@byosoft.com.cn<mailto:gaoliming@byosoft.com.cn>>; devel@edk2.groups.io<mailto:devel@edk2.groups.io>
Subject: Re: [edk2-devel] [Bug 3187] FaultTolerantWriteDxe defect will cause NVRAM not recovered after number of WorkSpaceRefresh().

HI Gaoliming,

The DupCodeChange is for simulating user force power off the system.
So I just use the CMOS to record the temporarily flag to ensure the code will not enter the infinite loop to reset the system.

Best Regards,
Marlboro.


==============================================
Marlboro Chuang
Firmware Engineer
Dell | TDC BIOS Core Team
Office : +886-2-23766313
Mobile: +886-986615685
Marlboro.Chuang@dell.com<mailto:Marlboro.Chuang@dell.com>
==============================================

-----Original Message-----
From: gaoliming <gaoliming@byosoft.com.cn<mailto:gaoliming@byosoft.com.cn>>
Sent: Thursday, February 4, 2021 2:55 PM
To: Chuang, Marlboro; devel@edk2.groups.io<mailto:devel@edk2.groups.io>
Subject: 回复: [Bug 3187] FaultTolerantWriteDxe defect will cause NVRAM not recovered after number of WorkSpaceRefresh().


[EXTERNAL EMAIL]

Chuang:
I see you directly use IO port 0x70, 0x71. What purpose to use them?

Thanks
Liming
> -----邮件原件-----
> 发件人: Chuang, Marlboro <Marlboro.Chuang@dell.com<mailto:Marlboro.Chuang@dell.com>>
> 发送时间: 2021年2月3日 12:59
> 收件人: devel@edk2.groups.io<mailto:devel@edk2.groups.io>
> 抄送: gaoliming@byosoft.com.cn<mailto:gaoliming@byosoft.com.cn>
> 主题: RE: [Bug 3187] FaultTolerantWriteDxe defect will cause NVRAM not
> recovered after number of WorkSpaceRefresh().
>
> Hi All,
>
> Regarding to Bug 3187, I have the duplicated code change and fix code
> change as the attachment.
> Please help to review and refine it.
>
> Thanks and Regards,
> Marlboro.
>
>
> -----Original Message-----
> From: bugzilla-daemon@bugzilla.tianocore.org<mailto:bugzilla-daemon@bugzilla.tianocore.org>
> <bugzilla-daemon@bugzilla.tianocore.org<mailto:bugzilla-daemon@bugzilla.tianocore.org>>
> Sent: Wednesday, February 3, 2021 11:04 AM
> To: Chuang, Marlboro
> Subject: [Bug 3187] FaultTolerantWriteDxe defect will cause NVRAM not
> recovered after number of WorkSpaceRefresh().
>
>
> [EXTERNAL EMAIL]
>
> https://bugzilla.tianocore.org/show_bug.cgi?id=3187<https://bugzilla.tianocore.org/show_bug.cgi?id=3187>
>
> gaoliming@byosoft.com.cn<mailto:gaoliming@byosoft.com.cn> changed:
>
> What |Removed |Added
> ----------------------------------------------------------------------------
> Priority|Lowest |Normal
> Status|UNCONFIRMED |CONFIRMED
> CC|
> |gaoliming@byosoft.com.cn<mailto:|gaoliming@byosoft.com.cn>
> Assignee|unassigned@tianocore.org<mailto:Assignee|unassigned@tianocore.org>
> |Marlboro.Chuang@dell.com<mailto:|Marlboro.Chuang@dell.com>
> Ever confirmed|0 |1
>
> --- Comment #2 from gaoliming@byosoft.com.cn<mailto:gaoliming@byosoft.com.cn> ---
> @Marlboro: can you send your patch to edk2 mail list?
>
> --
> You are receiving this mail because:
> You are on the CC list for the bug.
> You are the assignee for the bug.
> You reported the bug.







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