[OE-core] [PATCH] sstate: Add optimizing logic for crosssdk setscene dependencies
Ming Liu
ming.liu at windriver.com
Tue Jan 7 07:52:17 UTC 2014
On 01/06/2014 08:25 PM, Richard Purdie wrote:
> On Thu, 2013-11-14 at 18:51 +0800, Ming Liu wrote:
>> This patch mainly aims to add optimisation for crosssdk setscene dependency
>> validating which we haven't handled in current logic, and which I think we
>> could have as we've already implemented to native/cross, although there
>> are albeit not many crossdk tasks, we could still get some performance
>> enhancement.
>>
>> And it also fix a vulnerability of some certain workflow, think about the
>> following scenario with current logic:
>> bitbake nativesdk-eglibc-initial -c cleansstate
>> bitbake gcc-crosssdk-initial -c clean
>> bitbake gmp-native -c clean
>> bitbake libmpc-native -c clean
>> bitbake mpfr-native -c clean
>> bitbake gcc-crosssdk-initial
>> bitbake nativesdk-eglibc-initial
>>
>> Aboving will fail for absence of a few native libraries required by
>> gcc-crosssdk-initial.
>>
>> Also modified some places in current code except the optimisation, as
>> following:
>> 1 Remove isNative function since no code is referring it.
>> 2 Add do_package to the list that don't exist and are noexec.
> I've split this patch up as its doing too many things at once. In
> particular, I think we should keep the "isNativeCross()" function name
> instead of adding Crosssdk to the name since it just makes things more
> confusing to read.
I am on board with your comments.
>
> I've take a part for the crosssdk part in master-next which is being
> tested at the moment, can you resent the do_package part by itself
> please?
No problem, I will resend a patch soon with modifying the do_package
part only.
>
> I don't quite understand how the sequence of commands above breaks
> things or how this patch fixes it. Are you sure this wasn't fixed by:
>
> http://git.yoctoproject.org/cgit.cgi/poky-contrib/commit/?h=ciorga/PUs&id=1dcbf3096d7d42032faade96dae89c25a4feca7a
>
> which would be the real bug?
I am pretty sure they are**NOT the same issue. The 1dcbf309 is about to
fix gcc-crosssdk lacking its dependencies in bb, while in this case,
it's due to some other setscene tasks(populate_sysroot) depended by
gcc-crosssdk-initial's setscene task(populate_sysroot) were not
validated correctly, without the modification to isNativeCross, the
setscene_depvalid will return TRUE through:
......
# Native/Cross populate_sysroot need their dependencies
if isNativeCross(taskdependees[task][0]) and
isNativeCross(taskdependees[dep][0]):
return False
# Target populate_sysroot depended on by cross tools need
to be installed
if isNativeCross(taskdependees[dep][0]):
return False
# Native/cross tools depended upon by target sysroot are
not needed
if isNativeCross(taskdependees[task][0]):
continue
......
return True
......
for example, when the task is 'gmp-native', and the dep is
'gcc-crosssdk-initial', it is absolutely wrong for gcc-crosssdk-initial
needs its dependencies like gmp-native, libmpc-native, mpfr-native to be
present in sysroot or its binaries will fail to be executed.
However, by double-checked the patch, I found that it had introduced a
regression that binutils-crosssdk should not be added into safe dep list
for it's in the DEPENDS of gcc-crosssdk-initial, I will send a fix for
it too.
//Ming Liu
>
> Cheers,
>
> Richard
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20140107/6200e350/attachment-0002.html>
More information about the Openembedded-core
mailing list