Activity log for bug #1582181

Date Who What changed Old value New value Message
2016-05-16 11:04:52 Alexandru Avadanii bug added bug
2016-05-16 11:04:52 Alexandru Avadanii attachment added AArch64-cpuinfo-Remove-redundant-cpu-caps-loop.patch https://bugs.launchpad.net/bugs/1582181/+attachment/4663771/+files/AArch64-cpuinfo-Remove-redundant-cpu-caps-loop.patch
2016-05-16 12:34:09 Ubuntu Foundations Team Bug Bot tags patch
2016-05-16 12:34:16 Ubuntu Foundations Team Bug Bot bug added subscriber Ubuntu Review Team
2017-09-08 18:45:48 Launchpad Janitor lshw (Ubuntu): status New Confirmed
2018-02-27 13:00:05 Alexandru Avadanii bug added subscriber dann frazier
2018-02-27 15:05:07 dann frazier bug added subscriber Ike Panhc
2018-03-21 06:05:35 Ike Panhc lshw (Ubuntu): status Confirmed In Progress
2018-03-21 06:05:38 Ike Panhc lshw (Ubuntu): assignee Ike Panhc (ikepanhc)
2018-03-21 06:06:30 Ike Panhc nominated for series Ubuntu Bionic
2018-03-21 06:06:30 Ike Panhc nominated for series Ubuntu Artful
2018-03-21 06:06:30 Ike Panhc nominated for series Ubuntu Xenial
2018-03-21 08:19:07 Ike Panhc description lshw on AArch64 hardware is painfully slow. This affects both lshw in current Ubuntu releases and vanilla upstream. For a 48 core node, cpuinfo parsing added up to 30 seconds (8 lines per core in /proc/cpuinfo add up to 384 lines to parse). For a 96 core node, parsing took up to 5 minutes (!). I think the problem was introduced by [1], and can be summarized as: - CPU capabilities should be added only to the current CPU core, and NOT to all previous CPU cores parsed. My suggestion is dropping the loop in [1], thus calling the <addHint> and <describeCapability> only for currentcpu. I put together a small patch (basically removing the for loop in question) at [2] (or see attachement), which should be applied on top of version "02.16-2ubuntu1.3" from Ubuntu Trusty 14.04. After applying the patch at [2], parsing for the above system (48 cores) takes less than 1 second (instead of 30s), with the exact same results ... [1] https://github.com/lyonel/lshw/commit/beb89de5a3c10449fe73f1c77b2486d868e5bc9a#diff-f4010714738fa4cdd5999499579da2b3R217 [2] http://paste.ubuntu.com/16456620/ # lsb_release -rd Description: Ubuntu 14.04.4 LTS Release: 14.04 BR, Alex lshw on AArch64 hardware is painfully slow. This affects both lshw in current Ubuntu releases and vanilla upstream. For a 48 core node, cpuinfo parsing added up to 30 seconds (8 lines per core in /proc/cpuinfo add up to 384 lines to parse). For a 96 core node, parsing took up to 5 minutes (!). I think the problem was introduced by [1], and can be summarized as: - CPU capabilities should be added only to the current CPU core,   and NOT to all previous CPU cores parsed. My suggestion is dropping the loop in [1], thus calling the <addHint> and <describeCapability> only for currentcpu. I put together a small patch (basically removing the for loop in question) at [2] (or see attachement), which should be applied on top of version "02.16-2ubuntu1.3" from Ubuntu Trusty 14.04. After applying the patch at [2], parsing for the above system (48 cores) takes less than 1 second (instead of 30s), with the exact same results ... [1] https://github.com/lyonel/lshw/commit/beb89de5a3c10449fe73f1c77b2486d868e5bc9a#diff-f4010714738fa4cdd5999499579da2b3R217 [2] http://paste.ubuntu.com/16456620/ # lsb_release -rd Description: Ubuntu 14.04.4 LTS Release: 14.04 BR, Alex [Impact] lshw takes too long time on parsing /proc/cpuinfo on aarch64 system It takes minutes on 96cores and almost 10min on 224cores system. [Test Case] `time sudo lshw` and it shall be less then 15sec [Regression Potential] This patch only affect code for aarch64. No regression possible on other aarch64 and has been tested on aarch64 [Other Info] Patch has been merged into upstream branch. https://github.com/lyonel/lshw/commit/20cda77239e8604e798b87a0441e694edb0214d1
2018-03-21 08:20:16 Ike Panhc description lshw on AArch64 hardware is painfully slow. This affects both lshw in current Ubuntu releases and vanilla upstream. For a 48 core node, cpuinfo parsing added up to 30 seconds (8 lines per core in /proc/cpuinfo add up to 384 lines to parse). For a 96 core node, parsing took up to 5 minutes (!). I think the problem was introduced by [1], and can be summarized as: - CPU capabilities should be added only to the current CPU core,   and NOT to all previous CPU cores parsed. My suggestion is dropping the loop in [1], thus calling the <addHint> and <describeCapability> only for currentcpu. I put together a small patch (basically removing the for loop in question) at [2] (or see attachement), which should be applied on top of version "02.16-2ubuntu1.3" from Ubuntu Trusty 14.04. After applying the patch at [2], parsing for the above system (48 cores) takes less than 1 second (instead of 30s), with the exact same results ... [1] https://github.com/lyonel/lshw/commit/beb89de5a3c10449fe73f1c77b2486d868e5bc9a#diff-f4010714738fa4cdd5999499579da2b3R217 [2] http://paste.ubuntu.com/16456620/ # lsb_release -rd Description: Ubuntu 14.04.4 LTS Release: 14.04 BR, Alex [Impact] lshw takes too long time on parsing /proc/cpuinfo on aarch64 system It takes minutes on 96cores and almost 10min on 224cores system. [Test Case] `time sudo lshw` and it shall be less then 15sec [Regression Potential] This patch only affect code for aarch64. No regression possible on other aarch64 and has been tested on aarch64 [Other Info] Patch has been merged into upstream branch. https://github.com/lyonel/lshw/commit/20cda77239e8604e798b87a0441e694edb0214d1 lshw on AArch64 hardware is painfully slow. This affects both lshw in current Ubuntu releases and vanilla upstream. For a 48 core node, cpuinfo parsing added up to 30 seconds (8 lines per core in /proc/cpuinfo add up to 384 lines to parse). For a 96 core node, parsing took up to 5 minutes (!). I think the problem was introduced by [1], and can be summarized as: - CPU capabilities should be added only to the current CPU core,   and NOT to all previous CPU cores parsed. My suggestion is dropping the loop in [1], thus calling the <addHint> and <describeCapability> only for currentcpu. I put together a small patch (basically removing the for loop in question) at [2] (or see attachement), which should be applied on top of version "02.16-2ubuntu1.3" from Ubuntu Trusty 14.04. After applying the patch at [2], parsing for the above system (48 cores) takes less than 1 second (instead of 30s), with the exact same results ... [1] https://github.com/lyonel/lshw/commit/beb89de5a3c10449fe73f1c77b2486d868e5bc9a#diff-f4010714738fa4cdd5999499579da2b3R217 [2] http://paste.ubuntu.com/16456620/ # lsb_release -rd Description: Ubuntu 14.04.4 LTS Release: 14.04 BR, Alex [Impact] lshw takes too long time on parsing /proc/cpuinfo on aarch64 system It takes minutes on 96cores and almost 10min on 224cores system. [Test Case] `time sudo lshw` and it shall be less then 15sec [Regression Potential] This patch only affect code for aarch64. Lowest regression rick on other arch and has been tested on aarch64 [Other Info] Patch has been merged into upstream branch. https://github.com/lyonel/lshw/commit/20cda77239e8604e798b87a0441e694edb0214d1
2018-03-21 08:21:05 Ike Panhc description lshw on AArch64 hardware is painfully slow. This affects both lshw in current Ubuntu releases and vanilla upstream. For a 48 core node, cpuinfo parsing added up to 30 seconds (8 lines per core in /proc/cpuinfo add up to 384 lines to parse). For a 96 core node, parsing took up to 5 minutes (!). I think the problem was introduced by [1], and can be summarized as: - CPU capabilities should be added only to the current CPU core,   and NOT to all previous CPU cores parsed. My suggestion is dropping the loop in [1], thus calling the <addHint> and <describeCapability> only for currentcpu. I put together a small patch (basically removing the for loop in question) at [2] (or see attachement), which should be applied on top of version "02.16-2ubuntu1.3" from Ubuntu Trusty 14.04. After applying the patch at [2], parsing for the above system (48 cores) takes less than 1 second (instead of 30s), with the exact same results ... [1] https://github.com/lyonel/lshw/commit/beb89de5a3c10449fe73f1c77b2486d868e5bc9a#diff-f4010714738fa4cdd5999499579da2b3R217 [2] http://paste.ubuntu.com/16456620/ # lsb_release -rd Description: Ubuntu 14.04.4 LTS Release: 14.04 BR, Alex [Impact] lshw takes too long time on parsing /proc/cpuinfo on aarch64 system It takes minutes on 96cores and almost 10min on 224cores system. [Test Case] `time sudo lshw` and it shall be less then 15sec [Regression Potential] This patch only affect code for aarch64. Lowest regression rick on other arch and has been tested on aarch64 [Other Info] Patch has been merged into upstream branch. https://github.com/lyonel/lshw/commit/20cda77239e8604e798b87a0441e694edb0214d1 lshw on AArch64 hardware is painfully slow. This affects both lshw in current Ubuntu releases and vanilla upstream. For a 48 core node, cpuinfo parsing added up to 30 seconds (8 lines per core in /proc/cpuinfo add up to 384 lines to parse). For a 96 core node, parsing took up to 5 minutes (!). I think the problem was introduced by [1], and can be summarized as: - CPU capabilities should be added only to the current CPU core,   and NOT to all previous CPU cores parsed. My suggestion is dropping the loop in [1], thus calling the <addHint> and <describeCapability> only for currentcpu. I put together a small patch (basically removing the for loop in question) at [2] (or see attachement), which should be applied on top of version "02.16-2ubuntu1.3" from Ubuntu Trusty 14.04. After applying the patch at [2], parsing for the above system (48 cores) takes less than 1 second (instead of 30s), with the exact same results ... [1] https://github.com/lyonel/lshw/commit/beb89de5a3c10449fe73f1c77b2486d868e5bc9a#diff-f4010714738fa4cdd5999499579da2b3R217 [2] http://paste.ubuntu.com/16456620/ # lsb_release -rd Description: Ubuntu 14.04.4 LTS Release: 14.04 BR, Alex [Impact] lshw takes too long time on parsing /proc/cpuinfo on aarch64 system It takes minutes on 96cores and almost 10min on 224cores system. [Test Case] `time sudo lshw` and it shall be less then 15sec [Regression Potential] This patch only modifies codes for aarch64. Lowest regression rick on other arch and has been tested on aarch64 [Other Info] Patch has been merged into upstream branch. https://github.com/lyonel/lshw/commit/20cda77239e8604e798b87a0441e694edb0214d1
2018-03-21 17:58:43 dann frazier bug task added lshw (Ubuntu Artful)
2018-03-21 17:59:01 dann frazier bug task added lshw (Ubuntu Xenial)
2018-03-21 17:59:07 dann frazier bug task added lshw (Ubuntu Bionic)
2018-03-21 17:59:57 dann frazier description lshw on AArch64 hardware is painfully slow. This affects both lshw in current Ubuntu releases and vanilla upstream. For a 48 core node, cpuinfo parsing added up to 30 seconds (8 lines per core in /proc/cpuinfo add up to 384 lines to parse). For a 96 core node, parsing took up to 5 minutes (!). I think the problem was introduced by [1], and can be summarized as: - CPU capabilities should be added only to the current CPU core,   and NOT to all previous CPU cores parsed. My suggestion is dropping the loop in [1], thus calling the <addHint> and <describeCapability> only for currentcpu. I put together a small patch (basically removing the for loop in question) at [2] (or see attachement), which should be applied on top of version "02.16-2ubuntu1.3" from Ubuntu Trusty 14.04. After applying the patch at [2], parsing for the above system (48 cores) takes less than 1 second (instead of 30s), with the exact same results ... [1] https://github.com/lyonel/lshw/commit/beb89de5a3c10449fe73f1c77b2486d868e5bc9a#diff-f4010714738fa4cdd5999499579da2b3R217 [2] http://paste.ubuntu.com/16456620/ # lsb_release -rd Description: Ubuntu 14.04.4 LTS Release: 14.04 BR, Alex [Impact] lshw takes too long time on parsing /proc/cpuinfo on aarch64 system It takes minutes on 96cores and almost 10min on 224cores system. [Test Case] `time sudo lshw` and it shall be less then 15sec [Regression Potential] This patch only modifies codes for aarch64. Lowest regression rick on other arch and has been tested on aarch64 [Other Info] Patch has been merged into upstream branch. https://github.com/lyonel/lshw/commit/20cda77239e8604e798b87a0441e694edb0214d1 [Impact] lshw takes too long time on parsing /proc/cpuinfo on aarch64 system It takes minutes on 96cores and almost 10min on 224cores system. [Test Case] `time sudo lshw` and it shall be less then 15sec [Regression Potential] This patch only modifies codes for aarch64. Lowest regression rick on other arch and has been tested on aarch64 [Other Info] Patch has been merged into upstream branch. https://github.com/lyonel/lshw/commit/20cda77239e8604e798b87a0441e694edb0214d1
2018-03-21 19:58:24 Launchpad Janitor lshw (Ubuntu Bionic): status In Progress Fix Released
2018-03-22 17:15:52 Ike Panhc description [Impact] lshw takes too long time on parsing /proc/cpuinfo on aarch64 system It takes minutes on 96cores and almost 10min on 224cores system. [Test Case] `time sudo lshw` and it shall be less then 15sec [Regression Potential] This patch only modifies codes for aarch64. Lowest regression rick on other arch and has been tested on aarch64 [Other Info] Patch has been merged into upstream branch. https://github.com/lyonel/lshw/commit/20cda77239e8604e798b87a0441e694edb0214d1 [Impact] lshw takes too long time on parsing /proc/cpuinfo on aarch64 system It takes minutes on 96cores and almost 10min on 224cores system. [Test Case] `time lshw` and it shall be less then 15sec [Regression Potential] This patch only modifies codes for aarch64. Lowest regression rick on other arch and has been tested on aarch64 [Other Info] Patch has been merged into upstream branch. https://github.com/lyonel/lshw/commit/20cda77239e8604e798b87a0441e694edb0214d1
2018-03-22 19:26:48 Brian Murray lshw (Ubuntu Artful): status New Fix Committed
2018-03-22 19:26:52 Brian Murray bug added subscriber Ubuntu Stable Release Updates Team
2018-03-22 19:26:55 Brian Murray bug added subscriber SRU Verification
2018-03-22 19:27:00 Brian Murray tags patch patch verification-needed verification-needed-artful
2018-03-22 19:29:02 Brian Murray lshw (Ubuntu Xenial): status New Fix Committed
2018-03-22 19:29:08 Brian Murray tags patch verification-needed verification-needed-artful patch verification-needed verification-needed-artful verification-needed-xenial
2018-03-26 03:33:24 Ike Panhc tags patch verification-needed verification-needed-artful verification-needed-xenial patch verification-done-xenial verification-needed verification-needed-artful
2018-03-26 03:33:31 Ike Panhc lshw (Ubuntu Artful): assignee Ike Panhc (ikepanhc)
2018-03-26 03:33:32 Ike Panhc lshw (Ubuntu Xenial): assignee Ike Panhc (ikepanhc)
2018-03-26 04:12:36 Ike Panhc tags patch verification-done-xenial verification-needed verification-needed-artful patch verification-done verification-done-artful verification-done-xenial
2018-03-29 21:43:53 Brian Murray removed subscriber Ubuntu Stable Release Updates Team
2018-03-29 21:44:38 Launchpad Janitor lshw (Ubuntu Xenial): status Fix Committed Fix Released
2018-03-29 21:53:54 Launchpad Janitor lshw (Ubuntu Artful): status Fix Committed Fix Released