* [PATCH 0/4][Image Creator]Add mulitple extra cache fields request support
@ 2011-05-26 2:53 ` Ke Liping
0 siblings, 0 replies; 15+ messages in thread
From: Ke Liping @ 2011-05-26 2:51 UTC (permalink / raw)
To: yocto
From: Liping Ke <liping.ke@intel.com>
The series of patch are for mplementing multiple extra cache fields
request support.If user needs to request extra cache fields besides
CoreRecipeInfo fields, just add a new XXXRecipeInfo class definition
as Image Creator UI Hob Does.
For implementing this new feature, we refactory current cache code,
remove namedtuple and define corresponding class for dynamical cache
fields support per different cache usage request.
Liping Ke (4):
Refactory Current Cache implementation
Introduce extra cache class for image creator
Introduce new param caches_array into Cache impl.
Implement multiple extra cache fields request support
bitbake/lib/bb/cache.py | 532 ++++++++++++++++++++++-------------------
bitbake/lib/bb/cache_extra.py | 54 ++++
bitbake/lib/bb/cooker.py | 70 ++++--
bitbake/lib/bb/shell.py | 2 +-
bitbake/lib/bb/ui/hob.py | 2 +
5 files changed, 396 insertions(+), 264 deletions(-)
create mode 100644 bitbake/lib/bb/cache_extra.py
^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH 1/4][Image Creator]Refactory Current Cache implementation
2011-05-26 2:53 ` Ke Liping
@ 2011-05-26 2:53 ` Ke Liping
-1 siblings, 0 replies; 15+ messages in thread
From: Ke Liping @ 2011-05-26 2:52 UTC (permalink / raw)
To: yocto
From: Liping Ke <liping.ke@intel.com>
This patch is for refactorying current cache implementation, the
main reason is for introducing extra cache fields requests for
image creator as well as other users. The refactory parts include:
Move cache data retrieve methods out of Cache Data Fields
Definition. Since this retrieve methods will be shared for
both CoreRecipeInfo as well as the new introduced extra RecipeInfo
in the following patches.
Signed-off-by: Liping Ke <liping.ke@intel.com>
---
bitbake/lib/bb/cache.py | 358 ++++++++++++++++++++++-------------------------
1 files changed, 165 insertions(+), 193 deletions(-)
diff --git a/bitbake/lib/bb/cache.py b/bitbake/lib/bb/cache.py
index d083c51..09691d9 100644
--- a/bitbake/lib/bb/cache.py
+++ b/bitbake/lib/bb/cache.py
@@ -30,7 +30,7 @@
import os
import logging
-from collections import defaultdict, namedtuple
+from collections import defaultdict
import bb.data
import bb.utils
@@ -45,46 +45,11 @@ except ImportError:
__cache_version__ = "138"
-recipe_fields = (
- 'pn',
- 'pv',
- 'pr',
- 'pe',
- 'defaultpref',
- 'depends',
- 'provides',
- 'task_deps',
- 'stamp',
- 'stamp_extrainfo',
- 'broken',
- 'not_world',
- 'skipped',
- 'timestamp',
- 'packages',
- 'packages_dynamic',
- 'rdepends',
- 'rdepends_pkg',
- 'rprovides',
- 'rprovides_pkg',
- 'rrecommends',
- 'rrecommends_pkg',
- 'nocache',
- 'variants',
- 'file_depends',
- 'tasks',
- 'basetaskhashes',
- 'hashfilename',
- 'inherits',
- 'summary',
- 'license',
- 'section',
- 'fakerootenv',
- 'fakerootdirs'
-)
-
-
-class RecipeInfo(namedtuple('RecipeInfo', recipe_fields)):
- __slots__ = ()
+
+# RecipeInfoCommon defines common data retrieving methods
+# from meta data for caches. CoreRecipeInfo as well as other
+# Extra RecipeInfo needs to inherit this class
+class RecipeInfoCommon(object):
@classmethod
def listvar(cls, var, metadata):
@@ -117,69 +82,162 @@ class RecipeInfo(namedtuple('RecipeInfo', recipe_fields)):
def getvar(cls, var, metadata):
return metadata.getVar(var, True) or ''
- @classmethod
- def make_optional(cls, default=None, **kwargs):
- """Construct the namedtuple from the specified keyword arguments,
- with every value considered optional, using the default value if
- it was not specified."""
- for field in cls._fields:
- kwargs[field] = kwargs.get(field, default)
- return cls(**kwargs)
+
+class CoreRecipeInfo(RecipeInfoCommon):
+ __slots__ = ()
+
+ def __init__(self, filename, metadata):
+ self.name = "base"
+ # please override this member with the correct data cache file
+ # such as (bb_cache.dat, bb_extracache_hob.dat)
+ self.cachefile = "bb_cache.dat"
+
+ self.file_depends = metadata.getVar('__depends', False)
+ self.timestamp = bb.parse.cached_mtime(filename)
+ self.variants = self.listvar('__VARIANTS', metadata) + ['']
+ self.nocache = self.getvar('__BB_DONT_CACHE', metadata)
+
+ if self.getvar('__SKIPPED', metadata):
+ self.skipped = True
+ return
+
+ self.tasks = metadata.getVar('__BBTASKS', False)
+
+ self.pn = self.getvar('PN', metadata)
+ self.packages = self.listvar('PACKAGES', metadata)
+ if not self.pn in self.packages:
+ self.packages.append(self.pn)
+
+ self.basetaskhashes = self.taskvar('BB_BASEHASH', self.tasks, metadata)
+ self.hashfilename = self.getvar('BB_HASHFILENAME', metadata)
+
+ self.file_depends = metadata.getVar('__depends', False)
+ self.task_deps = metadata.getVar('_task_deps', False) or {'tasks': [], 'parents': {}}
+
+ self.skipped = False
+ self.pe = self.getvar('PE', metadata)
+ self.pv = self.getvar('PV', metadata)
+ self.pr = self.getvar('PR', metadata)
+ self.defaultpref = self.intvar('DEFAULT_PREFERENCE', metadata)
+ self.broken = self.getvar('BROKEN', metadata)
+ self.not_world = self.getvar('EXCLUDE_FROM_WORLD', metadata)
+ self.stamp = self.getvar('STAMP', metadata)
+ self.stamp_extrainfo = self.flaglist('stamp-extra-info', self.tasks, metadata)
+ self.packages_dynamic = self.listvar('PACKAGES_DYNAMIC', metadata)
+ self.depends = self.depvar('DEPENDS', metadata)
+ self.provides = self.depvar('PROVIDES', metadata)
+ self.rdepends = self.depvar('RDEPENDS', metadata)
+ self.rprovides = self.depvar('RPROVIDES', metadata)
+ self.rrecommends = self.depvar('RRECOMMENDS', metadata)
+ self.rprovides_pkg = self.pkgvar('RPROVIDES', self.packages, metadata)
+ self.rdepends_pkg = self.pkgvar('RDEPENDS', self.packages, metadata)
+ self.rrecommends_pkg = self.pkgvar('RRECOMMENDS', self.packages, metadata)
+ self.inherits = self.getvar('__inherit_cache', metadata)
+ self.summary = self.getvar('SUMMARY', metadata)
+ self.license = self.getvar('LICENSE', metadata)
+ self.section = self.getvar('SECTION', metadata)
+ self.fakerootenv = self.getvar('FAKEROOTENV', metadata)
+ self.fakerootdirs = self.getvar('FAKEROOTDIRS', metadata)
@classmethod
- def from_metadata(cls, filename, metadata):
- if cls.getvar('__SKIPPED', metadata):
- return cls.make_optional(skipped=True,
- file_depends=metadata.getVar('__depends', False),
- timestamp=bb.parse.cached_mtime(filename),
- variants=cls.listvar('__VARIANTS', metadata) + [''])
-
- tasks = metadata.getVar('__BBTASKS', False)
-
- pn = cls.getvar('PN', metadata)
- packages = cls.listvar('PACKAGES', metadata)
- if not pn in packages:
- packages.append(pn)
-
- return RecipeInfo(
- tasks = tasks,
- basetaskhashes = cls.taskvar('BB_BASEHASH', tasks, metadata),
- hashfilename = cls.getvar('BB_HASHFILENAME', metadata),
-
- file_depends = metadata.getVar('__depends', False),
- task_deps = metadata.getVar('_task_deps', False) or
- {'tasks': [], 'parents': {}},
- variants = cls.listvar('__VARIANTS', metadata) + [''],
-
- skipped = False,
- timestamp = bb.parse.cached_mtime(filename),
- packages = cls.listvar('PACKAGES', metadata),
- pn = pn,
- pe = cls.getvar('PE', metadata),
- pv = cls.getvar('PV', metadata),
- pr = cls.getvar('PR', metadata),
- nocache = cls.getvar('__BB_DONT_CACHE', metadata),
- defaultpref = cls.intvar('DEFAULT_PREFERENCE', metadata),
- broken = cls.getvar('BROKEN', metadata),
- not_world = cls.getvar('EXCLUDE_FROM_WORLD', metadata),
- stamp = cls.getvar('STAMP', metadata),
- stamp_extrainfo = cls.flaglist('stamp-extra-info', tasks, metadata),
- packages_dynamic = cls.listvar('PACKAGES_DYNAMIC', metadata),
- depends = cls.depvar('DEPENDS', metadata),
- provides = cls.depvar('PROVIDES', metadata),
- rdepends = cls.depvar('RDEPENDS', metadata),
- rprovides = cls.depvar('RPROVIDES', metadata),
- rrecommends = cls.depvar('RRECOMMENDS', metadata),
- rprovides_pkg = cls.pkgvar('RPROVIDES', packages, metadata),
- rdepends_pkg = cls.pkgvar('RDEPENDS', packages, metadata),
- rrecommends_pkg = cls.pkgvar('RRECOMMENDS', packages, metadata),
- inherits = cls.getvar('__inherit_cache', metadata),
- summary = cls.getvar('SUMMARY', metadata),
- license = cls.getvar('LICENSE', metadata),
- section = cls.getvar('SECTION', metadata),
- fakerootenv = cls.getvar('FAKEROOTENV', metadata),
- fakerootdirs = cls.getvar('FAKEROOTDIRS', metadata),
- )
+ def init_cacheData(cls, cachedata):
+ # CacheData in Core RecipeInfo Class
+ cachedata.task_deps = {}
+ cachedata.pkg_fn = {}
+ cachedata.pkg_pn = defaultdict(list)
+ cachedata.pkg_pepvpr = {}
+ cachedata.pkg_dp = {}
+
+ cachedata.stamp = {}
+ cachedata.stamp_extrainfo = {}
+ cachedata.fn_provides = {}
+ cachedata.pn_provides = defaultdict(list)
+ cachedata.all_depends = []
+
+ cachedata.deps = defaultdict(list)
+ cachedata.packages = defaultdict(list)
+ cachedata.providers = defaultdict(list)
+ cachedata.rproviders = defaultdict(list)
+ cachedata.packages_dynamic = defaultdict(list)
+
+ cachedata.rundeps = defaultdict(lambda: defaultdict(list))
+ cachedata.runrecs = defaultdict(lambda: defaultdict(list))
+ cachedata.possible_world = []
+ cachedata.universe_target = []
+ cachedata.hashfn = {}
+
+ cachedata.basetaskhash = {}
+ cachedata.inherits = {}
+ cachedata.summary = {}
+ cachedata.license = {}
+ cachedata.section = {}
+ cachedata.fakerootenv = {}
+ cachedata.fakerootdirs = {}
+
+ def add_cacheData(self, cachedata, fn):
+ cachedata.task_deps[fn] = self.task_deps
+ cachedata.pkg_fn[fn] = self.pn
+ cachedata.pkg_pn[self].append(fn)
+ cachedata.pkg_pepvpr[fn] = (self.pe, self.pv, self.pr)
+ cachedata.pkg_dp[fn] = self.defaultpref
+ cachedata.stamp[fn] = self.stamp
+ cachedata.stamp_extrainfo[fn] = self.stamp_extrainfo
+
+ provides = [self.pn]
+ for provide in self.provides:
+ if provide not in provides:
+ provides.append(provide)
+ cachedata.fn_provides[fn] = provides
+
+ for provide in provides:
+ cachedata.providers[provide].append(fn)
+ if provide not in cachedata.pn_provides[self.pn]:
+ cachedata.pn_provides[self.pn].append(provide)
+
+ for dep in self.depends:
+ if dep not in cachedata.deps[fn]:
+ cachedata.deps[fn].append(dep)
+ if dep not in cachedata.all_depends:
+ cachedata.all_depends.append(dep)
+
+ rprovides = self.rprovides
+ for package in self.packages:
+ cachedata.packages[package].append(fn)
+ rprovides += self.rprovides_pkg[package]
+
+ for rprovide in rprovides:
+ cachedata.rproviders[rprovide].append(fn)
+
+ for package in self.packages_dynamic:
+ cachedata.packages_dynamic[package].append(fn)
+
+ # Build hash of runtime depends and rececommends
+ for package in self.packages + [self.pn]:
+ cachedata.rundeps[fn][package] = list(self.rdepends) + self.rdepends_pkg[package]
+ cachedata.runrecs[fn][package] = list(self.rrecommends) + self.rrecommends_pkg[package]
+
+ # Collect files we may need for possible world-dep
+ # calculations
+ if not self.broken and not self.not_world:
+ cachedata.possible_world.append(fn)
+
+ # create a collection of all targets for sanity checking
+ # tasks, such as upstream versions, license, and tools for
+ # task and image creation.
+ cachedata.universe_target.append(self.pn)
+
+ cachedata.hashfn[fn] = self.hashfilename
+ for task, taskhash in self.basetaskhashes.iteritems():
+ identifier = '%s.%s' % (fn, task)
+ cachedata.basetaskhash[identifier] = taskhash
+
+ cachedata.inherits[fn] = self.inherits
+ cachedata.summary[fn] = self.summary
+ cachedata.license[fn] = self.license
+ cachedata.section[fn] = self.section
+ cachedata.fakerootenv[fn] = self.fakerootenv
+ cachedata.fakerootdirs[fn] = self.fakerootdirs
+
class Cache(object):
@@ -314,7 +372,7 @@ class Cache(object):
depends |= (data.getVar("__depends", False) or set())
if depends and not variant:
data.setVar("__depends", depends)
- info = RecipeInfo.from_metadata(filename, data)
+ info = CoreRecipeInfo(filename, data)
infos.append((virtualfn, info))
return infos
@@ -500,7 +558,7 @@ class Cache(object):
"""
realfn = self.virtualfn2realfn(file_name)[0]
- info = RecipeInfo.from_metadata(realfn, data)
+ info = CoreRecipeInfo(realfn, data)
self.add_info(file_name, info, cacheData, parsed)
@staticmethod
@@ -566,38 +624,11 @@ class CacheData(object):
"""
def __init__(self):
+ CoreRecipeInfo.init_cacheData(self)
# Direct cache variables
- self.providers = defaultdict(list)
- self.rproviders = defaultdict(list)
- self.packages = defaultdict(list)
- self.packages_dynamic = defaultdict(list)
- self.possible_world = []
- self.universe_target = []
- self.pkg_pn = defaultdict(list)
- self.pkg_fn = {}
- self.pkg_pepvpr = {}
- self.pkg_dp = {}
- self.pn_provides = defaultdict(list)
- self.fn_provides = {}
- self.all_depends = []
- self.deps = defaultdict(list)
- self.rundeps = defaultdict(lambda: defaultdict(list))
- self.runrecs = defaultdict(lambda: defaultdict(list))
self.task_queues = {}
- self.task_deps = {}
- self.stamp = {}
- self.stamp_extrainfo = {}
self.preferred = {}
self.tasks = {}
- self.basetaskhash = {}
- self.hashfn = {}
- self.inherits = {}
- self.summary = {}
- self.license = {}
- self.section = {}
- self.fakerootenv = {}
- self.fakerootdirs = {}
-
# Indirect Cache variables (set elsewhere)
self.ignored_dependencies = []
self.world_target = set()
@@ -605,65 +636,6 @@ class CacheData(object):
self.bbfile_config_priorities = []
def add_from_recipeinfo(self, fn, info):
- self.task_deps[fn] = info.task_deps
- self.pkg_fn[fn] = info.pn
- self.pkg_pn[info.pn].append(fn)
- self.pkg_pepvpr[fn] = (info.pe, info.pv, info.pr)
- self.pkg_dp[fn] = info.defaultpref
- self.stamp[fn] = info.stamp
- self.stamp_extrainfo[fn] = info.stamp_extrainfo
-
- provides = [info.pn]
- for provide in info.provides:
- if provide not in provides:
- provides.append(provide)
- self.fn_provides[fn] = provides
-
- for provide in provides:
- self.providers[provide].append(fn)
- if provide not in self.pn_provides[info.pn]:
- self.pn_provides[info.pn].append(provide)
-
- for dep in info.depends:
- if dep not in self.deps[fn]:
- self.deps[fn].append(dep)
- if dep not in self.all_depends:
- self.all_depends.append(dep)
-
- rprovides = info.rprovides
- for package in info.packages:
- self.packages[package].append(fn)
- rprovides += info.rprovides_pkg[package]
+ info.add_cacheData(self, fn)
- for rprovide in rprovides:
- self.rproviders[rprovide].append(fn)
-
- for package in info.packages_dynamic:
- self.packages_dynamic[package].append(fn)
-
- # Build hash of runtime depends and rececommends
- for package in info.packages + [info.pn]:
- self.rundeps[fn][package] = list(info.rdepends) + info.rdepends_pkg[package]
- self.runrecs[fn][package] = list(info.rrecommends) + info.rrecommends_pkg[package]
-
- # Collect files we may need for possible world-dep
- # calculations
- if not info.broken and not info.not_world:
- self.possible_world.append(fn)
-
- # create a collection of all targets for sanity checking
- # tasks, such as upstream versions, license, and tools for
- # task and image creation.
- self.universe_target.append(info.pn)
-
- self.hashfn[fn] = info.hashfilename
- for task, taskhash in info.basetaskhashes.iteritems():
- identifier = '%s.%s' % (fn, task)
- self.basetaskhash[identifier] = taskhash
-
- self.inherits[fn] = info.inherits
- self.summary[fn] = info.summary
- self.license[fn] = info.license
- self.section[fn] = info.section
- self.fakerootenv[fn] = info.fakerootenv
- self.fakerootdirs[fn] = info.fakerootdirs
+
--
1.7.0.4
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 2/4][Image Creator]Introduce extra cache class for image creator
2011-05-26 2:53 ` Ke Liping
@ 2011-05-26 2:53 ` Ke Liping
-1 siblings, 0 replies; 15+ messages in thread
From: Ke Liping @ 2011-05-26 2:52 UTC (permalink / raw)
To: yocto
From: Liping Ke <liping.ke@intel.com>
Extra RecipeInfo will be all defined in this file. Currently,
Only Hob (Image Creator) Requests some extra fields. So
HobRecipeInfo is defined. It's named HobRecipeInfo because it
is introduced by 'hob'. Users could also introduce other
RecipeInfo or simply use those already defined RecipeInfo.
In the following patch, this newly defined new extra RecipeInfo
will be dynamically loaded and used for loading/saving the extra
cache fields.
Signed-off-by: Liping Ke <liping.ke@intel.com>
---
| 54 +++++++++++++++++++++++++++++++++++++++++
1 files changed, 54 insertions(+), 0 deletions(-)
create mode 100644 bitbake/lib/bb/cache_extra.py
--git a/bitbake/lib/bb/cache_extra.py b/bitbake/lib/bb/cache_extra.py
new file mode 100644
index 0000000..6a402d5
--- /dev/null
+++ b/bitbake/lib/bb/cache_extra.py
@@ -0,0 +1,54 @@
+# ex:ts=4:sw=4:sts=4:et
+# -*- tab-width: 4; c-basic-offset: 4; indent-tabs-mode: nil -*-
+#
+# Extra RecipeInfo will be all defined in this file. Currently,
+# Only Hob (Image Creator) Requests some extra fields. So
+# HobRecipeInfo is defined. It's named HobRecipeInfo because it
+# is introduced by 'hob'. Users could also introduce other
+# RecipeInfo or simply use those already defined RecipeInfo.
+# In the following patch, this newly defined new extra RecipeInfo
+# will be dynamically loaded and used for loading/saving the extra
+# cache fields
+
+# Copyright (C) 2011 Liping Ke
+
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License version 2 as
+# published by the Free Software Foundation.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License along
+# with this program; if not, write to the Free Software Foundation, Inc.,
+# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+
+from bb.cache import RecipeInfoCommon
+
+class HobRecipeInfo(RecipeInfoCommon):
+ __slots__ = ()
+
+ classname = "HobRecipeInfo"
+ # please override this member with the correct data cache file
+ # such as (bb_cache.dat, bb_extracache_hob.dat)
+ cachefile = "bb_extracache_" + classname +".dat"
+
+ def __init__(self, filename, metadata):
+
+ self.summary = self.getvar('SUMMARY', metadata)
+ self.license = self.getvar('LICENSE', metadata)
+ self.section = self.getvar('SECTION', metadata)
+
+ @classmethod
+ def init_cacheData(cls, cachedata):
+ # CacheData in Hob RecipeInfo Class
+ cachedata.summary = {}
+ cachedata.license = {}
+ cachedata.section = {}
+
+ def add_cacheData(self, cachedata, fn):
+ cachedata.summary[fn] = self.summary
+ cachedata.license[fn] = self.license
+ cachedata.section[fn] = self.section
--
1.7.0.4
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 3/4][Image Creator]Introduce new param caches_array into Cache impl.
2011-05-26 2:53 ` Ke Liping
@ 2011-05-26 2:53 ` Ke Liping
-1 siblings, 0 replies; 15+ messages in thread
From: Ke Liping @ 2011-05-26 2:52 UTC (permalink / raw)
To: yocto
From: Liping Ke <liping.ke@intel.com>
When using hob ui interface, we need extra cache fields.
We will save ui required extra cache fields into a separate
cache file. This patch introduce this caches_array parameter.
It will be used in the extra cache implementation (following patch).
Caches_array at least contains CoreRecipeInfo. If users need extra
cache fields support, such as 'hob', caches_array will contain
more relevant elements such as HobRecipeInfo.
Signed-off-by: Liping Ke <liping.ke@intel.com>
---
bitbake/lib/bb/cache.py | 15 ++++++++----
bitbake/lib/bb/cooker.py | 54 +++++++++++++++++++++++++++++++++++++++------
bitbake/lib/bb/shell.py | 2 +-
bitbake/lib/bb/ui/hob.py | 2 +
4 files changed, 59 insertions(+), 14 deletions(-)
diff --git a/bitbake/lib/bb/cache.py b/bitbake/lib/bb/cache.py
index 09691d9..0620621 100644
--- a/bitbake/lib/bb/cache.py
+++ b/bitbake/lib/bb/cache.py
@@ -245,7 +245,11 @@ class Cache(object):
BitBake Cache implementation
"""
- def __init__(self, data):
+ def __init__(self, data, caches_array):
+ # Pass caches_array information into Cache Constructor
+ # It will be used in later for deciding whether we
+ # need extra cache file dump/load support
+ self.caches_array = caches_array
self.cachedir = bb.data.getVar("CACHE", data, True)
self.clean = set()
self.checked = set()
@@ -360,7 +364,7 @@ class Cache(object):
return bb_data[virtual]
@classmethod
- def parse(cls, filename, appends, configdata):
+ def parse(cls, filename, appends, configdata, caches_array):
"""Parse the specified filename, returning the recipe information"""
infos = []
datastores = cls.load_bbfile(filename, appends, configdata)
@@ -393,7 +397,7 @@ class Cache(object):
infos.append((virtualfn, self.depends_cache[virtualfn]))
else:
logger.debug(1, "Parsing %s", filename)
- return self.parse(filename, appends, configdata)
+ return self.parse(filename, appends, configdata, self.caches_array)
return cached, infos
@@ -623,8 +627,9 @@ class CacheData(object):
The data structures we compile from the cached data
"""
- def __init__(self):
- CoreRecipeInfo.init_cacheData(self)
+ def __init__(self, caches_array):
+ self.caches_array = caches_array
+ CoreRecipeInfo.init_cacheData(self)
# Direct cache variables
self.task_queues = {}
self.preferred = {}
diff --git a/bitbake/lib/bb/cooker.py b/bitbake/lib/bb/cooker.py
index a1cd4d7..0dc895a 100644
--- a/bitbake/lib/bb/cooker.py
+++ b/bitbake/lib/bb/cooker.py
@@ -72,6 +72,43 @@ class BBCooker:
self.configuration = configuration
+ self.caches_array = []
+ # Currently, only Image Creator hob ui needs extra cache.
+ # So, we save Extra Cache class name and container file
+ # information into a extraCaches field in hob UI.
+ # In future, having a registration mechanism for extra cache
+ # fields in cache_extra is a better solution. Also, we may
+ # need to consider adding a user-hidden parameter 'CacheRequest'
+ # for bitbake command line. It will be filled by those sub
+ # command who need to have extra cache support.
+ caches_name_array = ['bb.cache:CoreRecipeInfo']
+ if configuration.ui:
+ try:
+ module = __import__('bb.ui', fromlist=[configuration.ui])
+ name_array = (getattr(module, configuration.ui)).extraCaches
+ for recipeInfoName in name_array:
+ caches_name_array.append(recipeInfoName)
+ except ImportError, exc:
+ # bb.ui.XXX is not defined and imported. It's an error!
+ logger.critical("Unable to import '%s' interface from bb.ui: %s" % (configuration.ui, exc))
+ sys.exit("FATAL: Failed to import '%s' interface." % configuration.ui)
+ except AttributeError:
+ # This is not an error. If the field is not defined in the ui,
+ # this interface might need no extra cache fields, so
+ # just skip this error!
+ logger.info("UI '%s' does not require extra cache!" % (configuration.ui))
+
+ # At least CoreRecipeInfo will be loaded, so caches_array will never be empty!
+ # This is the entry point, no further check needed!
+ for var in caches_name_array:
+ try:
+ module_name, cache_name = var.split(':')
+ module = __import__(module_name, fromlist=(cache_name,))
+ self.caches_array.append(getattr(module, cache_name))
+ except ImportError, exc:
+ logger.critical("Unable to import extra RecipeInfo '%s' from bb.extra_cache: %s" % (cache_name, exc))
+ sys.exit("FATAL: Failed to import extra cache class '%s'." % cache_name)
+
self.configuration.data = bb.data.init()
if not server:
@@ -713,9 +750,10 @@ class BBCooker:
self.buildSetVars()
- self.status = bb.cache.CacheData()
+ self.status = bb.cache.CacheData(self.caches_array)
infos = bb.cache.Cache.parse(fn, self.get_file_appends(fn), \
- self.configuration.data)
+ self.configuration.data,
+ self.caches_array)
infos = dict(infos)
fn = bb.cache.Cache.realfn2virtual(buildfile, cls)
@@ -859,7 +897,7 @@ class BBCooker:
else:
collectlog.info("You have disabled Psyco. This decreases performance.")
- self.status = bb.cache.CacheData()
+ self.status = bb.cache.CacheData(self.caches_array)
ignore = bb.data.getVar("ASSUME_PROVIDED", self.configuration.data, 1) or ""
self.status.ignored_dependencies = set(ignore.split())
@@ -1066,9 +1104,9 @@ class ParsingFailure(Exception):
self.args = (realexception, recipe)
def parse_file(task):
- filename, appends = task
+ filename, appends, caches_array = task
try:
- return True, bb.cache.Cache.parse(filename, appends, parse_file.cfg)
+ return True, bb.cache.Cache.parse(filename, appends, parse_file.cfg, caches_array)
except Exception, exc:
exc.recipe = filename
raise exc
@@ -1098,13 +1136,13 @@ class CookerParser(object):
self.num_processes = int(self.cfgdata.getVar("BB_NUMBER_PARSE_THREADS", True) or
multiprocessing.cpu_count())
- self.bb_cache = bb.cache.Cache(self.cfgdata)
+ self.bb_cache = bb.cache.Cache(self.cfgdata, cooker.caches_array)
self.fromcache = []
self.willparse = []
for filename in self.filelist:
appends = self.cooker.get_file_appends(filename)
if not self.bb_cache.cacheValid(filename):
- self.willparse.append((filename, appends))
+ self.willparse.append((filename, appends, cooker.caches_array))
else:
self.fromcache.append((filename, appends))
self.toparse = self.total - len(self.fromcache)
@@ -1179,6 +1217,6 @@ class CookerParser(object):
def reparse(self, filename):
infos = self.bb_cache.parse(filename,
self.cooker.get_file_appends(filename),
- self.cfgdata)
+ self.cfgdata, self.cooker.caches_array)
for vfn, info in infos:
self.cooker.status.add_from_recipeinfo(vfn, info)
diff --git a/bitbake/lib/bb/shell.py b/bitbake/lib/bb/shell.py
index 3319e2d..1dd8d54 100644
--- a/bitbake/lib/bb/shell.py
+++ b/bitbake/lib/bb/shell.py
@@ -407,7 +407,7 @@ SRC_URI = ""
def parse( self, params ):
"""(Re-)parse .bb files and calculate the dependency graph"""
- cooker.status = cache.CacheData()
+ cooker.status = cache.CacheData(cooker.caches_array)
ignore = data.getVar("ASSUME_PROVIDED", cooker.configuration.data, 1) or ""
cooker.status.ignored_dependencies = set( ignore.split() )
cooker.handleCollections( data.getVar("BBFILE_COLLECTIONS", cooker.configuration.data, 1) )
diff --git a/bitbake/lib/bb/ui/hob.py b/bitbake/lib/bb/ui/hob.py
index 0f8fe8c..ab6022b 100644
--- a/bitbake/lib/bb/ui/hob.py
+++ b/bitbake/lib/bb/ui/hob.py
@@ -28,6 +28,8 @@ import xmlrpclib
import logging
import Queue
+extraCaches = ['bb.cache_extra:HobRecipeInfo']
+
class MainWindow (gtk.Window):
def __init__(self, taskmodel, handler, curr_mach=None, curr_distro=None):
--
1.7.0.4
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 4/4][Image Creator]Implement multiple extra cache fields request support
2011-05-26 2:53 ` Ke Liping
@ 2011-05-26 2:53 ` Ke Liping
-1 siblings, 0 replies; 15+ messages in thread
From: Ke Liping @ 2011-05-26 2:52 UTC (permalink / raw)
To: yocto
From: Liping Ke <liping.ke@intel.com>
This patch is to support extra cache. If user needs to request
extra cache fields besides CoreRecipeInfo fields, just add a new
XXXRecipeInfo class definition as Hob Does.
Currently supported Extra RecipeInfo name is an array. We can
support multiple extra cache fields at the same time besides
CoreRecipeInfo which is needed by all clients.
Signed-off-by: Liping Ke <liping.ke@intel.com>
---
bitbake/lib/bb/cache.py | 175 +++++++++++++++++++++++++++++++---------------
bitbake/lib/bb/cooker.py | 16 ++--
2 files changed, 126 insertions(+), 65 deletions(-)
diff --git a/bitbake/lib/bb/cache.py b/bitbake/lib/bb/cache.py
index 0620621..271ebd7 100644
--- a/bitbake/lib/bb/cache.py
+++ b/bitbake/lib/bb/cache.py
@@ -45,6 +45,8 @@ except ImportError:
__cache_version__ = "138"
+def getCacheFile(path, filename):
+ return os.path.join(path, filename)
# RecipeInfoCommon defines common data retrieving methods
# from meta data for caches. CoreRecipeInfo as well as other
@@ -86,12 +88,9 @@ class RecipeInfoCommon(object):
class CoreRecipeInfo(RecipeInfoCommon):
__slots__ = ()
- def __init__(self, filename, metadata):
- self.name = "base"
- # please override this member with the correct data cache file
- # such as (bb_cache.dat, bb_extracache_hob.dat)
- self.cachefile = "bb_cache.dat"
+ cachefile = "bb_cache.dat"
+ def __init__(self, filename, metadata):
self.file_depends = metadata.getVar('__depends', False)
self.timestamp = bb.parse.cached_mtime(filename)
self.variants = self.listvar('__VARIANTS', metadata) + ['']
@@ -265,7 +264,7 @@ class Cache(object):
return
self.has_cache = True
- self.cachefile = os.path.join(self.cachedir, "bb_cache.dat")
+ self.cachefile = getCacheFile(self.cachedir, "bb_cache.dat")
logger.debug(1, "Using cache in '%s'", self.cachedir)
bb.utils.mkdirhier(self.cachedir)
@@ -279,12 +278,21 @@ class Cache(object):
old_mtimes.append(newest_mtime)
newest_mtime = max(old_mtimes)
- if bb.parse.cached_mtime_noerror(self.cachefile) >= newest_mtime:
+ bNeedUpdate = True
+ if self.caches_array:
+ for cache_class in self.caches_array:
+ if type(cache_class) is type and issubclass(cache_class, RecipeInfoCommon):
+ cachefile = getCacheFile(self.cachedir, cache_class.cachefile)
+ bNeedUpdate = bNeedUpdate and (bb.parse.cached_mtime_noerror(cachefile) >= newest_mtime)
+ cache_class.init_cacheData(self)
+ if bNeedUpdate:
self.load_cachefile()
elif os.path.isfile(self.cachefile):
logger.info("Out of date cache found, rebuilding...")
def load_cachefile(self):
+ # Firstly, using core cache file information for
+ # valid checking
with open(self.cachefile, "rb") as cachefile:
pickled = pickle.Unpickler(cachefile)
try:
@@ -301,31 +309,52 @@ class Cache(object):
logger.info('Bitbake version mismatch, rebuilding...')
return
- cachesize = os.fstat(cachefile.fileno()).st_size
- bb.event.fire(bb.event.CacheLoadStarted(cachesize), self.data)
- previous_percent = 0
- while cachefile:
- try:
- key = pickled.load()
- value = pickled.load()
- except Exception:
- break
+ cachesize = 0
+ previous_progress = 0
+ previous_percent = 0
- self.depends_cache[key] = value
-
- # only fire events on even percentage boundaries
- current_progress = cachefile.tell()
- current_percent = 100 * current_progress / cachesize
- if current_percent > previous_percent:
- previous_percent = current_percent
- bb.event.fire(bb.event.CacheLoadProgress(current_progress),
- self.data)
-
- bb.event.fire(bb.event.CacheLoadCompleted(cachesize,
- len(self.depends_cache)),
- self.data)
+ # Calculate the correct cachesize of all those cache files
+ for cache_class in self.caches_array:
+ if type(cache_class) is type and issubclass(cache_class, RecipeInfoCommon):
+ cachefile = getCacheFile(self.cachedir, cache_class.cachefile)
+ with open(cachefile, "rb") as cachefile:
+ cachesize += os.fstat(cachefile.fileno()).st_size
+ bb.event.fire(bb.event.CacheLoadStarted(cachesize), self.data)
+
+ for cache_class in self.caches_array:
+ if type(cache_class) is type and issubclass(cache_class, RecipeInfoCommon):
+ cachefile = getCacheFile(self.cachedir, cache_class.cachefile)
+ with open(cachefile, "rb") as cachefile:
+ pickled = pickle.Unpickler(cachefile)
+ while cachefile:
+ try:
+ key = pickled.load()
+ value = pickled.load()
+ except Exception:
+ break
+ if self.depends_cache.has_key(key):
+ self.depends_cache[key].append(value)
+ else:
+ self.depends_cache[key] = [value]
+ # only fire events on even percentage boundaries
+ current_progress = cachefile.tell() + previous_progress
+ current_percent = 100 * current_progress / cachesize
+ if current_percent > previous_percent:
+ previous_percent = current_percent
+ bb.event.fire(bb.event.CacheLoadProgress(current_progress),
+ self.data)
+
+ previous_progress += current_progress
+
+ # Note: depends cache number is corresponding to the parsing file numbers.
+ # The same file has several caches, still regarded as one item in the cache
+ bb.event.fire(bb.event.CacheLoadCompleted(cachesize,
+ len(self.depends_cache)),
+ self.data)
+
+
@staticmethod
def virtualfn2realfn(virtualfn):
"""
@@ -376,8 +405,14 @@ class Cache(object):
depends |= (data.getVar("__depends", False) or set())
if depends and not variant:
data.setVar("__depends", depends)
- info = CoreRecipeInfo(filename, data)
- infos.append((virtualfn, info))
+
+ info_array = []
+ for cache_class in caches_array:
+ if type(cache_class) is type and issubclass(cache_class, RecipeInfoCommon):
+ info = cache_class(filename, data)
+ info_array.append(info)
+ infos.append((virtualfn, info_array))
+
return infos
def load(self, filename, appends, configdata):
@@ -391,8 +426,9 @@ class Cache(object):
cached = self.cacheValid(filename)
if cached:
infos = []
- info = self.depends_cache[filename]
- for variant in info.variants:
+ # info_array item is a list of [CoreRecipeInfo, XXXRecipeInfo]
+ info_array = self.depends_cache[filename]
+ for variant in info_array[0].variants:
virtualfn = self.realfn2virtual(filename, variant)
infos.append((virtualfn, self.depends_cache[virtualfn]))
else:
@@ -408,12 +444,12 @@ class Cache(object):
skipped, virtuals = 0, 0
cached, infos = self.load(fn, appends, cfgData)
- for virtualfn, info in infos:
- if info.skipped:
+ for virtualfn, info_array in infos:
+ if info_array[0].skipped:
logger.debug(1, "Skipping %s", virtualfn)
skipped += 1
else:
- self.add_info(virtualfn, info, cacheData, not cached)
+ self.add_info(virtualfn, info_array, cacheData, not cached)
virtuals += 1
return cached, skipped, virtuals
@@ -457,15 +493,15 @@ class Cache(object):
self.remove(fn)
return False
- info = self.depends_cache[fn]
+ info_array = self.depends_cache[fn]
# Check the file's timestamp
- if mtime != info.timestamp:
+ if mtime != info_array[0].timestamp:
logger.debug(2, "Cache: %s changed", fn)
self.remove(fn)
return False
# Check dependencies are still valid
- depends = info.file_depends
+ depends = info_array[0].file_depends
if depends:
for f, old_mtime in depends:
fmtime = bb.parse.cached_mtime_noerror(f)
@@ -483,7 +519,7 @@ class Cache(object):
return False
invalid = False
- for cls in info.variants:
+ for cls in info_array[0].variants:
virtualfn = self.realfn2virtual(fn, cls)
self.clean.add(virtualfn)
if virtualfn not in self.depends_cache:
@@ -530,13 +566,30 @@ class Cache(object):
logger.debug(2, "Cache is clean, not saving.")
return
- with open(self.cachefile, "wb") as cachefile:
- pickler = pickle.Pickler(cachefile, pickle.HIGHEST_PROTOCOL)
- pickler.dump(__cache_version__)
- pickler.dump(bb.__version__)
- for key, value in self.depends_cache.iteritems():
- pickler.dump(key)
- pickler.dump(value)
+ file_dict = {}
+ pickler_dict = {}
+ for cache_class in self.caches_array:
+ if type(cache_class) is type and issubclass(cache_class, RecipeInfoCommon):
+ cache_class_name = cache_class.__name__
+ cachefile = getCacheFile(self.cachedir, cache_class.cachefile)
+ file_dict[cache_class_name] = open(cachefile, "wb")
+ pickler_dict[cache_class_name] = pickle.Pickler(file_dict[cache_class_name], pickle.HIGHEST_PROTOCOL)
+
+ pickler_dict['CoreRecipeInfo'].dump(__cache_version__)
+ pickler_dict['CoreRecipeInfo'].dump(bb.__version__)
+
+ try:
+ for key, info_array in self.depends_cache.iteritems():
+ for info in info_array:
+ if isinstance(info, RecipeInfoCommon):
+ cache_class_name = info.__class__.__name__
+ pickler_dict[cache_class_name].dump(key)
+ pickler_dict[cache_class_name].dump(info)
+ finally:
+ for cache_class in self.caches_array:
+ if type(cache_class) is type and issubclass(cache_class, RecipeInfoCommon):
+ cache_class_name = cache_class.__name__
+ file_dict[cache_class_name].close()
del self.depends_cache
@@ -544,17 +597,17 @@ class Cache(object):
def mtime(cachefile):
return bb.parse.cached_mtime_noerror(cachefile)
- def add_info(self, filename, info, cacheData, parsed=None):
- if not info.skipped:
- cacheData.add_from_recipeinfo(filename, info)
+ def add_info(self, filename, info_array, cacheData, parsed=None):
+ if isinstance(info_array[0], CoreRecipeInfo) and (not info_array[0].skipped):
+ cacheData.add_from_recipeinfo(filename, info_array)
if not self.has_cache:
return
- if (info.skipped or 'SRCREVINACTION' not in info.pv) and not info.nocache:
+ if (info_array[0].skipped or 'SRCREVINACTION' not in info_array[0].pv) and not info_array[0].nocache:
if parsed:
self.cacheclean = False
- self.depends_cache[filename] = info
+ self.depends_cache[filename] = info_array
def add(self, file_name, data, cacheData, parsed=None):
"""
@@ -562,8 +615,12 @@ class Cache(object):
"""
realfn = self.virtualfn2realfn(file_name)[0]
- info = CoreRecipeInfo(realfn, data)
- self.add_info(file_name, info, cacheData, parsed)
+
+ info_array = []
+ for cache_class in self.caches_array:
+ if type(cache_class) is type and issubclass(cache_class, RecipeInfoCommon):
+ info_array.append(cache_class(realfn, data))
+ self.add_info(file_name, info_array, cacheData, parsed)
@staticmethod
def load_bbfile(bbfile, appends, config):
@@ -629,7 +686,10 @@ class CacheData(object):
def __init__(self, caches_array):
self.caches_array = caches_array
- CoreRecipeInfo.init_cacheData(self)
+ for cache_class in self.caches_array:
+ if type(cache_class) is type and issubclass(cache_class, RecipeInfoCommon):
+ cache_class.init_cacheData(self)
+
# Direct cache variables
self.task_queues = {}
self.preferred = {}
@@ -640,7 +700,8 @@ class CacheData(object):
self.bbfile_priority = {}
self.bbfile_config_priorities = []
- def add_from_recipeinfo(self, fn, info):
- info.add_cacheData(self, fn)
+ def add_from_recipeinfo(self, fn, info_array):
+ for info in info_array:
+ info.add_cacheData(self, fn)
diff --git a/bitbake/lib/bb/cooker.py b/bitbake/lib/bb/cooker.py
index 0dc895a..b76a4d2 100644
--- a/bitbake/lib/bb/cooker.py
+++ b/bitbake/lib/bb/cooker.py
@@ -758,13 +758,13 @@ class BBCooker:
fn = bb.cache.Cache.realfn2virtual(buildfile, cls)
try:
- maininfo = infos[fn]
+ info_array = infos[fn]
except KeyError:
bb.fatal("%s does not exist" % fn)
- self.status.add_from_recipeinfo(fn, maininfo)
+ self.status.add_from_recipeinfo(fn, info_array)
# Tweak some variables
- item = maininfo.pn
+ item = info_array[0].pn
self.status.ignored_dependencies = set()
self.status.bbfile_priority[fn] = 1
@@ -1207,10 +1207,10 @@ class CookerParser(object):
else:
self.cached += 1
- for virtualfn, info in result:
- if info.skipped:
+ for virtualfn, info_array in result:
+ if info_array[0].skipped:
self.skipped += 1
- self.bb_cache.add_info(virtualfn, info, self.cooker.status,
+ self.bb_cache.add_info(virtualfn, info_array, self.cooker.status,
parsed=parsed)
return True
@@ -1218,5 +1218,5 @@ class CookerParser(object):
infos = self.bb_cache.parse(filename,
self.cooker.get_file_appends(filename),
self.cfgdata, self.cooker.caches_array)
- for vfn, info in infos:
- self.cooker.status.add_from_recipeinfo(vfn, info)
+ for vfn, info_array in infos:
+ self.cooker.status.add_from_recipeinfo(vfn, info_array)
--
1.7.0.4
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 0/4][Image Creator]Add mulitple extra cache fields request support
@ 2011-05-26 2:53 ` Ke Liping
0 siblings, 0 replies; 15+ messages in thread
From: Ke Liping @ 2011-05-26 2:53 UTC (permalink / raw)
To: poky
From: Liping Ke <liping.ke@intel.com>
The series of patch are for mplementing multiple extra cache fields
request support.If user needs to request extra cache fields besides
CoreRecipeInfo fields, just add a new XXXRecipeInfo class definition
as Image Creator UI Hob Does.
For implementing this new feature, we refactory current cache code,
remove namedtuple and define corresponding class for dynamical cache
fields support per different cache usage request.
Liping Ke (4):
Refactory Current Cache implementation
Introduce extra cache class for image creator
Introduce new param caches_array into Cache impl.
Implement multiple extra cache fields request support
bitbake/lib/bb/cache.py | 532 ++++++++++++++++++++++-------------------
bitbake/lib/bb/cache_extra.py | 54 ++++
bitbake/lib/bb/cooker.py | 70 ++++--
bitbake/lib/bb/shell.py | 2 +-
bitbake/lib/bb/ui/hob.py | 2 +
5 files changed, 396 insertions(+), 264 deletions(-)
create mode 100644 bitbake/lib/bb/cache_extra.py
^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH 1/4][Image Creator]Refactory Current Cache implementation
@ 2011-05-26 2:53 ` Ke Liping
0 siblings, 0 replies; 15+ messages in thread
From: Ke Liping @ 2011-05-26 2:53 UTC (permalink / raw)
To: poky
From: Liping Ke <liping.ke@intel.com>
This patch is for refactorying current cache implementation, the
main reason is for introducing extra cache fields requests for
image creator as well as other users. The refactory parts include:
Move cache data retrieve methods out of Cache Data Fields
Definition. Since this retrieve methods will be shared for
both CoreRecipeInfo as well as the new introduced extra RecipeInfo
in the following patches.
Signed-off-by: Liping Ke <liping.ke@intel.com>
---
bitbake/lib/bb/cache.py | 358 ++++++++++++++++++++++-------------------------
1 files changed, 165 insertions(+), 193 deletions(-)
diff --git a/bitbake/lib/bb/cache.py b/bitbake/lib/bb/cache.py
index d083c51..09691d9 100644
--- a/bitbake/lib/bb/cache.py
+++ b/bitbake/lib/bb/cache.py
@@ -30,7 +30,7 @@
import os
import logging
-from collections import defaultdict, namedtuple
+from collections import defaultdict
import bb.data
import bb.utils
@@ -45,46 +45,11 @@ except ImportError:
__cache_version__ = "138"
-recipe_fields = (
- 'pn',
- 'pv',
- 'pr',
- 'pe',
- 'defaultpref',
- 'depends',
- 'provides',
- 'task_deps',
- 'stamp',
- 'stamp_extrainfo',
- 'broken',
- 'not_world',
- 'skipped',
- 'timestamp',
- 'packages',
- 'packages_dynamic',
- 'rdepends',
- 'rdepends_pkg',
- 'rprovides',
- 'rprovides_pkg',
- 'rrecommends',
- 'rrecommends_pkg',
- 'nocache',
- 'variants',
- 'file_depends',
- 'tasks',
- 'basetaskhashes',
- 'hashfilename',
- 'inherits',
- 'summary',
- 'license',
- 'section',
- 'fakerootenv',
- 'fakerootdirs'
-)
-
-
-class RecipeInfo(namedtuple('RecipeInfo', recipe_fields)):
- __slots__ = ()
+
+# RecipeInfoCommon defines common data retrieving methods
+# from meta data for caches. CoreRecipeInfo as well as other
+# Extra RecipeInfo needs to inherit this class
+class RecipeInfoCommon(object):
@classmethod
def listvar(cls, var, metadata):
@@ -117,69 +82,162 @@ class RecipeInfo(namedtuple('RecipeInfo', recipe_fields)):
def getvar(cls, var, metadata):
return metadata.getVar(var, True) or ''
- @classmethod
- def make_optional(cls, default=None, **kwargs):
- """Construct the namedtuple from the specified keyword arguments,
- with every value considered optional, using the default value if
- it was not specified."""
- for field in cls._fields:
- kwargs[field] = kwargs.get(field, default)
- return cls(**kwargs)
+
+class CoreRecipeInfo(RecipeInfoCommon):
+ __slots__ = ()
+
+ def __init__(self, filename, metadata):
+ self.name = "base"
+ # please override this member with the correct data cache file
+ # such as (bb_cache.dat, bb_extracache_hob.dat)
+ self.cachefile = "bb_cache.dat"
+
+ self.file_depends = metadata.getVar('__depends', False)
+ self.timestamp = bb.parse.cached_mtime(filename)
+ self.variants = self.listvar('__VARIANTS', metadata) + ['']
+ self.nocache = self.getvar('__BB_DONT_CACHE', metadata)
+
+ if self.getvar('__SKIPPED', metadata):
+ self.skipped = True
+ return
+
+ self.tasks = metadata.getVar('__BBTASKS', False)
+
+ self.pn = self.getvar('PN', metadata)
+ self.packages = self.listvar('PACKAGES', metadata)
+ if not self.pn in self.packages:
+ self.packages.append(self.pn)
+
+ self.basetaskhashes = self.taskvar('BB_BASEHASH', self.tasks, metadata)
+ self.hashfilename = self.getvar('BB_HASHFILENAME', metadata)
+
+ self.file_depends = metadata.getVar('__depends', False)
+ self.task_deps = metadata.getVar('_task_deps', False) or {'tasks': [], 'parents': {}}
+
+ self.skipped = False
+ self.pe = self.getvar('PE', metadata)
+ self.pv = self.getvar('PV', metadata)
+ self.pr = self.getvar('PR', metadata)
+ self.defaultpref = self.intvar('DEFAULT_PREFERENCE', metadata)
+ self.broken = self.getvar('BROKEN', metadata)
+ self.not_world = self.getvar('EXCLUDE_FROM_WORLD', metadata)
+ self.stamp = self.getvar('STAMP', metadata)
+ self.stamp_extrainfo = self.flaglist('stamp-extra-info', self.tasks, metadata)
+ self.packages_dynamic = self.listvar('PACKAGES_DYNAMIC', metadata)
+ self.depends = self.depvar('DEPENDS', metadata)
+ self.provides = self.depvar('PROVIDES', metadata)
+ self.rdepends = self.depvar('RDEPENDS', metadata)
+ self.rprovides = self.depvar('RPROVIDES', metadata)
+ self.rrecommends = self.depvar('RRECOMMENDS', metadata)
+ self.rprovides_pkg = self.pkgvar('RPROVIDES', self.packages, metadata)
+ self.rdepends_pkg = self.pkgvar('RDEPENDS', self.packages, metadata)
+ self.rrecommends_pkg = self.pkgvar('RRECOMMENDS', self.packages, metadata)
+ self.inherits = self.getvar('__inherit_cache', metadata)
+ self.summary = self.getvar('SUMMARY', metadata)
+ self.license = self.getvar('LICENSE', metadata)
+ self.section = self.getvar('SECTION', metadata)
+ self.fakerootenv = self.getvar('FAKEROOTENV', metadata)
+ self.fakerootdirs = self.getvar('FAKEROOTDIRS', metadata)
@classmethod
- def from_metadata(cls, filename, metadata):
- if cls.getvar('__SKIPPED', metadata):
- return cls.make_optional(skipped=True,
- file_depends=metadata.getVar('__depends', False),
- timestamp=bb.parse.cached_mtime(filename),
- variants=cls.listvar('__VARIANTS', metadata) + [''])
-
- tasks = metadata.getVar('__BBTASKS', False)
-
- pn = cls.getvar('PN', metadata)
- packages = cls.listvar('PACKAGES', metadata)
- if not pn in packages:
- packages.append(pn)
-
- return RecipeInfo(
- tasks = tasks,
- basetaskhashes = cls.taskvar('BB_BASEHASH', tasks, metadata),
- hashfilename = cls.getvar('BB_HASHFILENAME', metadata),
-
- file_depends = metadata.getVar('__depends', False),
- task_deps = metadata.getVar('_task_deps', False) or
- {'tasks': [], 'parents': {}},
- variants = cls.listvar('__VARIANTS', metadata) + [''],
-
- skipped = False,
- timestamp = bb.parse.cached_mtime(filename),
- packages = cls.listvar('PACKAGES', metadata),
- pn = pn,
- pe = cls.getvar('PE', metadata),
- pv = cls.getvar('PV', metadata),
- pr = cls.getvar('PR', metadata),
- nocache = cls.getvar('__BB_DONT_CACHE', metadata),
- defaultpref = cls.intvar('DEFAULT_PREFERENCE', metadata),
- broken = cls.getvar('BROKEN', metadata),
- not_world = cls.getvar('EXCLUDE_FROM_WORLD', metadata),
- stamp = cls.getvar('STAMP', metadata),
- stamp_extrainfo = cls.flaglist('stamp-extra-info', tasks, metadata),
- packages_dynamic = cls.listvar('PACKAGES_DYNAMIC', metadata),
- depends = cls.depvar('DEPENDS', metadata),
- provides = cls.depvar('PROVIDES', metadata),
- rdepends = cls.depvar('RDEPENDS', metadata),
- rprovides = cls.depvar('RPROVIDES', metadata),
- rrecommends = cls.depvar('RRECOMMENDS', metadata),
- rprovides_pkg = cls.pkgvar('RPROVIDES', packages, metadata),
- rdepends_pkg = cls.pkgvar('RDEPENDS', packages, metadata),
- rrecommends_pkg = cls.pkgvar('RRECOMMENDS', packages, metadata),
- inherits = cls.getvar('__inherit_cache', metadata),
- summary = cls.getvar('SUMMARY', metadata),
- license = cls.getvar('LICENSE', metadata),
- section = cls.getvar('SECTION', metadata),
- fakerootenv = cls.getvar('FAKEROOTENV', metadata),
- fakerootdirs = cls.getvar('FAKEROOTDIRS', metadata),
- )
+ def init_cacheData(cls, cachedata):
+ # CacheData in Core RecipeInfo Class
+ cachedata.task_deps = {}
+ cachedata.pkg_fn = {}
+ cachedata.pkg_pn = defaultdict(list)
+ cachedata.pkg_pepvpr = {}
+ cachedata.pkg_dp = {}
+
+ cachedata.stamp = {}
+ cachedata.stamp_extrainfo = {}
+ cachedata.fn_provides = {}
+ cachedata.pn_provides = defaultdict(list)
+ cachedata.all_depends = []
+
+ cachedata.deps = defaultdict(list)
+ cachedata.packages = defaultdict(list)
+ cachedata.providers = defaultdict(list)
+ cachedata.rproviders = defaultdict(list)
+ cachedata.packages_dynamic = defaultdict(list)
+
+ cachedata.rundeps = defaultdict(lambda: defaultdict(list))
+ cachedata.runrecs = defaultdict(lambda: defaultdict(list))
+ cachedata.possible_world = []
+ cachedata.universe_target = []
+ cachedata.hashfn = {}
+
+ cachedata.basetaskhash = {}
+ cachedata.inherits = {}
+ cachedata.summary = {}
+ cachedata.license = {}
+ cachedata.section = {}
+ cachedata.fakerootenv = {}
+ cachedata.fakerootdirs = {}
+
+ def add_cacheData(self, cachedata, fn):
+ cachedata.task_deps[fn] = self.task_deps
+ cachedata.pkg_fn[fn] = self.pn
+ cachedata.pkg_pn[self].append(fn)
+ cachedata.pkg_pepvpr[fn] = (self.pe, self.pv, self.pr)
+ cachedata.pkg_dp[fn] = self.defaultpref
+ cachedata.stamp[fn] = self.stamp
+ cachedata.stamp_extrainfo[fn] = self.stamp_extrainfo
+
+ provides = [self.pn]
+ for provide in self.provides:
+ if provide not in provides:
+ provides.append(provide)
+ cachedata.fn_provides[fn] = provides
+
+ for provide in provides:
+ cachedata.providers[provide].append(fn)
+ if provide not in cachedata.pn_provides[self.pn]:
+ cachedata.pn_provides[self.pn].append(provide)
+
+ for dep in self.depends:
+ if dep not in cachedata.deps[fn]:
+ cachedata.deps[fn].append(dep)
+ if dep not in cachedata.all_depends:
+ cachedata.all_depends.append(dep)
+
+ rprovides = self.rprovides
+ for package in self.packages:
+ cachedata.packages[package].append(fn)
+ rprovides += self.rprovides_pkg[package]
+
+ for rprovide in rprovides:
+ cachedata.rproviders[rprovide].append(fn)
+
+ for package in self.packages_dynamic:
+ cachedata.packages_dynamic[package].append(fn)
+
+ # Build hash of runtime depends and rececommends
+ for package in self.packages + [self.pn]:
+ cachedata.rundeps[fn][package] = list(self.rdepends) + self.rdepends_pkg[package]
+ cachedata.runrecs[fn][package] = list(self.rrecommends) + self.rrecommends_pkg[package]
+
+ # Collect files we may need for possible world-dep
+ # calculations
+ if not self.broken and not self.not_world:
+ cachedata.possible_world.append(fn)
+
+ # create a collection of all targets for sanity checking
+ # tasks, such as upstream versions, license, and tools for
+ # task and image creation.
+ cachedata.universe_target.append(self.pn)
+
+ cachedata.hashfn[fn] = self.hashfilename
+ for task, taskhash in self.basetaskhashes.iteritems():
+ identifier = '%s.%s' % (fn, task)
+ cachedata.basetaskhash[identifier] = taskhash
+
+ cachedata.inherits[fn] = self.inherits
+ cachedata.summary[fn] = self.summary
+ cachedata.license[fn] = self.license
+ cachedata.section[fn] = self.section
+ cachedata.fakerootenv[fn] = self.fakerootenv
+ cachedata.fakerootdirs[fn] = self.fakerootdirs
+
class Cache(object):
@@ -314,7 +372,7 @@ class Cache(object):
depends |= (data.getVar("__depends", False) or set())
if depends and not variant:
data.setVar("__depends", depends)
- info = RecipeInfo.from_metadata(filename, data)
+ info = CoreRecipeInfo(filename, data)
infos.append((virtualfn, info))
return infos
@@ -500,7 +558,7 @@ class Cache(object):
"""
realfn = self.virtualfn2realfn(file_name)[0]
- info = RecipeInfo.from_metadata(realfn, data)
+ info = CoreRecipeInfo(realfn, data)
self.add_info(file_name, info, cacheData, parsed)
@staticmethod
@@ -566,38 +624,11 @@ class CacheData(object):
"""
def __init__(self):
+ CoreRecipeInfo.init_cacheData(self)
# Direct cache variables
- self.providers = defaultdict(list)
- self.rproviders = defaultdict(list)
- self.packages = defaultdict(list)
- self.packages_dynamic = defaultdict(list)
- self.possible_world = []
- self.universe_target = []
- self.pkg_pn = defaultdict(list)
- self.pkg_fn = {}
- self.pkg_pepvpr = {}
- self.pkg_dp = {}
- self.pn_provides = defaultdict(list)
- self.fn_provides = {}
- self.all_depends = []
- self.deps = defaultdict(list)
- self.rundeps = defaultdict(lambda: defaultdict(list))
- self.runrecs = defaultdict(lambda: defaultdict(list))
self.task_queues = {}
- self.task_deps = {}
- self.stamp = {}
- self.stamp_extrainfo = {}
self.preferred = {}
self.tasks = {}
- self.basetaskhash = {}
- self.hashfn = {}
- self.inherits = {}
- self.summary = {}
- self.license = {}
- self.section = {}
- self.fakerootenv = {}
- self.fakerootdirs = {}
-
# Indirect Cache variables (set elsewhere)
self.ignored_dependencies = []
self.world_target = set()
@@ -605,65 +636,6 @@ class CacheData(object):
self.bbfile_config_priorities = []
def add_from_recipeinfo(self, fn, info):
- self.task_deps[fn] = info.task_deps
- self.pkg_fn[fn] = info.pn
- self.pkg_pn[info.pn].append(fn)
- self.pkg_pepvpr[fn] = (info.pe, info.pv, info.pr)
- self.pkg_dp[fn] = info.defaultpref
- self.stamp[fn] = info.stamp
- self.stamp_extrainfo[fn] = info.stamp_extrainfo
-
- provides = [info.pn]
- for provide in info.provides:
- if provide not in provides:
- provides.append(provide)
- self.fn_provides[fn] = provides
-
- for provide in provides:
- self.providers[provide].append(fn)
- if provide not in self.pn_provides[info.pn]:
- self.pn_provides[info.pn].append(provide)
-
- for dep in info.depends:
- if dep not in self.deps[fn]:
- self.deps[fn].append(dep)
- if dep not in self.all_depends:
- self.all_depends.append(dep)
-
- rprovides = info.rprovides
- for package in info.packages:
- self.packages[package].append(fn)
- rprovides += info.rprovides_pkg[package]
+ info.add_cacheData(self, fn)
- for rprovide in rprovides:
- self.rproviders[rprovide].append(fn)
-
- for package in info.packages_dynamic:
- self.packages_dynamic[package].append(fn)
-
- # Build hash of runtime depends and rececommends
- for package in info.packages + [info.pn]:
- self.rundeps[fn][package] = list(info.rdepends) + info.rdepends_pkg[package]
- self.runrecs[fn][package] = list(info.rrecommends) + info.rrecommends_pkg[package]
-
- # Collect files we may need for possible world-dep
- # calculations
- if not info.broken and not info.not_world:
- self.possible_world.append(fn)
-
- # create a collection of all targets for sanity checking
- # tasks, such as upstream versions, license, and tools for
- # task and image creation.
- self.universe_target.append(info.pn)
-
- self.hashfn[fn] = info.hashfilename
- for task, taskhash in info.basetaskhashes.iteritems():
- identifier = '%s.%s' % (fn, task)
- self.basetaskhash[identifier] = taskhash
-
- self.inherits[fn] = info.inherits
- self.summary[fn] = info.summary
- self.license[fn] = info.license
- self.section[fn] = info.section
- self.fakerootenv[fn] = info.fakerootenv
- self.fakerootdirs[fn] = info.fakerootdirs
+
--
1.7.0.4
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 2/4][Image Creator]Introduce extra cache class for image creator
@ 2011-05-26 2:53 ` Ke Liping
0 siblings, 0 replies; 15+ messages in thread
From: Ke Liping @ 2011-05-26 2:53 UTC (permalink / raw)
To: poky
From: Liping Ke <liping.ke@intel.com>
Extra RecipeInfo will be all defined in this file. Currently,
Only Hob (Image Creator) Requests some extra fields. So
HobRecipeInfo is defined. It's named HobRecipeInfo because it
is introduced by 'hob'. Users could also introduce other
RecipeInfo or simply use those already defined RecipeInfo.
In the following patch, this newly defined new extra RecipeInfo
will be dynamically loaded and used for loading/saving the extra
cache fields.
Signed-off-by: Liping Ke <liping.ke@intel.com>
---
| 54 +++++++++++++++++++++++++++++++++++++++++
1 files changed, 54 insertions(+), 0 deletions(-)
create mode 100644 bitbake/lib/bb/cache_extra.py
--git a/bitbake/lib/bb/cache_extra.py b/bitbake/lib/bb/cache_extra.py
new file mode 100644
index 0000000..6a402d5
--- /dev/null
+++ b/bitbake/lib/bb/cache_extra.py
@@ -0,0 +1,54 @@
+# ex:ts=4:sw=4:sts=4:et
+# -*- tab-width: 4; c-basic-offset: 4; indent-tabs-mode: nil -*-
+#
+# Extra RecipeInfo will be all defined in this file. Currently,
+# Only Hob (Image Creator) Requests some extra fields. So
+# HobRecipeInfo is defined. It's named HobRecipeInfo because it
+# is introduced by 'hob'. Users could also introduce other
+# RecipeInfo or simply use those already defined RecipeInfo.
+# In the following patch, this newly defined new extra RecipeInfo
+# will be dynamically loaded and used for loading/saving the extra
+# cache fields
+
+# Copyright (C) 2011 Liping Ke
+
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License version 2 as
+# published by the Free Software Foundation.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License along
+# with this program; if not, write to the Free Software Foundation, Inc.,
+# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+
+from bb.cache import RecipeInfoCommon
+
+class HobRecipeInfo(RecipeInfoCommon):
+ __slots__ = ()
+
+ classname = "HobRecipeInfo"
+ # please override this member with the correct data cache file
+ # such as (bb_cache.dat, bb_extracache_hob.dat)
+ cachefile = "bb_extracache_" + classname +".dat"
+
+ def __init__(self, filename, metadata):
+
+ self.summary = self.getvar('SUMMARY', metadata)
+ self.license = self.getvar('LICENSE', metadata)
+ self.section = self.getvar('SECTION', metadata)
+
+ @classmethod
+ def init_cacheData(cls, cachedata):
+ # CacheData in Hob RecipeInfo Class
+ cachedata.summary = {}
+ cachedata.license = {}
+ cachedata.section = {}
+
+ def add_cacheData(self, cachedata, fn):
+ cachedata.summary[fn] = self.summary
+ cachedata.license[fn] = self.license
+ cachedata.section[fn] = self.section
--
1.7.0.4
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 3/4][Image Creator]Introduce new param caches_array into Cache impl.
@ 2011-05-26 2:53 ` Ke Liping
0 siblings, 0 replies; 15+ messages in thread
From: Ke Liping @ 2011-05-26 2:53 UTC (permalink / raw)
To: poky
From: Liping Ke <liping.ke@intel.com>
When using hob ui interface, we need extra cache fields.
We will save ui required extra cache fields into a separate
cache file. This patch introduce this caches_array parameter.
It will be used in the extra cache implementation (following patch).
Caches_array at least contains CoreRecipeInfo. If users need extra
cache fields support, such as 'hob', caches_array will contain
more relevant elements such as HobRecipeInfo.
Signed-off-by: Liping Ke <liping.ke@intel.com>
---
bitbake/lib/bb/cache.py | 15 ++++++++----
bitbake/lib/bb/cooker.py | 54 +++++++++++++++++++++++++++++++++++++++------
bitbake/lib/bb/shell.py | 2 +-
bitbake/lib/bb/ui/hob.py | 2 +
4 files changed, 59 insertions(+), 14 deletions(-)
diff --git a/bitbake/lib/bb/cache.py b/bitbake/lib/bb/cache.py
index 09691d9..0620621 100644
--- a/bitbake/lib/bb/cache.py
+++ b/bitbake/lib/bb/cache.py
@@ -245,7 +245,11 @@ class Cache(object):
BitBake Cache implementation
"""
- def __init__(self, data):
+ def __init__(self, data, caches_array):
+ # Pass caches_array information into Cache Constructor
+ # It will be used in later for deciding whether we
+ # need extra cache file dump/load support
+ self.caches_array = caches_array
self.cachedir = bb.data.getVar("CACHE", data, True)
self.clean = set()
self.checked = set()
@@ -360,7 +364,7 @@ class Cache(object):
return bb_data[virtual]
@classmethod
- def parse(cls, filename, appends, configdata):
+ def parse(cls, filename, appends, configdata, caches_array):
"""Parse the specified filename, returning the recipe information"""
infos = []
datastores = cls.load_bbfile(filename, appends, configdata)
@@ -393,7 +397,7 @@ class Cache(object):
infos.append((virtualfn, self.depends_cache[virtualfn]))
else:
logger.debug(1, "Parsing %s", filename)
- return self.parse(filename, appends, configdata)
+ return self.parse(filename, appends, configdata, self.caches_array)
return cached, infos
@@ -623,8 +627,9 @@ class CacheData(object):
The data structures we compile from the cached data
"""
- def __init__(self):
- CoreRecipeInfo.init_cacheData(self)
+ def __init__(self, caches_array):
+ self.caches_array = caches_array
+ CoreRecipeInfo.init_cacheData(self)
# Direct cache variables
self.task_queues = {}
self.preferred = {}
diff --git a/bitbake/lib/bb/cooker.py b/bitbake/lib/bb/cooker.py
index a1cd4d7..0dc895a 100644
--- a/bitbake/lib/bb/cooker.py
+++ b/bitbake/lib/bb/cooker.py
@@ -72,6 +72,43 @@ class BBCooker:
self.configuration = configuration
+ self.caches_array = []
+ # Currently, only Image Creator hob ui needs extra cache.
+ # So, we save Extra Cache class name and container file
+ # information into a extraCaches field in hob UI.
+ # In future, having a registration mechanism for extra cache
+ # fields in cache_extra is a better solution. Also, we may
+ # need to consider adding a user-hidden parameter 'CacheRequest'
+ # for bitbake command line. It will be filled by those sub
+ # command who need to have extra cache support.
+ caches_name_array = ['bb.cache:CoreRecipeInfo']
+ if configuration.ui:
+ try:
+ module = __import__('bb.ui', fromlist=[configuration.ui])
+ name_array = (getattr(module, configuration.ui)).extraCaches
+ for recipeInfoName in name_array:
+ caches_name_array.append(recipeInfoName)
+ except ImportError, exc:
+ # bb.ui.XXX is not defined and imported. It's an error!
+ logger.critical("Unable to import '%s' interface from bb.ui: %s" % (configuration.ui, exc))
+ sys.exit("FATAL: Failed to import '%s' interface." % configuration.ui)
+ except AttributeError:
+ # This is not an error. If the field is not defined in the ui,
+ # this interface might need no extra cache fields, so
+ # just skip this error!
+ logger.info("UI '%s' does not require extra cache!" % (configuration.ui))
+
+ # At least CoreRecipeInfo will be loaded, so caches_array will never be empty!
+ # This is the entry point, no further check needed!
+ for var in caches_name_array:
+ try:
+ module_name, cache_name = var.split(':')
+ module = __import__(module_name, fromlist=(cache_name,))
+ self.caches_array.append(getattr(module, cache_name))
+ except ImportError, exc:
+ logger.critical("Unable to import extra RecipeInfo '%s' from bb.extra_cache: %s" % (cache_name, exc))
+ sys.exit("FATAL: Failed to import extra cache class '%s'." % cache_name)
+
self.configuration.data = bb.data.init()
if not server:
@@ -713,9 +750,10 @@ class BBCooker:
self.buildSetVars()
- self.status = bb.cache.CacheData()
+ self.status = bb.cache.CacheData(self.caches_array)
infos = bb.cache.Cache.parse(fn, self.get_file_appends(fn), \
- self.configuration.data)
+ self.configuration.data,
+ self.caches_array)
infos = dict(infos)
fn = bb.cache.Cache.realfn2virtual(buildfile, cls)
@@ -859,7 +897,7 @@ class BBCooker:
else:
collectlog.info("You have disabled Psyco. This decreases performance.")
- self.status = bb.cache.CacheData()
+ self.status = bb.cache.CacheData(self.caches_array)
ignore = bb.data.getVar("ASSUME_PROVIDED", self.configuration.data, 1) or ""
self.status.ignored_dependencies = set(ignore.split())
@@ -1066,9 +1104,9 @@ class ParsingFailure(Exception):
self.args = (realexception, recipe)
def parse_file(task):
- filename, appends = task
+ filename, appends, caches_array = task
try:
- return True, bb.cache.Cache.parse(filename, appends, parse_file.cfg)
+ return True, bb.cache.Cache.parse(filename, appends, parse_file.cfg, caches_array)
except Exception, exc:
exc.recipe = filename
raise exc
@@ -1098,13 +1136,13 @@ class CookerParser(object):
self.num_processes = int(self.cfgdata.getVar("BB_NUMBER_PARSE_THREADS", True) or
multiprocessing.cpu_count())
- self.bb_cache = bb.cache.Cache(self.cfgdata)
+ self.bb_cache = bb.cache.Cache(self.cfgdata, cooker.caches_array)
self.fromcache = []
self.willparse = []
for filename in self.filelist:
appends = self.cooker.get_file_appends(filename)
if not self.bb_cache.cacheValid(filename):
- self.willparse.append((filename, appends))
+ self.willparse.append((filename, appends, cooker.caches_array))
else:
self.fromcache.append((filename, appends))
self.toparse = self.total - len(self.fromcache)
@@ -1179,6 +1217,6 @@ class CookerParser(object):
def reparse(self, filename):
infos = self.bb_cache.parse(filename,
self.cooker.get_file_appends(filename),
- self.cfgdata)
+ self.cfgdata, self.cooker.caches_array)
for vfn, info in infos:
self.cooker.status.add_from_recipeinfo(vfn, info)
diff --git a/bitbake/lib/bb/shell.py b/bitbake/lib/bb/shell.py
index 3319e2d..1dd8d54 100644
--- a/bitbake/lib/bb/shell.py
+++ b/bitbake/lib/bb/shell.py
@@ -407,7 +407,7 @@ SRC_URI = ""
def parse( self, params ):
"""(Re-)parse .bb files and calculate the dependency graph"""
- cooker.status = cache.CacheData()
+ cooker.status = cache.CacheData(cooker.caches_array)
ignore = data.getVar("ASSUME_PROVIDED", cooker.configuration.data, 1) or ""
cooker.status.ignored_dependencies = set( ignore.split() )
cooker.handleCollections( data.getVar("BBFILE_COLLECTIONS", cooker.configuration.data, 1) )
diff --git a/bitbake/lib/bb/ui/hob.py b/bitbake/lib/bb/ui/hob.py
index 0f8fe8c..ab6022b 100644
--- a/bitbake/lib/bb/ui/hob.py
+++ b/bitbake/lib/bb/ui/hob.py
@@ -28,6 +28,8 @@ import xmlrpclib
import logging
import Queue
+extraCaches = ['bb.cache_extra:HobRecipeInfo']
+
class MainWindow (gtk.Window):
def __init__(self, taskmodel, handler, curr_mach=None, curr_distro=None):
--
1.7.0.4
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 4/4][Image Creator]Implement multiple extra cache fields request support
@ 2011-05-26 2:53 ` Ke Liping
0 siblings, 0 replies; 15+ messages in thread
From: Ke Liping @ 2011-05-26 2:53 UTC (permalink / raw)
To: poky
From: Liping Ke <liping.ke@intel.com>
This patch is to support extra cache. If user needs to request
extra cache fields besides CoreRecipeInfo fields, just add a new
XXXRecipeInfo class definition as Hob Does.
Currently supported Extra RecipeInfo name is an array. We can
support multiple extra cache fields at the same time besides
CoreRecipeInfo which is needed by all clients.
Signed-off-by: Liping Ke <liping.ke@intel.com>
---
bitbake/lib/bb/cache.py | 175 +++++++++++++++++++++++++++++++---------------
bitbake/lib/bb/cooker.py | 16 ++--
2 files changed, 126 insertions(+), 65 deletions(-)
diff --git a/bitbake/lib/bb/cache.py b/bitbake/lib/bb/cache.py
index 0620621..271ebd7 100644
--- a/bitbake/lib/bb/cache.py
+++ b/bitbake/lib/bb/cache.py
@@ -45,6 +45,8 @@ except ImportError:
__cache_version__ = "138"
+def getCacheFile(path, filename):
+ return os.path.join(path, filename)
# RecipeInfoCommon defines common data retrieving methods
# from meta data for caches. CoreRecipeInfo as well as other
@@ -86,12 +88,9 @@ class RecipeInfoCommon(object):
class CoreRecipeInfo(RecipeInfoCommon):
__slots__ = ()
- def __init__(self, filename, metadata):
- self.name = "base"
- # please override this member with the correct data cache file
- # such as (bb_cache.dat, bb_extracache_hob.dat)
- self.cachefile = "bb_cache.dat"
+ cachefile = "bb_cache.dat"
+ def __init__(self, filename, metadata):
self.file_depends = metadata.getVar('__depends', False)
self.timestamp = bb.parse.cached_mtime(filename)
self.variants = self.listvar('__VARIANTS', metadata) + ['']
@@ -265,7 +264,7 @@ class Cache(object):
return
self.has_cache = True
- self.cachefile = os.path.join(self.cachedir, "bb_cache.dat")
+ self.cachefile = getCacheFile(self.cachedir, "bb_cache.dat")
logger.debug(1, "Using cache in '%s'", self.cachedir)
bb.utils.mkdirhier(self.cachedir)
@@ -279,12 +278,21 @@ class Cache(object):
old_mtimes.append(newest_mtime)
newest_mtime = max(old_mtimes)
- if bb.parse.cached_mtime_noerror(self.cachefile) >= newest_mtime:
+ bNeedUpdate = True
+ if self.caches_array:
+ for cache_class in self.caches_array:
+ if type(cache_class) is type and issubclass(cache_class, RecipeInfoCommon):
+ cachefile = getCacheFile(self.cachedir, cache_class.cachefile)
+ bNeedUpdate = bNeedUpdate and (bb.parse.cached_mtime_noerror(cachefile) >= newest_mtime)
+ cache_class.init_cacheData(self)
+ if bNeedUpdate:
self.load_cachefile()
elif os.path.isfile(self.cachefile):
logger.info("Out of date cache found, rebuilding...")
def load_cachefile(self):
+ # Firstly, using core cache file information for
+ # valid checking
with open(self.cachefile, "rb") as cachefile:
pickled = pickle.Unpickler(cachefile)
try:
@@ -301,31 +309,52 @@ class Cache(object):
logger.info('Bitbake version mismatch, rebuilding...')
return
- cachesize = os.fstat(cachefile.fileno()).st_size
- bb.event.fire(bb.event.CacheLoadStarted(cachesize), self.data)
- previous_percent = 0
- while cachefile:
- try:
- key = pickled.load()
- value = pickled.load()
- except Exception:
- break
+ cachesize = 0
+ previous_progress = 0
+ previous_percent = 0
- self.depends_cache[key] = value
-
- # only fire events on even percentage boundaries
- current_progress = cachefile.tell()
- current_percent = 100 * current_progress / cachesize
- if current_percent > previous_percent:
- previous_percent = current_percent
- bb.event.fire(bb.event.CacheLoadProgress(current_progress),
- self.data)
-
- bb.event.fire(bb.event.CacheLoadCompleted(cachesize,
- len(self.depends_cache)),
- self.data)
+ # Calculate the correct cachesize of all those cache files
+ for cache_class in self.caches_array:
+ if type(cache_class) is type and issubclass(cache_class, RecipeInfoCommon):
+ cachefile = getCacheFile(self.cachedir, cache_class.cachefile)
+ with open(cachefile, "rb") as cachefile:
+ cachesize += os.fstat(cachefile.fileno()).st_size
+ bb.event.fire(bb.event.CacheLoadStarted(cachesize), self.data)
+
+ for cache_class in self.caches_array:
+ if type(cache_class) is type and issubclass(cache_class, RecipeInfoCommon):
+ cachefile = getCacheFile(self.cachedir, cache_class.cachefile)
+ with open(cachefile, "rb") as cachefile:
+ pickled = pickle.Unpickler(cachefile)
+ while cachefile:
+ try:
+ key = pickled.load()
+ value = pickled.load()
+ except Exception:
+ break
+ if self.depends_cache.has_key(key):
+ self.depends_cache[key].append(value)
+ else:
+ self.depends_cache[key] = [value]
+ # only fire events on even percentage boundaries
+ current_progress = cachefile.tell() + previous_progress
+ current_percent = 100 * current_progress / cachesize
+ if current_percent > previous_percent:
+ previous_percent = current_percent
+ bb.event.fire(bb.event.CacheLoadProgress(current_progress),
+ self.data)
+
+ previous_progress += current_progress
+
+ # Note: depends cache number is corresponding to the parsing file numbers.
+ # The same file has several caches, still regarded as one item in the cache
+ bb.event.fire(bb.event.CacheLoadCompleted(cachesize,
+ len(self.depends_cache)),
+ self.data)
+
+
@staticmethod
def virtualfn2realfn(virtualfn):
"""
@@ -376,8 +405,14 @@ class Cache(object):
depends |= (data.getVar("__depends", False) or set())
if depends and not variant:
data.setVar("__depends", depends)
- info = CoreRecipeInfo(filename, data)
- infos.append((virtualfn, info))
+
+ info_array = []
+ for cache_class in caches_array:
+ if type(cache_class) is type and issubclass(cache_class, RecipeInfoCommon):
+ info = cache_class(filename, data)
+ info_array.append(info)
+ infos.append((virtualfn, info_array))
+
return infos
def load(self, filename, appends, configdata):
@@ -391,8 +426,9 @@ class Cache(object):
cached = self.cacheValid(filename)
if cached:
infos = []
- info = self.depends_cache[filename]
- for variant in info.variants:
+ # info_array item is a list of [CoreRecipeInfo, XXXRecipeInfo]
+ info_array = self.depends_cache[filename]
+ for variant in info_array[0].variants:
virtualfn = self.realfn2virtual(filename, variant)
infos.append((virtualfn, self.depends_cache[virtualfn]))
else:
@@ -408,12 +444,12 @@ class Cache(object):
skipped, virtuals = 0, 0
cached, infos = self.load(fn, appends, cfgData)
- for virtualfn, info in infos:
- if info.skipped:
+ for virtualfn, info_array in infos:
+ if info_array[0].skipped:
logger.debug(1, "Skipping %s", virtualfn)
skipped += 1
else:
- self.add_info(virtualfn, info, cacheData, not cached)
+ self.add_info(virtualfn, info_array, cacheData, not cached)
virtuals += 1
return cached, skipped, virtuals
@@ -457,15 +493,15 @@ class Cache(object):
self.remove(fn)
return False
- info = self.depends_cache[fn]
+ info_array = self.depends_cache[fn]
# Check the file's timestamp
- if mtime != info.timestamp:
+ if mtime != info_array[0].timestamp:
logger.debug(2, "Cache: %s changed", fn)
self.remove(fn)
return False
# Check dependencies are still valid
- depends = info.file_depends
+ depends = info_array[0].file_depends
if depends:
for f, old_mtime in depends:
fmtime = bb.parse.cached_mtime_noerror(f)
@@ -483,7 +519,7 @@ class Cache(object):
return False
invalid = False
- for cls in info.variants:
+ for cls in info_array[0].variants:
virtualfn = self.realfn2virtual(fn, cls)
self.clean.add(virtualfn)
if virtualfn not in self.depends_cache:
@@ -530,13 +566,30 @@ class Cache(object):
logger.debug(2, "Cache is clean, not saving.")
return
- with open(self.cachefile, "wb") as cachefile:
- pickler = pickle.Pickler(cachefile, pickle.HIGHEST_PROTOCOL)
- pickler.dump(__cache_version__)
- pickler.dump(bb.__version__)
- for key, value in self.depends_cache.iteritems():
- pickler.dump(key)
- pickler.dump(value)
+ file_dict = {}
+ pickler_dict = {}
+ for cache_class in self.caches_array:
+ if type(cache_class) is type and issubclass(cache_class, RecipeInfoCommon):
+ cache_class_name = cache_class.__name__
+ cachefile = getCacheFile(self.cachedir, cache_class.cachefile)
+ file_dict[cache_class_name] = open(cachefile, "wb")
+ pickler_dict[cache_class_name] = pickle.Pickler(file_dict[cache_class_name], pickle.HIGHEST_PROTOCOL)
+
+ pickler_dict['CoreRecipeInfo'].dump(__cache_version__)
+ pickler_dict['CoreRecipeInfo'].dump(bb.__version__)
+
+ try:
+ for key, info_array in self.depends_cache.iteritems():
+ for info in info_array:
+ if isinstance(info, RecipeInfoCommon):
+ cache_class_name = info.__class__.__name__
+ pickler_dict[cache_class_name].dump(key)
+ pickler_dict[cache_class_name].dump(info)
+ finally:
+ for cache_class in self.caches_array:
+ if type(cache_class) is type and issubclass(cache_class, RecipeInfoCommon):
+ cache_class_name = cache_class.__name__
+ file_dict[cache_class_name].close()
del self.depends_cache
@@ -544,17 +597,17 @@ class Cache(object):
def mtime(cachefile):
return bb.parse.cached_mtime_noerror(cachefile)
- def add_info(self, filename, info, cacheData, parsed=None):
- if not info.skipped:
- cacheData.add_from_recipeinfo(filename, info)
+ def add_info(self, filename, info_array, cacheData, parsed=None):
+ if isinstance(info_array[0], CoreRecipeInfo) and (not info_array[0].skipped):
+ cacheData.add_from_recipeinfo(filename, info_array)
if not self.has_cache:
return
- if (info.skipped or 'SRCREVINACTION' not in info.pv) and not info.nocache:
+ if (info_array[0].skipped or 'SRCREVINACTION' not in info_array[0].pv) and not info_array[0].nocache:
if parsed:
self.cacheclean = False
- self.depends_cache[filename] = info
+ self.depends_cache[filename] = info_array
def add(self, file_name, data, cacheData, parsed=None):
"""
@@ -562,8 +615,12 @@ class Cache(object):
"""
realfn = self.virtualfn2realfn(file_name)[0]
- info = CoreRecipeInfo(realfn, data)
- self.add_info(file_name, info, cacheData, parsed)
+
+ info_array = []
+ for cache_class in self.caches_array:
+ if type(cache_class) is type and issubclass(cache_class, RecipeInfoCommon):
+ info_array.append(cache_class(realfn, data))
+ self.add_info(file_name, info_array, cacheData, parsed)
@staticmethod
def load_bbfile(bbfile, appends, config):
@@ -629,7 +686,10 @@ class CacheData(object):
def __init__(self, caches_array):
self.caches_array = caches_array
- CoreRecipeInfo.init_cacheData(self)
+ for cache_class in self.caches_array:
+ if type(cache_class) is type and issubclass(cache_class, RecipeInfoCommon):
+ cache_class.init_cacheData(self)
+
# Direct cache variables
self.task_queues = {}
self.preferred = {}
@@ -640,7 +700,8 @@ class CacheData(object):
self.bbfile_priority = {}
self.bbfile_config_priorities = []
- def add_from_recipeinfo(self, fn, info):
- info.add_cacheData(self, fn)
+ def add_from_recipeinfo(self, fn, info_array):
+ for info in info_array:
+ info.add_cacheData(self, fn)
diff --git a/bitbake/lib/bb/cooker.py b/bitbake/lib/bb/cooker.py
index 0dc895a..b76a4d2 100644
--- a/bitbake/lib/bb/cooker.py
+++ b/bitbake/lib/bb/cooker.py
@@ -758,13 +758,13 @@ class BBCooker:
fn = bb.cache.Cache.realfn2virtual(buildfile, cls)
try:
- maininfo = infos[fn]
+ info_array = infos[fn]
except KeyError:
bb.fatal("%s does not exist" % fn)
- self.status.add_from_recipeinfo(fn, maininfo)
+ self.status.add_from_recipeinfo(fn, info_array)
# Tweak some variables
- item = maininfo.pn
+ item = info_array[0].pn
self.status.ignored_dependencies = set()
self.status.bbfile_priority[fn] = 1
@@ -1207,10 +1207,10 @@ class CookerParser(object):
else:
self.cached += 1
- for virtualfn, info in result:
- if info.skipped:
+ for virtualfn, info_array in result:
+ if info_array[0].skipped:
self.skipped += 1
- self.bb_cache.add_info(virtualfn, info, self.cooker.status,
+ self.bb_cache.add_info(virtualfn, info_array, self.cooker.status,
parsed=parsed)
return True
@@ -1218,5 +1218,5 @@ class CookerParser(object):
infos = self.bb_cache.parse(filename,
self.cooker.get_file_appends(filename),
self.cfgdata, self.cooker.caches_array)
- for vfn, info in infos:
- self.cooker.status.add_from_recipeinfo(vfn, info)
+ for vfn, info_array in infos:
+ self.cooker.status.add_from_recipeinfo(vfn, info_array)
--
1.7.0.4
^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH 0/4][Image Creator]Add mulitple extra cache fields request support
2011-05-26 2:53 ` Ke Liping
` (4 preceding siblings ...)
(?)
@ 2011-05-26 3:11 ` Ke, Liping
-1 siblings, 0 replies; 15+ messages in thread
From: Ke, Liping @ 2011-05-26 3:11 UTC (permalink / raw)
To: Ke, Liping, poky
Hi, All
I have test the memory usage with htop. It will show that when loading cache,
Bitbake will use 1.3~1.4%. There's no change compared with namedutple.
So I guess it should not impact memeory usage very much.
And the cahce file (core/base) part increased < 200k.
So this change should not impact performance greatly ( class vs namedtuple)
Thanks & Regards,
criping
> -----Original Message-----
> From: poky-bounces@yoctoproject.org
> [mailto:poky-bounces@yoctoproject.org] On Behalf Of Ke Liping
> Sent: Thursday, May 26, 2011 10:54 AM
> To: poky@yoctoproject.org
> Subject: [poky] [PATCH 0/4][Image Creator]Add mulitple extra cache fields
> request support
>
> From: Liping Ke <liping.ke@intel.com>
>
> The series of patch are for mplementing multiple extra cache fields
> request support.If user needs to request extra cache fields besides
> CoreRecipeInfo fields, just add a new XXXRecipeInfo class definition
> as Image Creator UI Hob Does.
> For implementing this new feature, we refactory current cache code,
> remove namedtuple and define corresponding class for dynamical cache
> fields support per different cache usage request.
>
> Liping Ke (4):
> Refactory Current Cache implementation
> Introduce extra cache class for image creator
> Introduce new param caches_array into Cache impl.
> Implement multiple extra cache fields request support
>
> bitbake/lib/bb/cache.py | 532
> ++++++++++++++++++++++-------------------
> bitbake/lib/bb/cache_extra.py | 54 ++++
> bitbake/lib/bb/cooker.py | 70 ++++--
> bitbake/lib/bb/shell.py | 2 +-
> bitbake/lib/bb/ui/hob.py | 2 +
> 5 files changed, 396 insertions(+), 264 deletions(-)
> create mode 100644 bitbake/lib/bb/cache_extra.py
>
> _______________________________________________
> poky mailing list
> poky@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/poky
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 3/4][Image Creator]Introduce new param caches_array into Cache impl.
2011-05-26 2:53 ` Ke Liping
(?)
@ 2011-05-26 10:27 ` Richard Purdie
2011-05-31 8:11 ` Ke, Liping
-1 siblings, 1 reply; 15+ messages in thread
From: Richard Purdie @ 2011-05-26 10:27 UTC (permalink / raw)
To: Ke Liping; +Cc: poky
Hi Liping,
This looks good, just a couple more minor tweaks:
On Thu, 2011-05-26 at 10:53 +0800, Ke Liping wrote:
> diff --git a/bitbake/lib/bb/cooker.py b/bitbake/lib/bb/cooker.py
> index a1cd4d7..0dc895a 100644
> --- a/bitbake/lib/bb/cooker.py
> +++ b/bitbake/lib/bb/cooker.py
> @@ -72,6 +72,43 @@ class BBCooker:
>
> self.configuration = configuration
>
> + self.caches_array = []
> + # Currently, only Image Creator hob ui needs extra cache.
> + # So, we save Extra Cache class name and container file
> + # information into a extraCaches field in hob UI.
> + # In future, having a registration mechanism for extra cache
> + # fields in cache_extra is a better solution. Also, we may
> + # need to consider adding a user-hidden parameter 'CacheRequest'
> + # for bitbake command line. It will be filled by those sub
> + # command who need to have extra cache support.
> + caches_name_array = ['bb.cache:CoreRecipeInfo']
> + if configuration.ui:
> + try:
> + module = __import__('bb.ui', fromlist=[configuration.ui])
> + name_array = (getattr(module, configuration.ui)).extraCaches
> + for recipeInfoName in name_array:
> + caches_name_array.append(recipeInfoName)
> + except ImportError, exc:
> + # bb.ui.XXX is not defined and imported. It's an error!
> + logger.critical("Unable to import '%s' interface from bb.ui: %s" % (configuration.ui, exc))
> + sys.exit("FATAL: Failed to import '%s' interface." % configuration.ui)
> + except AttributeError:
> + # This is not an error. If the field is not defined in the ui,
> + # this interface might need no extra cache fields, so
> + # just skip this error!
> + logger.info("UI '%s' does not require extra cache!" % (configuration.ui))
This should be logger.debug as we don't need to show this to the user in
general.
Whilst this code block is better than what we had previously, I think
bin/bitbake should pass some kind of information into cooker about the
required caches rather than cooker poking around the ui variables. We
can leave that as is for now but its a cleanup we should really do when
we get time. There are various UI startup issues we should probably
address at the same time.
> + # At least CoreRecipeInfo will be loaded, so caches_array will never be empty!
> + # This is the entry point, no further check needed!
> + for var in caches_name_array:
> + try:
> + module_name, cache_name = var.split(':')
> + module = __import__(module_name, fromlist=(cache_name,))
> + self.caches_array.append(getattr(module, cache_name))
> + except ImportError, exc:
> + logger.critical("Unable to import extra RecipeInfo '%s' from bb.extra_cache: %s" % (cache_name, exc))
We should use module_name instead of bb.extra_cache here :)
I'm really happy with these changes in general though, looks good!
Cheers,
Richard
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 3/4][Image Creator]Introduce new param caches_array into Cache impl.
2011-05-26 10:27 ` Richard Purdie
@ 2011-05-31 8:11 ` Ke, Liping
0 siblings, 0 replies; 15+ messages in thread
From: Ke, Liping @ 2011-05-31 8:11 UTC (permalink / raw)
To: Richard Purdie; +Cc: poky
Hi, Richard
Thanks for the comment! I will resend the patch. Sorry for the delay.
And also, for how to pass extra_cache parameters, yes, current impl
is not perfect. I add your comment into TODO comment above the
corresponding code part.
Thanks& Regards,
criping
> -----Original Message-----
> From: Richard Purdie [mailto:richard.purdie@linuxfoundation.org]
> Sent: Thursday, May 26, 2011 6:27 PM
> To: Ke, Liping
> Cc: poky@yoctoproject.org
> Subject: Re: [poky] [PATCH 3/4][Image Creator]Introduce new param
> caches_array into Cache impl.
>
> Hi Liping,
>
> This looks good, just a couple more minor tweaks:
>
> On Thu, 2011-05-26 at 10:53 +0800, Ke Liping wrote:
> > diff --git a/bitbake/lib/bb/cooker.py b/bitbake/lib/bb/cooker.py
> > index a1cd4d7..0dc895a 100644
> > --- a/bitbake/lib/bb/cooker.py
> > +++ b/bitbake/lib/bb/cooker.py
> > @@ -72,6 +72,43 @@ class BBCooker:
> >
> > self.configuration = configuration
> >
> > + self.caches_array = []
> > + # Currently, only Image Creator hob ui needs extra cache.
> > + # So, we save Extra Cache class name and container file
> > + # information into a extraCaches field in hob UI.
> > + # In future, having a registration mechanism for extra cache
> > + # fields in cache_extra is a better solution. Also, we may
> > + # need to consider adding a user-hidden parameter
> 'CacheRequest'
> > + # for bitbake command line. It will be filled by those sub
> > + # command who need to have extra cache support.
> > + caches_name_array = ['bb.cache:CoreRecipeInfo']
> > + if configuration.ui:
> > + try:
> > + module = __import__('bb.ui', fromlist=[configuration.ui])
> > + name_array = (getattr(module,
> configuration.ui)).extraCaches
> > + for recipeInfoName in name_array:
> > + caches_name_array.append(recipeInfoName)
> > + except ImportError, exc:
> > + # bb.ui.XXX is not defined and imported. It's an error!
> > + logger.critical("Unable to import '%s' interface from
> bb.ui: %s" % (configuration.ui, exc))
> > + sys.exit("FATAL: Failed to import '%s' interface." %
> configuration.ui)
> > + except AttributeError:
> > + # This is not an error. If the field is not defined in the ui,
> > + # this interface might need no extra cache fields, so
> > + # just skip this error!
> > + logger.info("UI '%s' does not require extra cache!" %
> (configuration.ui))
>
> This should be logger.debug as we don't need to show this to the user in
> general.
>
> Whilst this code block is better than what we had previously, I think
> bin/bitbake should pass some kind of information into cooker about the
> required caches rather than cooker poking around the ui variables. We
> can leave that as is for now but its a cleanup we should really do when
> we get time. There are various UI startup issues we should probably
> address at the same time.
>
> > + # At least CoreRecipeInfo will be loaded, so caches_array will
> never be empty!
> > + # This is the entry point, no further check needed!
> > + for var in caches_name_array:
> > + try:
> > + module_name, cache_name = var.split(':')
> > + module = __import__(module_name,
> fromlist=(cache_name,))
> > + self.caches_array.append(getattr(module,
> cache_name))
> > + except ImportError, exc:
> > + logger.critical("Unable to import extra RecipeInfo '%s'
> from bb.extra_cache: %s" % (cache_name, exc))
>
> We should use module_name instead of bb.extra_cache here :)
>
> I'm really happy with these changes in general though, looks good!
>
> Cheers,
>
> Richard
>
>
^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH 3/4][Image Creator]Introduce new param caches_array into Cache impl.
2011-06-03 1:06 [PATCH 0/4][Image Creator]Add mulitple extra cache fields request support Ke Liping
@ 2011-06-03 1:06 ` Ke Liping
0 siblings, 0 replies; 15+ messages in thread
From: Ke Liping @ 2011-06-03 1:06 UTC (permalink / raw)
To: poky, bitbake-devel
From: Liping Ke <liping.ke@intel.com>
When using hob ui interface, we need extra cache fields.
We will save ui required extra cache fields into a separate
cache file. This patch introduce this caches_array parameter.
It will be used in the extra cache implementation (following patch).
Caches_array at least contains CoreRecipeInfo. If users need extra
cache fields support, such as 'hob', caches_array will contain
more relevant elements such as HobRecipeInfo.
Signed-off-by: Liping Ke <liping.ke@intel.com>
---
bitbake/lib/bb/cache.py | 15 ++++++++----
bitbake/lib/bb/cooker.py | 52 ++++++++++++++++++++++++++++++++++++++-------
bitbake/lib/bb/shell.py | 2 +-
bitbake/lib/bb/ui/hob.py | 2 +
4 files changed, 57 insertions(+), 14 deletions(-)
diff --git a/bitbake/lib/bb/cache.py b/bitbake/lib/bb/cache.py
index 09691d9..0620621 100644
--- a/bitbake/lib/bb/cache.py
+++ b/bitbake/lib/bb/cache.py
@@ -245,7 +245,11 @@ class Cache(object):
BitBake Cache implementation
"""
- def __init__(self, data):
+ def __init__(self, data, caches_array):
+ # Pass caches_array information into Cache Constructor
+ # It will be used in later for deciding whether we
+ # need extra cache file dump/load support
+ self.caches_array = caches_array
self.cachedir = bb.data.getVar("CACHE", data, True)
self.clean = set()
self.checked = set()
@@ -360,7 +364,7 @@ class Cache(object):
return bb_data[virtual]
@classmethod
- def parse(cls, filename, appends, configdata):
+ def parse(cls, filename, appends, configdata, caches_array):
"""Parse the specified filename, returning the recipe information"""
infos = []
datastores = cls.load_bbfile(filename, appends, configdata)
@@ -393,7 +397,7 @@ class Cache(object):
infos.append((virtualfn, self.depends_cache[virtualfn]))
else:
logger.debug(1, "Parsing %s", filename)
- return self.parse(filename, appends, configdata)
+ return self.parse(filename, appends, configdata, self.caches_array)
return cached, infos
@@ -623,8 +627,9 @@ class CacheData(object):
The data structures we compile from the cached data
"""
- def __init__(self):
- CoreRecipeInfo.init_cacheData(self)
+ def __init__(self, caches_array):
+ self.caches_array = caches_array
+ CoreRecipeInfo.init_cacheData(self)
# Direct cache variables
self.task_queues = {}
self.preferred = {}
diff --git a/bitbake/lib/bb/cooker.py b/bitbake/lib/bb/cooker.py
index 0b52f18..89da7e9 100644
--- a/bitbake/lib/bb/cooker.py
+++ b/bitbake/lib/bb/cooker.py
@@ -72,6 +72,41 @@ class BBCooker:
self.configuration = configuration
+ self.caches_array = []
+ # Currently, only Image Creator hob ui needs extra cache.
+ # So, we save Extra Cache class name and container file
+ # information into a extraCaches field in hob UI.
+ # TODO: In future, bin/bitbake should pass information into cooker,
+ # instead of getting information from configuration.ui. Also, some
+ # UI start up issues need to be addressed at the same time.
+ caches_name_array = ['bb.cache:CoreRecipeInfo']
+ if configuration.ui:
+ try:
+ module = __import__('bb.ui', fromlist=[configuration.ui])
+ name_array = (getattr(module, configuration.ui)).extraCaches
+ for recipeInfoName in name_array:
+ caches_name_array.append(recipeInfoName)
+ except ImportError, exc:
+ # bb.ui.XXX is not defined and imported. It's an error!
+ logger.critical("Unable to import '%s' interface from bb.ui: %s" % (configuration.ui, exc))
+ sys.exit("FATAL: Failed to import '%s' interface." % configuration.ui)
+ except AttributeError:
+ # This is not an error. If the field is not defined in the ui,
+ # this interface might need no extra cache fields, so
+ # just skip this error!
+ logger.debug("UI '%s' does not require extra cache!" % (configuration.ui))
+
+ # At least CoreRecipeInfo will be loaded, so caches_array will never be empty!
+ # This is the entry point, no further check needed!
+ for var in caches_name_array:
+ try:
+ module_name, cache_name = var.split(':')
+ module = __import__(module_name, fromlist=(cache_name,))
+ self.caches_array.append(getattr(module, cache_name))
+ except ImportError, exc:
+ logger.critical("Unable to import extra RecipeInfo '%s' from '%s': %s" % (cache_name, module_name, exc))
+ sys.exit("FATAL: Failed to import extra cache class '%s'." % cache_name)
+
self.configuration.data = bb.data.init()
if not server:
@@ -730,9 +765,10 @@ class BBCooker:
self.buildSetVars()
- self.status = bb.cache.CacheData()
+ self.status = bb.cache.CacheData(self.caches_array)
infos = bb.cache.Cache.parse(fn, self.get_file_appends(fn), \
- self.configuration.data)
+ self.configuration.data,
+ self.caches_array)
infos = dict(infos)
fn = bb.cache.Cache.realfn2virtual(fn, cls)
@@ -876,7 +912,7 @@ class BBCooker:
else:
collectlog.info("You have disabled Psyco. This decreases performance.")
- self.status = bb.cache.CacheData()
+ self.status = bb.cache.CacheData(self.caches_array)
ignore = bb.data.getVar("ASSUME_PROVIDED", self.configuration.data, 1) or ""
self.status.ignored_dependencies = set(ignore.split())
@@ -1091,9 +1127,9 @@ class ParsingFailure(Exception):
self.args = (realexception, recipe)
def parse_file(task):
- filename, appends = task
+ filename, appends, caches_array = task
try:
- return True, bb.cache.Cache.parse(filename, appends, parse_file.cfg)
+ return True, bb.cache.Cache.parse(filename, appends, parse_file.cfg, caches_array)
except Exception, exc:
exc.recipe = filename
raise exc
@@ -1123,13 +1159,13 @@ class CookerParser(object):
self.num_processes = int(self.cfgdata.getVar("BB_NUMBER_PARSE_THREADS", True) or
multiprocessing.cpu_count())
- self.bb_cache = bb.cache.Cache(self.cfgdata)
+ self.bb_cache = bb.cache.Cache(self.cfgdata, cooker.caches_array)
self.fromcache = []
self.willparse = []
for filename in self.filelist:
appends = self.cooker.get_file_appends(filename)
if not self.bb_cache.cacheValid(filename):
- self.willparse.append((filename, appends))
+ self.willparse.append((filename, appends, cooker.caches_array))
else:
self.fromcache.append((filename, appends))
self.toparse = self.total - len(self.fromcache)
@@ -1205,6 +1241,6 @@ class CookerParser(object):
def reparse(self, filename):
infos = self.bb_cache.parse(filename,
self.cooker.get_file_appends(filename),
- self.cfgdata)
+ self.cfgdata, self.cooker.caches_array)
for vfn, info in infos:
self.cooker.status.add_from_recipeinfo(vfn, info)
diff --git a/bitbake/lib/bb/shell.py b/bitbake/lib/bb/shell.py
index 3319e2d..1dd8d54 100644
--- a/bitbake/lib/bb/shell.py
+++ b/bitbake/lib/bb/shell.py
@@ -407,7 +407,7 @@ SRC_URI = ""
def parse( self, params ):
"""(Re-)parse .bb files and calculate the dependency graph"""
- cooker.status = cache.CacheData()
+ cooker.status = cache.CacheData(cooker.caches_array)
ignore = data.getVar("ASSUME_PROVIDED", cooker.configuration.data, 1) or ""
cooker.status.ignored_dependencies = set( ignore.split() )
cooker.handleCollections( data.getVar("BBFILE_COLLECTIONS", cooker.configuration.data, 1) )
diff --git a/bitbake/lib/bb/ui/hob.py b/bitbake/lib/bb/ui/hob.py
index 0f8fe8c..ab6022b 100644
--- a/bitbake/lib/bb/ui/hob.py
+++ b/bitbake/lib/bb/ui/hob.py
@@ -28,6 +28,8 @@ import xmlrpclib
import logging
import Queue
+extraCaches = ['bb.cache_extra:HobRecipeInfo']
+
class MainWindow (gtk.Window):
def __init__(self, taskmodel, handler, curr_mach=None, curr_distro=None):
--
1.7.0.4
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 3/4][Image Creator]Introduce new param caches_array into Cache impl.
2011-05-31 8:15 Ke Liping
@ 2011-05-31 8:15 ` Ke Liping
0 siblings, 0 replies; 15+ messages in thread
From: Ke Liping @ 2011-05-31 8:15 UTC (permalink / raw)
To: poky
From: Liping Ke <liping.ke@intel.com>
When using hob ui interface, we need extra cache fields.
We will save ui required extra cache fields into a separate
cache file. This patch introduce this caches_array parameter.
It will be used in the extra cache implementation (following patch).
Caches_array at least contains CoreRecipeInfo. If users need extra
cache fields support, such as 'hob', caches_array will contain
more relevant elements such as HobRecipeInfo.
Signed-off-by: Liping Ke <liping.ke@intel.com>
---
bitbake/lib/bb/cache.py | 15 ++++++++----
bitbake/lib/bb/cooker.py | 52 ++++++++++++++++++++++++++++++++++++++-------
bitbake/lib/bb/shell.py | 2 +-
bitbake/lib/bb/ui/hob.py | 2 +
4 files changed, 57 insertions(+), 14 deletions(-)
diff --git a/bitbake/lib/bb/cache.py b/bitbake/lib/bb/cache.py
index 09691d9..0620621 100644
--- a/bitbake/lib/bb/cache.py
+++ b/bitbake/lib/bb/cache.py
@@ -245,7 +245,11 @@ class Cache(object):
BitBake Cache implementation
"""
- def __init__(self, data):
+ def __init__(self, data, caches_array):
+ # Pass caches_array information into Cache Constructor
+ # It will be used in later for deciding whether we
+ # need extra cache file dump/load support
+ self.caches_array = caches_array
self.cachedir = bb.data.getVar("CACHE", data, True)
self.clean = set()
self.checked = set()
@@ -360,7 +364,7 @@ class Cache(object):
return bb_data[virtual]
@classmethod
- def parse(cls, filename, appends, configdata):
+ def parse(cls, filename, appends, configdata, caches_array):
"""Parse the specified filename, returning the recipe information"""
infos = []
datastores = cls.load_bbfile(filename, appends, configdata)
@@ -393,7 +397,7 @@ class Cache(object):
infos.append((virtualfn, self.depends_cache[virtualfn]))
else:
logger.debug(1, "Parsing %s", filename)
- return self.parse(filename, appends, configdata)
+ return self.parse(filename, appends, configdata, self.caches_array)
return cached, infos
@@ -623,8 +627,9 @@ class CacheData(object):
The data structures we compile from the cached data
"""
- def __init__(self):
- CoreRecipeInfo.init_cacheData(self)
+ def __init__(self, caches_array):
+ self.caches_array = caches_array
+ CoreRecipeInfo.init_cacheData(self)
# Direct cache variables
self.task_queues = {}
self.preferred = {}
diff --git a/bitbake/lib/bb/cooker.py b/bitbake/lib/bb/cooker.py
index 3973529..c4e60d4 100644
--- a/bitbake/lib/bb/cooker.py
+++ b/bitbake/lib/bb/cooker.py
@@ -72,6 +72,41 @@ class BBCooker:
self.configuration = configuration
+ self.caches_array = []
+ # Currently, only Image Creator hob ui needs extra cache.
+ # So, we save Extra Cache class name and container file
+ # information into a extraCaches field in hob UI.
+ # TODO: In future, bin/bitbake should pass information into cooker,
+ # instead of getting information from configuration.ui. Also, some
+ # UI start up issues need to be addressed at the same time.
+ caches_name_array = ['bb.cache:CoreRecipeInfo']
+ if configuration.ui:
+ try:
+ module = __import__('bb.ui', fromlist=[configuration.ui])
+ name_array = (getattr(module, configuration.ui)).extraCaches
+ for recipeInfoName in name_array:
+ caches_name_array.append(recipeInfoName)
+ except ImportError, exc:
+ # bb.ui.XXX is not defined and imported. It's an error!
+ logger.critical("Unable to import '%s' interface from bb.ui: %s" % (configuration.ui, exc))
+ sys.exit("FATAL: Failed to import '%s' interface." % configuration.ui)
+ except AttributeError:
+ # This is not an error. If the field is not defined in the ui,
+ # this interface might need no extra cache fields, so
+ # just skip this error!
+ logger.debug("UI '%s' does not require extra cache!" % (configuration.ui))
+
+ # At least CoreRecipeInfo will be loaded, so caches_array will never be empty!
+ # This is the entry point, no further check needed!
+ for var in caches_name_array:
+ try:
+ module_name, cache_name = var.split(':')
+ module = __import__(module_name, fromlist=(cache_name,))
+ self.caches_array.append(getattr(module, cache_name))
+ except ImportError, exc:
+ logger.critical("Unable to import extra RecipeInfo '%s' from '%s': %s" % (cache_name, module_name, exc))
+ sys.exit("FATAL: Failed to import extra cache class '%s'." % cache_name)
+
self.configuration.data = bb.data.init()
if not server:
@@ -729,9 +764,10 @@ class BBCooker:
self.buildSetVars()
- self.status = bb.cache.CacheData()
+ self.status = bb.cache.CacheData(self.caches_array)
infos = bb.cache.Cache.parse(fn, self.get_file_appends(fn), \
- self.configuration.data)
+ self.configuration.data,
+ self.caches_array)
infos = dict(infos)
fn = bb.cache.Cache.realfn2virtual(fn, cls)
@@ -875,7 +911,7 @@ class BBCooker:
else:
collectlog.info("You have disabled Psyco. This decreases performance.")
- self.status = bb.cache.CacheData()
+ self.status = bb.cache.CacheData(self.caches_array)
ignore = bb.data.getVar("ASSUME_PROVIDED", self.configuration.data, 1) or ""
self.status.ignored_dependencies = set(ignore.split())
@@ -1087,9 +1123,9 @@ class ParsingFailure(Exception):
self.args = (realexception, recipe)
def parse_file(task):
- filename, appends = task
+ filename, appends, caches_array = task
try:
- return True, bb.cache.Cache.parse(filename, appends, parse_file.cfg)
+ return True, bb.cache.Cache.parse(filename, appends, parse_file.cfg, caches_array)
except Exception, exc:
exc.recipe = filename
raise exc
@@ -1119,13 +1155,13 @@ class CookerParser(object):
self.num_processes = int(self.cfgdata.getVar("BB_NUMBER_PARSE_THREADS", True) or
multiprocessing.cpu_count())
- self.bb_cache = bb.cache.Cache(self.cfgdata)
+ self.bb_cache = bb.cache.Cache(self.cfgdata, cooker.caches_array)
self.fromcache = []
self.willparse = []
for filename in self.filelist:
appends = self.cooker.get_file_appends(filename)
if not self.bb_cache.cacheValid(filename):
- self.willparse.append((filename, appends))
+ self.willparse.append((filename, appends, cooker.caches_array))
else:
self.fromcache.append((filename, appends))
self.toparse = self.total - len(self.fromcache)
@@ -1201,6 +1237,6 @@ class CookerParser(object):
def reparse(self, filename):
infos = self.bb_cache.parse(filename,
self.cooker.get_file_appends(filename),
- self.cfgdata)
+ self.cfgdata, self.cooker.caches_array)
for vfn, info in infos:
self.cooker.status.add_from_recipeinfo(vfn, info)
diff --git a/bitbake/lib/bb/shell.py b/bitbake/lib/bb/shell.py
index 3319e2d..1dd8d54 100644
--- a/bitbake/lib/bb/shell.py
+++ b/bitbake/lib/bb/shell.py
@@ -407,7 +407,7 @@ SRC_URI = ""
def parse( self, params ):
"""(Re-)parse .bb files and calculate the dependency graph"""
- cooker.status = cache.CacheData()
+ cooker.status = cache.CacheData(cooker.caches_array)
ignore = data.getVar("ASSUME_PROVIDED", cooker.configuration.data, 1) or ""
cooker.status.ignored_dependencies = set( ignore.split() )
cooker.handleCollections( data.getVar("BBFILE_COLLECTIONS", cooker.configuration.data, 1) )
diff --git a/bitbake/lib/bb/ui/hob.py b/bitbake/lib/bb/ui/hob.py
index 0f8fe8c..ab6022b 100644
--- a/bitbake/lib/bb/ui/hob.py
+++ b/bitbake/lib/bb/ui/hob.py
@@ -28,6 +28,8 @@ import xmlrpclib
import logging
import Queue
+extraCaches = ['bb.cache_extra:HobRecipeInfo']
+
class MainWindow (gtk.Window):
def __init__(self, taskmodel, handler, curr_mach=None, curr_distro=None):
--
1.7.0.4
^ permalink raw reply related [flat|nested] 15+ messages in thread
end of thread, other threads:[~2011-06-03 1:23 UTC | newest]
Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-05-26 2:51 [PATCH 0/4][Image Creator]Add mulitple extra cache fields request support Ke Liping
2011-05-26 2:53 ` Ke Liping
2011-05-26 2:52 ` [PATCH 1/4][Image Creator]Refactory Current Cache implementation Ke Liping
2011-05-26 2:53 ` Ke Liping
2011-05-26 2:52 ` [PATCH 2/4][Image Creator]Introduce extra cache class for image creator Ke Liping
2011-05-26 2:53 ` Ke Liping
2011-05-26 2:52 ` [PATCH 3/4][Image Creator]Introduce new param caches_array into Cache impl Ke Liping
2011-05-26 2:53 ` Ke Liping
2011-05-26 10:27 ` Richard Purdie
2011-05-31 8:11 ` Ke, Liping
2011-05-26 2:52 ` [PATCH 4/4][Image Creator]Implement multiple extra cache fields request support Ke Liping
2011-05-26 2:53 ` Ke Liping
2011-05-26 3:11 ` [PATCH 0/4][Image Creator]Add mulitple " Ke, Liping
2011-05-31 8:15 Ke Liping
2011-05-31 8:15 ` [PATCH 3/4][Image Creator]Introduce new param caches_array into Cache impl Ke Liping
2011-06-03 1:06 [PATCH 0/4][Image Creator]Add mulitple extra cache fields request support Ke Liping
2011-06-03 1:06 ` [PATCH 3/4][Image Creator]Introduce new param caches_array into Cache impl Ke Liping
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.