{"action":"created","issue":{"url":"https://api.github.com/repos/ARMmbed/mbed-os/issues/2604","repository_url":"https://api.github.com/repos/ARMmbed/mbed-os","labels_url":"https://api.github.com/repos/ARMmbed/mbed-os/issues/2604/labels{/name}","comments_url":"https://api.github.com/repos/ARMmbed/mbed-os/issues/2604/comments","events_url":"https://api.github.com/repos/ARMmbed/mbed-os/issues/2604/events","html_url":"https://github.com/ARMmbed/mbed-os/pull/2604","id":174678067,"number":2604,"title":"Tools - Fix fill section size variation","user":{"login":"tung7970","id":9383560,"avatar_url":"https://avatars.githubusercontent.com/u/9383560?v=3","gravatar_id":"","url":"https://api.github.com/users/tung7970","html_url":"https://github.com/tung7970","followers_url":"https://api.github.com/users/tung7970/followers","following_url":"https://api.github.com/users/tung7970/following{/other_user}","gists_url":"https://api.github.com/users/tung7970/gists{/gist_id}","starred_url":"https://api.github.com/users/tung7970/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tung7970/subscriptions","organizations_url":"https://api.github.com/users/tung7970/orgs","repos_url":"https://api.github.com/users/tung7970/repos","events_url":"https://api.github.com/users/tung7970/events{/privacy}","received_events_url":"https://api.github.com/users/tung7970/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2016-09-02T04:25:36Z","updated_at":"2016-09-09T21:09:36Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ARMmbed/mbed-os/pulls/2604","html_url":"https://github.com/ARMmbed/mbed-os/pull/2604","diff_url":"https://github.com/ARMmbed/mbed-os/pull/2604.diff","patch_url":"https://github.com/ARMmbed/mbed-os/pull/2604.patch"},"body":"The linking order of object files affects the actual code placement,\r\nwhich in turn affects the size of fill section due to the number of\r\nzeros required to maintain appropriate data/code alignment may change.\r\nThis is observed when building on Mac and Linux host.\r\n\r\nexample: mbed compile -m K64F -t GCC_ARM (build 1)\r\n```\r\n+---------------------+-------+-------+-------+\r\n| Module              | .text | .data |  .bss |\r\n+---------------------+-------+-------+-------+\r\n| Fill                |   120 |     4 |  2381 |\r\n| Misc                | 28755 |  2216 |    84 |\r\n| features/frameworks |  4236 |    52 |   744 |\r\n| hal/common          |  2745 |     4 |   325 |\r\n| hal/targets         | 12116 |    12 |   200 |\r\n| rtos/rtos           |   119 |     4 |     0 |\r\n| rtos/rtx            |  5721 |    20 |  6786 |\r\n| Subtotals           | 53812 |  2312 | 10520 |\r\n+---------------------+-------+-------+-------+\r\n```\r\nexample: mbed compile -m K64F -t GCC_ARM (build 2)\r\n```\r\n+---------------------+-------+-------+-------+\r\n| Module              | .text | .data |  .bss |\r\n+---------------------+-------+-------+-------+\r\n| Fill                |   128 |     4 |  2381 |\r\n| Misc                | 28755 |  2216 |    84 |\r\n| features/frameworks |  4236 |    52 |   744 |\r\n| hal/common          |  2745 |     4 |   325 |\r\n| hal/targets         | 12116 |    12 |   200 |\r\n| rtos/rtos           |   119 |     4 |     0 |\r\n| rtos/rtx            |  5721 |    20 |  6786 |\r\n| Subtotals           | 53820 |  2312 | 10520 |\r\n+---------------------+-------+-------+-------+\r\n```\r\nThis patch fixes fill section size variation by sorting object\r\nfiles before passing to linker.\r\n\r\nSigned-off-by: Tony Wu <tung7970@gmail.com>"},"comment":{"url":"https://api.github.com/repos/ARMmbed/mbed-os/issues/comments/246040657","html_url":"https://github.com/ARMmbed/mbed-os/pull/2604#issuecomment-246040657","issue_url":"https://api.github.com/repos/ARMmbed/mbed-os/issues/2604","id":246040657,"user":{"login":"sg-","id":496002,"avatar_url":"https://avatars.githubusercontent.com/u/496002?v=3","gravatar_id":"","url":"https://api.github.com/users/sg-","html_url":"https://github.com/sg-","followers_url":"https://api.github.com/users/sg-/followers","following_url":"https://api.github.com/users/sg-/following{/other_user}","gists_url":"https://api.github.com/users/sg-/gists{/gist_id}","starred_url":"https://api.github.com/users/sg-/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sg-/subscriptions","organizations_url":"https://api.github.com/users/sg-/orgs","repos_url":"https://api.github.com/users/sg-/repos","events_url":"https://api.github.com/users/sg-/events{/privacy}","received_events_url":"https://api.github.com/users/sg-/received_events","type":"User","site_admin":false},"created_at":"2016-09-09T21:09:36Z","updated_at":"2016-09-09T21:09:36Z","body":"@mbed-bot: TEST\r\n\r\nHOST_OSES=ALL\r\nBUILD_TOOLCHAINS=ALL\r\nTARGETS=ALL"}}
{"actor":{"display_login":"sg-"}}