-
Notifications
You must be signed in to change notification settings - Fork 90
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Threads and events #42
base: master
Are you sure you want to change the base?
Conversation
…ons on command line
That's a big pull request! thanks for your work. Since github says "This pull request cannot be automatically merged." and you have done a lot of changes without much commit messages i have to review all the commits and cherry pick what's I understand / think it looks nice. Some changes looks like more a personal preference of yours than general issue, like changing default behaviour in 1869fd4 . If there is a chance that you can pick the fixes for real issues in smaller batches in other pull request it'll be helpful. It'll take time to review all these stuff. |
With benefit of hindsight, the batch/interactive default behavior change in 1869fd4 was unnecessary -- a simple start-up wrapper in bash would have sufficed. But that change points to a larger issue. You see, I needed to apply Bootchart to a wide variety of post-boot scenarios, and on fine time-scales. For such purposes, the 'batch' mode soon proved unhelpful, and in the later patches I dropped maintenance and testing of 'batch' mode altogether in favor of 'interactive'. You will also see many cosmetic changes, mostly forced by the new applications of the tool. If you're unable to take more than cherry-picked bug fixes, a scan of the first 20-40 patches might find all the fixes you can use. I've already pushed all bug fixes proper as early in the stack as was practical (with the help of some quilt scripting). |
This fixes a number of arithmetic bugs in the viewer, and extends functionality in various ways.
The two main extensions are:
I've rebased to as recent a commit on mainline as was possible with moderate effort.
Backward-compatibility with the bootchart.tgz format from standard distro collectors is preserved -- tested with Ubuntu.