Skip to content

equeue: Added profiling reports to Travis #6378

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

Merged
merged 1 commit into from
Mar 21, 2018

Conversation

geky
Copy link
Contributor

@geky geky commented Mar 16, 2018

Description

Adds equeue profiling reports to Travis. Just like the code size report for LittleFS and inbound Astyle warning reports. You should be able to see the report in the travis-ci/events job.

The equeue make prof also measures code size and size of important structures. But it's hard to report more than one number without cluttering up the Travis statuses.

Pull request type

  • Fix
  • Refactor
  • New target
  • Feature
  • Breaking change

@geky geky force-pushed the events-travis-prof branch from 027a3f1 to d828fc0 Compare March 16, 2018 05:03
@cmonr
Copy link
Contributor

cmonr commented Mar 19, 2018

/morph build

@mbed-ci
Copy link

mbed-ci commented Mar 19, 2018

Build : SUCCESS

Build number : 1481
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/6378/

Triggering tests

/morph test
/morph uvisor-test
/morph export-build
/morph mbed2-build

@mbed-ci
Copy link

mbed-ci commented Mar 20, 2018

@mbed-ci
Copy link

mbed-ci commented Mar 20, 2018

@0xc0170 0xc0170 requested review from cmonr and adbridge March 20, 2018 10:37
- |
CURR=$(grep -o '[0-9]\+ cycles' prof | awk '{sum += $1} END {print sum}')
PREV=$(curl https://api.github.com/repos/$TRAVIS_REPO_SLUG/status/master \
| jq -re "select(.sha != \"$TRAVIS_COMMIT\")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

jq: Command-line JSON parser.

Good to know.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's not on all Ubuntu distros yet, but Travis guarantees it's their machines.

Copy link
Contributor

@cmonr cmonr Mar 21, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As long as Travis guarantees it 👍

Also, I take it that's one of the reasons why || echo 0 exists?

Copy link
Contributor Author

@geky geky Mar 21, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, that's a trick on my end. Since we're calculating a ratio below, if we fetch an old value of 0, we would divide by zero.

So for any errors, we just treat it as zero, and check it later.

@cmonr
Copy link
Contributor

cmonr commented Mar 20, 2018

Gonna wait on @adbridge

@cmonr cmonr merged commit 58f5795 into ARMmbed:master Mar 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants