From: Ming Huang <huangming23@huawei.com>
To: "Jiang, Guomin" <guomin.jiang@intel.com>,
"devel@edk2.groups.io" <devel@edk2.groups.io>,
"Wang, Jian J" <jian.j.wang@intel.com>,
"Wu, Hao A" <hao.a.wu@intel.com>,
"Gao, Liming" <liming.gao@intel.com>
Cc: "lidongzhan@huawei.com" <lidongzhan@huawei.com>,
"songdongkuang@huawei.com" <songdongkuang@huawei.com>,
"wanghuiqiang@huawei.com" <wanghuiqiang@huawei.com>,
"qiuliangen@huawei.com" <qiuliangen@huawei.com>,
"shenlimei@huawei.com" <shenlimei@huawei.com>
Subject: Re: [edk2-devel] [RFC edk2 v1 1/1] MdeModulePkg/Variable: Move FindVariable after AutoUpdateLangVariable
Date: Wed, 27 May 2020 22:40:15 +0800 [thread overview]
Message-ID: <394ac7a5-7010-f8d0-62f8-fa51f807af8d@huawei.com> (raw)
In-Reply-To: <DM6PR11MB295587FD4D8F64981E11E3A09DB10@DM6PR11MB2955.namprd11.prod.outlook.com>
Hi Guomin,
This patch change the position of invoking the FindVariable() function to
after AutoUpdateLangVariable() invoked.
Thanks,
Ming
在 2020/5/27 8:12, Jiang, Guomin 写道:
> Hi Huangming,
>
> I will clarify it when I am free, please be patient.
>
> If it is urgent for you, I suggest that you can use your patch temporarily.
>
> Any other thing, I check your patch in the mail, it seem that not any change in code, can you double confirm it?
>
> Best Regards
>> -----Original Message-----
>> From: Ming Huang <huangming23@huawei.com>
>> Sent: Tuesday, May 26, 2020 2:08 PM
>> To: Jiang, Guomin <guomin.jiang@intel.com>; devel@edk2.groups.io; Wang,
>> Jian J <jian.j.wang@intel.com>; Wu, Hao A <hao.a.wu@intel.com>; Gao,
>> Liming <liming.gao@intel.com>
>> Cc: lidongzhan@huawei.com; songdongkuang@huawei.com;
>> wanghuiqiang@huawei.com; qiuliangen@huawei.com;
>> shenlimei@huawei.com
>> Subject: Re: [edk2-devel] [RFC edk2 v1 1/1] MdeModulePkg/Variable: Move
>> FindVariable after AutoUpdateLangVariable
>>
>>
>>
>> 在 2020/5/26 8:39, Jiang, Guomin 写道:
>>> Hi Huangming,
>>>
>>> I am taking the bugzilla and I am sorry that I haven't provide you with
>> productive comment.
>>>
>>> I am still busy until August.
>>>
>>> I just want to know that:
>>> 1. Have you verified that the symptom will disappear after invoked
>> FindVariable() function?
>>
>> Yes, the symptom will disappeare after add this patch.
>>
>>> 2. Is it your suggestion that the FindVariable() need to be invoked but you
>> have no idea that how to fix it?
>>
>> This patch can fix this issue, and I guess this issue was resulted by adding
>> AutoUpdateLangVariable feature.
>> I hope this patch can be merged to edk2 master.
>>
>> Thanks
>> Ming
>>
>>>
>>> Best Regards
>>> Guomin
>>>> -----Original Message-----
>>>> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Ming
>>>> Huang
>>>> Sent: Monday, May 25, 2020 7:34 PM
>>>> To: devel@edk2.groups.io; Wang, Jian J <jian.j.wang@intel.com>; Wu,
>>>> Hao A <hao.a.wu@intel.com>; Gao, Liming <liming.gao@intel.com>
>>>> Cc: lidongzhan@huawei.com; huangming23@huawei.com;
>>>> songdongkuang@huawei.com; wanghuiqiang@huawei.com;
>>>> qiuliangen@huawei.com; shenlimei@huawei.com
>>>> Subject: [edk2-devel] [RFC edk2 v1 1/1] MdeModulePkg/Variable: Move
>>>> FindVariable after AutoUpdateLangVariable
>>>>
>>>> When occur reclaim in AutoUpdateLangVariable(), the CurrPtr of
>>>> Variable is invalid. The State will be update with wrong position
>>>> after UpdateVariable in this situation and two valid PlatformLang or Lang
>> variables will exist.
>>>> BmForEachVariable() will enter endless loop while exist two valid
>>>> PlatformLang variables. So FindVariable() should be invoked atfer
>>>> AutoUpdateLangVariable().
>>>>
>>>> https://bugzilla.tianocore.org/show_bug.cgi?id=2667
>>>>
>>>> Signed-off-by: Ming Huang <huangming23@huawei.com>
>>>> ---
>>>> MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c | 26
>>>> ++++++++++----------
>>>> 1 file changed, 13 insertions(+), 13 deletions(-)
>>>>
>>>> diff --git a/MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c
>>>> b/MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c
>>>> index 1e71fc6..0cec981 100644
>>>> --- a/MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c
>>>> +++ b/MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c
>>>> @@ -2741,6 +2741,19 @@ VariableServiceSetVariable (
>>>> mVariableModuleGlobal->NonVolatileLastVariableOffset = (UINTN)
>>>> NextVariable - (UINTN) Point;
>>>> }
>>>>
>>>> + if (!FeaturePcdGet (PcdUefiVariableDefaultLangDeprecate)) {
>>>> + //
>>>> + // Hook the operation of setting PlatformLangCodes/PlatformLang
>>>> + and
>>>> LangCodes/Lang.
>>>> + //
>>>> + Status = AutoUpdateLangVariable (VariableName, Data, DataSize);
>>>> + if (EFI_ERROR (Status)) {
>>>> + //
>>>> + // The auto update operation failed, directly return to avoid
>>>> inconsistency between PlatformLang and Lang.
>>>> + //
>>>> + goto Done;
>>>> + }
>>>> + }
>>>> +
>>>> //
>>>> // Check whether the input variable is already existed.
>>>> //
>>>> @@ -2763,19 +2776,6 @@ VariableServiceSetVariable (
>>>> }
>>>> }
>>>>
>>>> - if (!FeaturePcdGet (PcdUefiVariableDefaultLangDeprecate)) {
>>>> - //
>>>> - // Hook the operation of setting PlatformLangCodes/PlatformLang and
>>>> LangCodes/Lang.
>>>> - //
>>>> - Status = AutoUpdateLangVariable (VariableName, Data, DataSize);
>>>> - if (EFI_ERROR (Status)) {
>>>> - //
>>>> - // The auto update operation failed, directly return to avoid
>> inconsistency
>>>> between PlatformLang and Lang.
>>>> - //
>>>> - goto Done;
>>>> - }
>>>> - }
>>>> -
>>>> if (mVariableModuleGlobal->VariableGlobal.AuthSupport) {
>>>> Status = AuthVariableLibProcessVariable (VariableName,
>>>> VendorGuid, Data, DataSize, Attributes);
>>>> } else {
>>>> --
>>>> 2.8.1
>>>>
>>>>
>>>>
>>>
>>>
>>>
>
>
> .
>
next prev parent reply other threads:[~2020-05-27 14:40 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-05-25 11:34 [RFC edk2 v1 0/1] Fix a infrequent issue in variable Ming Huang
2020-05-25 11:34 ` [RFC edk2 v1 1/1] MdeModulePkg/Variable: Move FindVariable after AutoUpdateLangVariable Ming Huang
2020-05-26 0:39 ` [edk2-devel] " Guomin Jiang
2020-05-26 6:08 ` Ming Huang
2020-05-27 0:12 ` Guomin Jiang
2020-05-27 14:40 ` Ming Huang [this message]
2020-05-28 0:28 ` Guomin Jiang
2020-06-29 3:03 ` [edk2-devel] [RFC edk2 v1 0/1] Fix a infrequent issue in variable Guomin Jiang
2020-06-29 3:59 ` Ming Huang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=394ac7a5-7010-f8d0-62f8-fa51f807af8d@huawei.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox