diff options
| author |   Richard Purdie <richard.purdie@linuxfoundation.org> | 2014-04-13 11:49:45 +0100 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2014-04-21 23:05:55 +0100 |
| commit | 92526eadd09d19938762290e0492076174367583 (patch) | |
| tree | 52e2dda6880662c92a4643d19b0209793062a792 | |
| parent | b56918c7ef7913e84356c69ee9b269844a446728 (diff) | |
| download | bitbake-92526eadd09d19938762290e0492076174367583.tar.gz | |
bitbake-worker: Drop BBHASH variables
Iterating through and calling setVar on this number of variables has significant
overhead in the profiling data. By not setting this, we save 3,000 calls
to setVar which gives a noticeable improvement to the speed of task execution.
The BBHASH variables have since been replaced by accessing that data through
the siggen code and going forward, that is the preferred way work with it.
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
| -rwxr-xr-x | bin/bitbake-worker | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/bin/bitbake-worker b/bin/bitbake-worker index 68e2bf457..05e0cf631 100755 --- a/bin/bitbake-worker +++ b/bin/bitbake-worker @@ -159,10 +159,6 @@ def fork_off_task(cfg, data, workerdata, fn, task, taskname, appends, taskdepdat try: the_data = bb.cache.Cache.loadDataFull(fn, appends, data) the_data.setVar('BB_TASKHASH', workerdata["runq_hash"][task]) - for h in workerdata["hashes"]: - the_data.setVar("BBHASH_%s" % h, workerdata["hashes"][h]) - for h in workerdata["hash_deps"]: - the_data.setVar("BBHASHDEPS_%s" % h, workerdata["hash_deps"][h]) # exported_vars() returns a generator which *cannot* be passed to os.environ.update() # successfully. We also need to unset anything from the environment which shouldn't be there |