summaryrefslogtreecommitdiffstats
path: root/bitbake
Commit message (Collapse)AuthorAgeFilesLines
* bitbake: runqueue: Remove use of waitpid on worker processesRichard Purdie2014-03-191-16/+12
| | | | | | | | | | | | | Use of waitpid on the worker processes is a bad idea since it conflicts directly with subprocess internals. Instead use the poll() method and returncode to determine if the process has exitted, if it has, we can shut down the system. This should resolve the hangs once and for all, famous last words. (Bitbake rev: 60969cd62e21e7d4af161bf8504b7643a879c73f) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: runqueue: Revert child signal handler for nowRichard Purdie2014-03-191-28/+17
| | | | | | | | | | | | | | | | | | | | | | | | | We're running into processes using 100% cpu. It appears theses are locked in a subprocess.poll() type loop where the process has exited but the code is looping as its not handling the ECHILD error. http://bugs.python.org/issue14396 http://bugs.python.org/issue15756 This is likely due to one or both of the above bugs. The question is what actually grabbed the child exit code as it wasn't this code. Its likely there is therefore some other code racing and taking that code, it may be some kind of race like: http://hg.python.org/cpython/rev/767420808a62/ where the fix effectively catches the childs codes in a different part of the system. We could try and get everyone onto python 2.7.4 where the above bugs are fixed however for now its safer to admit defeat and go back to polling explictly for our worker exit codes. (Bitbake rev: 5b9a099ec2a1dc954b614e12a306595f55b6a99e) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: runqueue: Don't catch all child return codesRichard Purdie2014-03-191-3/+4
| | | | | | | | | | Catching all child exit status values is a bad idea. Setting an http sstate mirror is a great way to view that spectacularly break things. The previous change did have good code changes so don't revert those parts. (Bitbake rev: fa7ffb62d510ac1124ae7e08fa4d190a710f5b54) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: runqueue: Really fix sigchld handlingRichard Purdie2014-03-181-14/+14
| | | | | | | | | | | | | | | | | | | There are several problems. Firstly, a return value of "None" can mean there is a C signal handler installed so we need to better handle that case. signal.SIG_DFL is 0 which equates to false so we also need to handle that by testing explicitly for None. Finally, the signal handler *must* call waitpid on all child processes else it will just get called repeatedly, leading to the hanging behaviour we've been seeing. The solution is to only error for the worker children, we warn about any other stray children which we'll have to figure out the sources of in due course. Hopefully this patch gets things working again properly though. (Bitbake rev: 973876c706f08735c1b68c791a5a137e5f083dd2) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: runqueue: Ensure handler does not recurseRichard Purdie2014-03-181-1/+1
| | | | | | | | | Failures on the autobuilder look like this handler is recursing. That shouldn't be possible but it doesn't hurt to code as such. (Bitbake rev: e39e85803cbe1ef9413a118868c19087c0546d01) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: runqueue: More carefully handle the sigchld handlerRichard Purdie2014-03-181-2/+4
| | | | | | | | | | | | We've noticed hanging processes which appear to be looping around waitpid. Its possible multiple calls to teardown are causing problem or in theory multiple registrations (although the code should not allow that). Regardless, put better guards around signal handler registration. (Bitbake rev: 79acfb0853aa3215215cee89a945f8e97b0a8fae) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: hob: sync after image deployValentin Popa2014-03-171-1/+1
| | | | | | | | | | | | | | | Showing "Deploy image successful" after 'dd' returns may determine the user to disconnect the usb stick even though the writing operations are not finished. This patch makes sure that the entire image is deployed on the usb stick before the user is informed about any result. [YOCTO #5892] (Bitbake rev: cc98b19112ab875ebc7cb604cd96acadac4cbf21) Signed-off-by: Valentin Popa <valentin.popa@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: runqueue: Don't error if we never setup workersRichard Purdie2014-03-171-1/+2
| | | | | | | | | | If we didn't setup any workers (such as bitbake -S), this would error since we're trying to set a signal handler to None. This patch avoids that problem. (Bitbake rev: ce17478c8197abf178c00774f5bbe23fd4375ee2) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: added file types to the Outputs column in the build pageFarrell Wymore2014-03-172-1/+32
| | | | | | | | | | | | The file types are displayed in the Outputs column in the build page. The file types are derived from the target image filenames. [YOCTO #5947] (Bitbake rev: 37ae4e94d6991d4f05b0236b525e29797ed6e49c) Signed-off-by: Farrell Wymore <farrell.wymore@windriver.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: Sort layers in alphabetical orderBelen Barros Pena2014-03-171-1/+1
| | | | | | | | | | | In the Summary tab of the Configuration page, make sure that the table of layers is sorted by layer name in ascending alphabetical order. (Bitbake rev: f339555df40307420ce80a4ef8cba1a4d284d380) Signed-off-by: Belen Barros Pena <belen.barros.pena@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: Remove the data-toggle attributeBelen Barros Pena2014-03-171-1/+1
| | | | | | | | | | | | Help icons with the .get-help class do not need the data-toggle attribute. That attribute initialises the Bootstrap tooltips, but they are already initialised in main.js for any element with the .get-help class. (Bitbake rev: 886a87de2d36382f40830c0d28fd2f55871d0993) Signed-off-by: Belen Barros Pena <belen.barros.pena@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: Don't show clear search button if text input field is emptyBelen Barros Pena2014-03-174-4/+4
| | | | | | | | | | | The clear search button should only show when the search text input field is populated. If it is empty (as it happens when a filter returns no results) the clear search button should not display. (Bitbake rev: 6cc4aaf08b51bb6cd8d1813c33989a788bd06990) Signed-off-by: Belen Barros Pena <belen.barros.pena@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: Fix Recipe sorting in tasks tableBelen Barros Pena2014-03-171-2/+2
| | | | | | | | | | | | In the tasks table and the other tables derived from it (Time, CPU usage and Disk I/O) sorting by Recipe was not working correctly. This change fixes the problem by specifying use of the recipe name to sort. (Bitbake rev: fa24c857e6f5b3c56f89a2a85dea6ad271f82c01) Signed-off-by: Belen Barros Pena <belen.barros.pena@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: Not using task_color tag for execution headingBelen Barros Pena2014-03-172-7/+8
| | | | | | | | | | | | | | The modifications to the task_color tag in commit 23a7c338d387ac2ba13a7a1114a4abc75228c960 broke the styling of failed tasks in the tasks.html template. Undo the changes to the task_color tag and use an if statement instead to set the .muted class when the execution heading says "Not executed". (Bitbake rev: e3b3205674f606b927f1bf568202a592ca6453c9) Signed-off-by: Belen Barros Pena <belen.barros.pena@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: Presentation fixes for task.htmlBelen Barros Pena2014-03-173-18/+26
| | | | | | | | | | | | | | | | | | | | | | | | Tidying up the presentation in the task.html template. The changes include: * Correct the markup for the help tooltips * Fix the help content for the outcome heading * Make sure <dt> tags do not show for empty log file, time, cpu and disk I/O values * Eliminate an extra <dl> for tasks with sstate attempts * Add <strong> tag to the sstate restored alert * Replace the .alert-info class with the .muted class for the no dependencies messages * Make sure the Executed heading does not inherit the .red class for failed tasks * Format time and cpu values to make sure they only show 2 decimal digits (Bitbake rev: bbc22958ab37dcd44c03420a7b8f842a1f4e51b1) Signed-off-by: Belen Barros Pena <belen.barros.pena@intel.com> Signed-off-by: Alexandru DAMIAN <alexandru.damian@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: Shorten dependency labelsBelen Barros Pena2014-03-171-2/+2
| | | | | | | | | | | | | Changing "Task depends on" to "Dependencies" and "Task reverse dependencies" to "Reverse dependencies". This matches the labeling to recipe and package information, with the additional advantage of making the labels shorter. (Bitbake rev: 6147339ac2f93a9e60eed8975ac1784caa06bb0b) Signed-off-by: Belen Barros Pena <belen.barros.pena@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: Give an extra space to the caretBelen Barros Pena2014-03-171-1/+1
| | | | | | | | | | | In the "Collapse variable value" button in configvars.html, the caret was too close to the button label. Giving it a bit of extra space. (Bitbake rev: 5e45f2bb4b0d9b56b5f4734cf40d3a267654ccb9) Signed-off-by: Belen Barros Pena <belen.barros.pena@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: Add no search results pageBelen Barros Pena2014-03-174-0/+56
| | | | | | | | | | | | | | | Adding the no search results page to the built packages (bpackage.html), variables (configvars.html), recipes (recipes.html) and tasks (tasks.html) tables. The change copies the code from the build.html template into the other 4 templates. There is probably a smarter way of doing this, though. (Bitbake rev: 3feff77091b3926c10ca8f30890c9049633ccb66) Signed-off-by: Belen Barros Pena <belen.barros.pena@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: Fixing "Show all builds" linkBelen Barros Pena2014-03-171-1/+1
| | | | | | | | | | | | Replacing the placeholder attribute with the value attribute in the no search results page broke the "Show all builds" link. This change applies the inline javacript used for the clear search button for the "Show all builds" link, which fixes the problem. (Bitbake rev: a58b88aebfde54ea97740f27335582fac195dfdc) Signed-off-by: Belen Barros Pena <belen.barros.pena@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: Amend help text in package_built_detail.htmlBelen Barros Pena2014-03-171-2/+2
| | | | | | | | | Matching the help text in the tabs to the design specification. (Bitbake rev: cc478b3b0befc0c8fce93d896074f8d07eaac362) Signed-off-by: Belen Barros Pena <belen.barros.pena@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: Small tweaks to the no results pageBelen Barros Pena2014-03-172-8/+6
| | | | | | | | | | | | | | | | | Correctly align the Search button to the text input field, add .btn class to the clear search button, replace the placeholder attribute with the value attribute so that you can edit your search query, remove the clear search button from the tab index so that you don't clear the search by mistake and edit the margins of the .no-results class. (Bitbake rev: 4ada4307623c1e27b589831d359dc6351e6e3ad5) Signed-off-by: Belen Barros Pena <belen.barros.pena@intel.com> Conflicts: bitbake/lib/toaster/toastergui/templates/build.html Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: Replace hyphens with underscores in package nameBelen Barros Pena2014-03-174-8/+8
| | | | | | | | | | Package details pages were using hyphens to separate package name from package version. Changing them to underscores. (Bitbake rev: c0820f93e0387d2b6ead9e2fa7205d49001d27e6) Signed-off-by: Belen Barros Pena <belen.barros.pena@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: Format package size in recipe.htmlBelen Barros Pena2014-03-171-1/+1
| | | | | | | | | | | | Package sizes in the recipe details page (recipe.html) were displaying in bytes. Apply the filtered_filesizeformat project tag to show the package size in a more human readable format. (Bitbake rev: 018db5cf683755a7a41b0ef491e130809e1ff003) Signed-off-by: Belen Barros Pena <belen.barros.pena@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: Make Order column part of the minimum tableBelen Barros Pena2014-03-171-1/+1
| | | | | | | | | | | | The Order column in the tasks table should be always shown and be part of the minimum table (i.e. its checkbox should be disabled in the Edit columns menu). Changing views.py to make sure this is the case. (Bitbake rev: a928e0cfb82dac0eb920913e3f9c5b45e3dcdf2f) Signed-off-by: Belen Barros Pena <belen.barros.pena@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: Move <tbody> outside for statementBelen Barros Pena2014-03-175-59/+56
| | | | | | | | | | | | | | In the package details pages, the <tbody> tags where inside the for statements, which caused multiple <tbody> tags to be generated inside a single table. To make sure only one <tbody> tag exists per table, moving the <tbody> tag outside the for statement. (Bitbake rev: 0c111b24e9f86130bc43c1327a6d12026e92cdf2) Signed-off-by: Belen Barros Pena <belen.barros.pena@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: improve recipe matching for native tasksAlexandru DAMIAN2014-03-171-6/+12
| | | | | | | | | | This patch improves the recipe matching algorithm for for matching recipes for native tasks. (Bitbake rev: c350e4924abab8688c539608fd7f3af687d7265a) Signed-off-by: Alexandru DAMIAN <alexandru.damian@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: fix target file inode type informationAlexandru DAMIAN2014-03-171-4/+4
| | | | | | | | | | This patches fixes the inode type saved when writing the target file list information. (Bitbake rev: 9f34a1c5e94d73cdba1def7059c60211514e054c) Signed-off-by: Alexandru DAMIAN <alexandru.damian@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: bitbake: cooker: some IMAGE_FEATURES not recognizedMarius Avram2014-03-141-1/+1
| | | | | | | | | | | | | | | Fixes an issue in hob which happened when the local.conf file was modified externally by appending "eclipse-debug" to the IMAGE_FEATURES variable. The reason of the problem is that some IMAGE_FEATURES are not available in the image.bbclass file and they are declared in the core-image.bbclass. Now a default hob image will inherit core-image. [YOCTO #5711] (Bitbake rev: 81413d94f40f58d790d7a7dc4259108f9c5d4fc0) Signed-off-by: Marius Avram <marius.avram@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: add errors and warnings listingAmit Kumar Chaudhary2014-03-143-3/+76
| | | | | | | | | | Add the errors and warnings listing to the build dashboard page. (Bitbake rev: 2004c0fd1dd049d26279ee10a18e69e6852247fa) Signed-off-by: Amit Kumar Chaudhary <amit@floatingpondtech.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: Add counters to filtersRavi Chintakunta2014-03-142-40/+58
| | | | | | | | | | | | Added count to filters in Build, Tasks and Config variables. [YOCTO #5821] (Bitbake rev: afd73f8ee47d0fcf600fa919f2d2bc1b7d2eeadf) Signed-off-by: Ravi Chintakunta <ravi.chintakunta@timesys.com> Signed-off-by: Alexandru DAMIAN <alexandru.damian@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: add license manifest path to databaseCristiana Voicu2014-03-142-3/+19
| | | | | | | | | | | | Based on image_name, the target is obtained, and the path is added to the database. [YOCTO #5649] (Bitbake rev: 911b5191133956c30d53f57629c115db196b9ac8) Signed-off-by: Cristiana Voicu <cristiana.voicu@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: populate target image file tableCristiana Voicu2014-03-144-1/+18
| | | | | | | | | | | | | | Using ImageFileSize Metadata event, the image output file and its size are populated into target_image_file table. [YOCTO #5189] [YOCTO #5228] (Bitbake rev: a0b06d362b9aa08fda293489467af343c6ca6de4) Signed-off-by: Cristiana Voicu <cristiana.voicu@intel.com> Signed-off-by: Alexandru DAMIAN <alexandru.damian@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: runqueue: Improve sigchld handlerRichard Purdie2014-03-121-5/+10
| | | | | | | | | | | | | | | | | | | The sigchld handler was reaping any processes and this was leading to confusion with any other process handling code that could be active. This patch: a) Ensures we only read any process results for the worker processes we want to monitor b) Ensures we pass the event to any other sigchld handler if it isn't an event we're interested in so the functions are properly chained. Together this should resolve some of the reports of unknown processes people have been reporting. (Bitbake rev: fe8baaa2f533db7a1b7203476c675588923d8d45) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: bitbake-worker: Ensure children have default sigterm handlerRichard Purdie2014-03-111-0/+2
| | | | | | | | | | The children of the worker should have the default SIGTERM handler, else they'll try and do cleanup which should only happen in the parent leading to all kinds of bizarre build failures. (Bitbake rev: a53c8d1f846d94082aa459996c4114f10970b8ef) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: bitbake: fix typo in variable nameRoss Burton2014-03-111-1/+1
| | | | | | | | | | When passing -t with an incorrect server type, the error message was using a variable that doesn't exist. (Bitbake rev: 98b991287df06cd89955c1d0591fce3b5d4403d1) Signed-off-by: Ross Burton <ross.burton@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: fix 'show all' labels for tasks and variablesDavid Reyna2014-03-111-1/+1
| | | | | | | | | | | | | Fix the 'Show all' button label to indicate 'variables' and 'tasks' instead of the internal page label for the variables page and the Disk I/O, CPU Usage, and Time pages. [YOCTO #5915] (Bitbake rev: e84866b7aa5a1b272c81fde548f3c7828b036bb9) Signed-off-by: David Reyna <David.Reyna@windriver.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: add 'bblayers.conf' files to local config filterDavid Reyna2014-03-111-1/+3
| | | | | | | | | | | | | In the variables table, add to the local configuration filter the 'bblayers.conf' in addition to the 'local.conf' files. [YOCTO #5912] (Bitbake rev: 3c41481fd00056712348d305d0246d84e9a2ff0a) Signed-off-by: David Reyna <David.Reyna@windriver.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: add empty outcome support to task pageDavid Reyna2014-03-113-1/+6
| | | | | | | | | | | | | Add the new "empty" outcome value to the task detail page, with help text, help notice, plus sorting for empty in the tasks page. [YOCTO #5917] (Bitbake rev: 874b627b96809c274023b5fa330dfbdd14676d19) Signed-off-by: David Reyna <David.Reyna@windriver.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: Fix help text for task details outcomeBelen Barros Pena2014-03-112-11/+11
| | | | | | | | | | | | Changes made to models.py and task.html to make sure the help text for the task outcome is displayed. [YOCTO #5917] (Bitbake rev: f3001af0b73f103a5d37af180e8c4c54a6249e97) Signed-off-by: Belen Barros Pena <belen.barros.pena@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: erase checks for stop commandAlexandru DAMIAN2014-03-111-15/+2
| | | | | | | | | | | | | We remove the checks for the stop command, since we assume the user really means it wants to shutdown the system when he issues the command. [YOCTO #5376] (Bitbake rev: 5d6594de8876be8ce4df56b846fee0fc687a6261) Signed-off-by: Alexandru DAMIAN <alexandru.damian@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: add commands to list and delete buildsAlexandru DAMIAN2014-03-114-0/+46
| | | | | | | | | | | | | | We add Django commands for the manage.py to manage the database content. The two commands added are: * buildslist - produces a list of current builds * builddelete - deletes a build and all associated data from the database (Bitbake rev: e9a8c32512bb270cda3dee4a3ed5fd22204c24bc) Signed-off-by: Alexandru DAMIAN <alexandru.damian@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: write files-in-image to the databaseAlexandru DAMIAN2014-03-111-2/+113
| | | | | | | | | | Adding code to write files-in-image data from the metadata event to the database. (Bitbake rev: f3a69ef7cc536a4b879d60936199a1002584c4f4) Signed-off-by: Alexandru DAMIAN <alexandru.damian@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: toaster: update database schemaAlexandru DAMIAN2014-03-112-5/+241
| | | | | | | | | | | | | | This is a schema update to account for * relaxing constraints in Target_Image_File table * modifying permission filed to string value * adding license_manifest_path field * adding image_size field (Bitbake rev: f8488845ee9670e16a63c331dd2e97fdb314929c) Signed-off-by: Alexandru DAMIAN <alexandru.damian@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: runqueue: Use SIGCHLD instead of polling waitpidRichard Purdie2014-03-101-14/+19
| | | | | | | | | Instead of a significant number of calls to waitpid, register a SIGCHLD handler instead. (Bitbake rev: 76029d08ad56a0a264ff9738a0336971a455b7f5) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: server/process: Optimise latency when finishing idle functionsRichard Purdie2014-03-101-0/+1
| | | | | | | | | | When idle functions finish, its likely we have some other work to do, so don't sleep in the select call but instead, skip it. This removes small amounts of latency in common commands. (Bitbake rev: 069d6538f83b607cb46c6fe21bf6c596e8b99242) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: server/process: Drop unnecessary exit delayRichard Purdie2014-03-101-1/+0
| | | | | | | | | | When the server exits, we no longer appear to need this delay. This is likely due to improvements in the various exit codepaths. There is therefore no longer any point in taking the latency hit. (Bitbake rev: 8e75ee29ae07e13f23525c5c6045fbf6cdbe7675) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: server/process: Use a pipe for quit events instead of Event()Richard Purdie2014-03-101-5/+9
| | | | | | | | | | | | | | Its not possible to notice the change of status of an Event() in the select call we sleep in. It would be possible in python 3.3 but for now use a pipe instead. This removes small latency when bitbake commands finish since the system doesn't sit in the select call. (Debugging these kind of issues is apparent by setting a long sleep for the select call) (Bitbake rev: def28239b0f0d5f1cf13214b263114a5328538b7) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: knotty: Remove latency when exitingRichard Purdie2014-03-101-2/+2
| | | | | | | | | | There is no point in waiting 0.25s for when we should be processing the shutdown. This simply reordering removes latency from the bitbake command. (Bitbake rev: f147b41bcaf9d05b5ba3a70100f1ca799979aee7) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: knotty: Add missing continue statement for runQueueExitWait eventRichard Purdie2014-03-101-2/+1
| | | | | | | | | The continue statement was missing for this event and the event was then listed in the "known safe to ignore list". Clean this up. (Bitbake rev: c4ee342300bf905e6e3bef581c61b86289461536) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake: providers/runqueue/taskdata: Optimise logger.debug callsRichard Purdie2014-03-103-5/+11
| | | | | | | | | | | | | A run of "bitbake bash -c unpack" when the task has already been completed resulted in about 9000 calls to logger.debug(). With this patch which comments out some noisy/less usefull logging and moves other logging calls outside loops, this number is reduced to 1000 calls. This results in cleaner logs and gives a small but measurable 0.15s speedup. The log size dropped from 900kb to 160kb. (Bitbake rev: d2677f084fe1d8846db77d89ef5e6ffb18dc171a) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>