On 07/21/2017 03:11 AM, Richard Purdie wrote: > On Mon, 2017-07-17 at 16:45 -0500, Aníbal Limón wrote: >> This v3 makes changes on tinfoil busy status patch and adds an >> environment variable to oe-seltest script to specify number of >> threads. >> There are patches for bitbake [3] and meta-yocto-bsp [4] that was >> send >> to their specific ML. >> >> The first 6 patches previous enable support of threads in oeqa core >> are independent and can be merged first. >> >> The v2 address comments did by Patrick about add better explanation >> on the commit messages and some typos. >> >> Other interesting information to add is that with this series we will >> able to execute oe-selftest in around half [1] of the original time >> [2], (9406.782s vs 17303.014s). > > I tested this, admittedly on top of the server rework changes and see > some odd failures on the autobuilder. Things aren't looking very > deterministic and I'm a little worried about merging them. > > (e.g. https://autobuilder.yocto.io/builders/nightly-oe-selftest/builds/ > 397) > > The second issue I'd like to ask questions about is whether the tinfoil > changes and server rework we've just done remove the need for some of > this patchset?
Yes if the tinfoil API can work within a thread we can remove the functionality to run into the main thread, but the end_thread functionality needs to stay because the devtool test content will need to be modified to don't touch meta/ folder. > > I ask since now I'm hoping the "main thread" issues might be resolved > and we truly can have independent tinfoil connections. There were > issues there but I believe Paul has resolved those. > > For that reason I'm going to defer this patchset from M2 (I did take > some of the initial patches) and ask that we review the patchset in > light of the server rework and tinfoil changes that just merged. > > I do know that for example, "oeqa/selftest/case: Support bitbake memres > mode in per build director" is no obsolete as we removed the memres > init script and the -t option to bitbake. Ok, i understand, after merge the patches from Paul and you, i will rebase and test again searching for remove has much code as i can. Cheers, Anibal > > Cheers, > > Richard > -- _______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
