From: Leif Lindholm <leif.lindholm@linaro.org>
To: Laszlo Ersek <lersek@redhat.com>
Cc: "Philippe Mathieu-Daudé" <philmd@redhat.com>,
edk2-devel@lists.01.org, "Liming Gao" <liming.gao@intel.com>,
"Bob Feng" <bob.c.feng@intel.com>,
"Zhiju . Fan" <zhijux.fan@intel.com>,
"Yonghong Zhu" <yonghong.zhu@intel.com>,
"Ard Biesheuvel" <ard.biesheuvel@linaro.org>
Subject: Re: [PATCH] BaseTools: Fix build failure when specifying multiple BUILDTARGET
Date: Tue, 5 Feb 2019 11:47:01 +0000 [thread overview]
Message-ID: <20190205114701.rzhrwb75uecwswjl@bivouac.eciton.net> (raw)
In-Reply-To: <bf5228c7-6ecc-4654-4449-a22217467d46@redhat.com>
On Tue, Feb 05, 2019 at 10:03:21AM +0100, Laszlo Ersek wrote:
> On 02/05/19 02:23, Philippe Mathieu-Daudé wrote:
> > Since 9c2d68c0a299 the build tools default to the python version
> > provided by the ${PYTHON} environment variable.
> > However the Python3 transition is not effective before d943b0c339fe.
>
> (1) Do you mean "functional" rather than "effective"?
>
> (2) Why is this information relevant for this commit? I see that commit
> f8d11e5a4aaa, referenced below, falls between the above two, but I'm
> unsure if that has any special relevance.
>
> If the above paragraph is just background info, that's OK with me, of
> course.
>
> > With Python3, the dict.value() method returns an iterator.
> > If a dictionary is updated while an iterator on his keys is used,
>
> (3) s/his/its/
>
> > a RuntimeError is generated.
> > Converting the iterator to a list() forces a copy of the mutable
> > keys in an immutable list which can be safely iterated.
> >
> > Commit f8d11e5a4aaa converted various uses but missed one:
> > When specifying multiple BUILDTARGET, the first target builds
> > successfully, but then the PGen.BuildDatabase._CACHE_ dictionary is
> > updated, and the next target accessing it triggers a RuntimeError.
>
> (4) Can we clarify this please; I think it's not the "next target" that
> accesses the dictionary, instead the code accesses the next target in
> the dictionary. How about
>
> s/the next target accessing it/accessing the next target/
>
> ?
>
> >
> > Convert this iterator to an immutable list, to solve this build error:
> >
> > $ build -a IA32 -t GCC5 -b RELEASE -b NOOPT -p OvmfPkg/OvmfPkgIa32.dsc
> > [...]
> > Processing meta-data ...
> > build.py...
> > : error C0DE: Unknown fatal error when processing [OvmfPkg/OvmfPkgIa32.dsc]
> >
> > (Please send email to edk2-devel@lists.01.org for help, attaching following call stack trace!)
> >
> > (Python 3.5.3 on linux) Traceback (most recent call last):
> > File "BaseTools/BinWrappers/PosixLike/../../Source/Python/build/build.py", line 2387, in Main
> > MyBuild.Launch()
> > File "BaseTools/BinWrappers/PosixLike/../../Source/Python/build/build.py", line 2141, in Launch
> > self._MultiThreadBuildPlatform()
> > File "BaseTools/BinWrappers/PosixLike/../../Source/Python/build/build.py", line 1921, in _MultiThreadBuildPlatform
> > self.Progress
> > File "BaseTools/Source/Python/AutoGen/AutoGen.py", line 304, in __init__
> > self._InitWorker(Workspace, MetaFile, Target, Toolchain, Arch, *args, **kwargs)
> > File "BaseTools/Source/Python/AutoGen/AutoGen.py", line 477, in _InitWorker
> > for BuildData in PGen.BuildDatabase._CACHE_.values():
> > RuntimeError: dictionary changed size during iteration
> >
> > Reported-by: Leif Lindholm <leif.lindholm@linaro.org>
> > Fixes: f8d11e5a4aaa90bf63b4789f3993dd6d16c60787
> > Contributed-under: TianoCore Contribution Agreement 1.1
> > Signed-off-by: Philippe Mathieu-Daude <philmd@redhat.com>
> > Tested-by: Leif Lindholm <leif.lindholm@linaro.org>
> > ---
> > BaseTools/Source/Python/AutoGen/AutoGen.py | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/BaseTools/Source/Python/AutoGen/AutoGen.py b/BaseTools/Source/Python/AutoGen/AutoGen.py
> > index a95d2c710e..12592a2a46 100644
> > --- a/BaseTools/Source/Python/AutoGen/AutoGen.py
> > +++ b/BaseTools/Source/Python/AutoGen/AutoGen.py
> > @@ -474,7 +474,7 @@ class WorkspaceAutoGen(AutoGen):
> >
> > # generate the SourcePcdDict and BinaryPcdDict
> > PGen = PlatformAutoGen(self, self.MetaFile, Target, Toolchain, Arch)
> > - for BuildData in PGen.BuildDatabase._CACHE_.values():
> > + for BuildData in list(PGen.BuildDatabase._CACHE_.values()):
> > if BuildData.Arch != Arch:
> > continue
> > if BuildData.MetaFile.Ext == '.inf':
> >
>
> LGTM :)
>
> With the commit message updated (as you prefer):
>
> Acked-by: Laszlo Ersek <lersek@redhat.com>
I would be surprised if we hear back from the BaseTools maintainers
this week (Chinese New Year). For me, I have both a simple workaround
and this patch, so I'm OK with waiting.
But if anyone reports issues with CI environments or similar, I would
say either you or I could push this patch in their absence.
/
Leif
next prev parent reply other threads:[~2019-02-05 11:47 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-02-05 1:23 [PATCH] BaseTools: Fix build failure when specifying multiple BUILDTARGET Philippe Mathieu-Daudé
2019-02-05 9:03 ` Laszlo Ersek
2019-02-05 11:47 ` Leif Lindholm [this message]
2019-02-05 12:04 ` Laszlo Ersek
2019-02-05 21:20 ` Carsey, Jaben
2019-02-06 16:16 ` Laszlo Ersek
2019-02-05 12:02 ` Philippe Mathieu-Daudé
2019-02-05 12:06 ` Laszlo Ersek
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=20190205114701.rzhrwb75uecwswjl@bivouac.eciton.net \
--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