Activity log for bug #1597365

Date Who What changed Old value New value Message
2016-06-29 14:30:46 bugproxy bug added bug
2016-06-29 14:30:48 bugproxy tags architecture-ppc64le bugnameltc-143197 severity-critical targetmilestone-inin---
2016-06-29 14:30:49 bugproxy ubuntu: assignee Taco Screen team (taco-screen-team)
2016-06-29 14:30:59 bugproxy affects ubuntu powerpc-utils (Ubuntu)
2016-06-29 22:37:03 Steve Langasek powerpc-utils (Ubuntu): importance Undecided Medium
2016-06-29 22:37:03 Steve Langasek powerpc-utils (Ubuntu): status New Triaged
2016-06-29 22:37:03 Steve Langasek powerpc-utils (Ubuntu): assignee Taco Screen team (taco-screen-team) Adam Conrad (adconrad)
2016-07-22 23:27:02 Adam Conrad nominated for series Ubuntu Xenial
2016-07-22 23:27:02 Adam Conrad bug task added powerpc-utils (Ubuntu Xenial)
2016-07-23 22:38:16 Adam Conrad powerpc-utils (Ubuntu): status Triaged Fix Released
2016-09-30 09:30:11 bugproxy tags architecture-ppc64le bugnameltc-143197 severity-critical targetmilestone-inin--- architecture-ppc64le bugnameltc-143197 severity-critical targetmilestone-inin16042
2017-01-15 03:44:43 Steve Langasek description == Comment: #0 - Manvanthara B. Puttashankar - 2016-06-29 01:40:42 == ---Problem Description--- errinjct tool error while running EEH tests (powerpc-utils) ---uname output--- Linux ltciofvtr-s822l2-lp3 4.4.0-24-generic #43-Ubuntu SMP Wed Jun 8 19:25:36 UTC 2016 ppc64le ppc64le ppc64le GNU/Linux Machine Type = s822l ---Steps to Reproduce--- errinjct tool needs to pick two patch to address the below error errinjct eeh -v -f 6 -s net/eth1 -a 0xde000000 -m 0x0000FFFFFF800000 Injecting an ioa-bus-error with the following data: BUS ADDR: de000000 ADDR MASK: ffffff800000 CONFIG ADDR: 900000 PHB UNIT_ID: 800000020000135 FUNCTION: 6 Store to PCI Memory Address Space - inject an Adress Parity Error errinjct: Could not open RTAS error injection facility errinjct: librtas returned an unknown error code (-50331649) for function ioa-bus-error Parent bug: Bug 121866 : EEH via errinjct tool fails with librtas error in Ubuntu 16.04 (Shiner-T & Grand Canyon) errinjct issues will be solved with multiple patches 1 librtas + 2 errijnct. Based on suggestion from developer in Bug 121866 reporting this specific issue to powerpc-utils package , two patch needs to be picked by canonical. Note: have verified the powerpc-utils for 16.04.01 , the patch hasn't gone into yet. upstream patch fix link: https://github.com/nfont/powerpc-utils/tree/next/src/errinjct Userspace tool common name: errinjct The userspace tool has the following bit modes: ppc64le Userspace rpm: powerpc-utils == Comment: #1 - Manvanthara B. Puttashankar <mputtash@in.ibm.com> - 2016-06-29 01:48:02 == commits to be picked up commit 386a07c4dc6c09d35116dc6064f7a83d76f4415b Author: Nathan Fontenot <nfont@linux.vnet.ibm.com> Date: Tue May 24 16:32:39 2016 -0400 errinjct: Correct read_file() error checking When reading files from syfs the file length returned from a stat() call does not always match the actual length of the file. The read_file() routine should be checking for an error return from the read() call instead of checking that the amount read matches the file length returned from a stat() call. Signed-off-by: Nathan Fontenot <nfont@linux.vnet.ibm.com> commit 42b83884bfb51c0c8b23bf2949157430f8468caa Author: Nathan Fontenot <nfont@linux.vnet.ibm.com> Date: Tue May 24 16:31:37 2016 -0400 errinjct: Recognize -5 return from rtas Update the errinjct command to recognize the rturn code of -5 from rtas indicating that PCI error injection is not enabled. Signed-off-by: Nathan Fontenot <nfont@linux.vnet.ibm.com> commit ab848a8d5cc4c4b9f5faef1b31be3982fb7d4bf2 Author: Nathan Fontenot <nfont@linux.vnet.ibm.com> Date: Tue May 24 16:30:33 2016 -0400 errinjct: Correct config_addr parsing A recent patch (commit id: ccc9c72cec2a4) to move the errinjct function to using a common read_file() routine did not correctly update the get_config_addr_from_reg() function. This patch corrects this to return the proper config address. Signed-off-by: Nathan Fontenot <nfont@linux.vnet.ibm.com> [SRU Justification] The errinjct tool cannot run EEH tests on certain systems. This is a relatively low priority hardware enablement SRU (low priority because the tool in question is a test tool only). [Test case] 1. On a Shiner-T or Grand Canyon system, run the command: sudo errinjct eeh -v -f 6 -s net/enp1s0d0 -a 0x82100000 -m 0x0000FFFFFF800000 2. Verify that this returns an error. 3. Enable xenial-proposed, and upgrade powerpc-utils. 4. Rerun the errinjct command. Verify that it now succeeds. [Regression potential] Minimal; this is a test tool only so any regressions will be low impact, and the upstream changes that have been cherry-picked are reasonably limited in scope. == Comment: #0 - Manvanthara B. Puttashankar - 2016-06-29 01:40:42 == ---Problem Description--- errinjct tool error while running EEH tests (powerpc-utils) ---uname output--- Linux ltciofvtr-s822l2-lp3 4.4.0-24-generic #43-Ubuntu SMP Wed Jun 8 19:25:36 UTC 2016 ppc64le ppc64le ppc64le GNU/Linux Machine Type = s822l ---Steps to Reproduce---  errinjct tool needs to pick two patch to address the below error errinjct eeh -v -f 6 -s net/eth1 -a 0xde000000 -m 0x0000FFFFFF800000 Injecting an ioa-bus-error with the following data: BUS ADDR: de000000 ADDR MASK: ffffff800000 CONFIG ADDR: 900000 PHB UNIT_ID: 800000020000135 FUNCTION: 6 Store to PCI Memory Address Space - inject an Adress Parity Error errinjct: Could not open RTAS error injection facility errinjct: librtas returned an unknown error code (-50331649) for function ioa-bus-error Parent bug: Bug 121866 : EEH via errinjct tool fails with librtas error in Ubuntu 16.04 (Shiner-T & Grand Canyon) errinjct issues will be solved with multiple patches 1 librtas + 2 errijnct. Based on suggestion from developer in Bug 121866 reporting this specific issue to powerpc-utils package , two patch needs to be picked by canonical. Note: have verified the powerpc-utils for 16.04.01 , the patch hasn't gone into yet. upstream patch fix link: https://github.com/nfont/powerpc-utils/tree/next/src/errinjct Userspace tool common name: errinjct The userspace tool has the following bit modes: ppc64le Userspace rpm: powerpc-utils == Comment: #1 - Manvanthara B. Puttashankar <mputtash@in.ibm.com> - 2016-06-29 01:48:02 == commits to be picked up commit 386a07c4dc6c09d35116dc6064f7a83d76f4415b Author: Nathan Fontenot <nfont@linux.vnet.ibm.com> Date: Tue May 24 16:32:39 2016 -0400     errinjct: Correct read_file() error checking     When reading files from syfs the file length returned from a stat()     call does not always match the actual length of the file. The read_file()     routine should be checking for an error return from the read() call     instead of checking that the amount read matches the file length returned     from a stat() call.     Signed-off-by: Nathan Fontenot <nfont@linux.vnet.ibm.com> commit 42b83884bfb51c0c8b23bf2949157430f8468caa Author: Nathan Fontenot <nfont@linux.vnet.ibm.com> Date: Tue May 24 16:31:37 2016 -0400     errinjct: Recognize -5 return from rtas     Update the errinjct command to recognize the rturn code of -5 from     rtas indicating that PCI error injection is not enabled.     Signed-off-by: Nathan Fontenot <nfont@linux.vnet.ibm.com> commit ab848a8d5cc4c4b9f5faef1b31be3982fb7d4bf2 Author: Nathan Fontenot <nfont@linux.vnet.ibm.com> Date: Tue May 24 16:30:33 2016 -0400     errinjct: Correct config_addr parsing     A recent patch (commit id: ccc9c72cec2a4) to move the errinjct function to     using a common read_file() routine did not correctly update the     get_config_addr_from_reg() function. This patch corrects this to return the     proper config address.     Signed-off-by: Nathan Fontenot <nfont@linux.vnet.ibm.com>
2017-01-15 03:45:08 Steve Langasek powerpc-utils (Ubuntu Xenial): importance Undecided Medium
2017-01-15 03:45:14 Steve Langasek powerpc-utils (Ubuntu Xenial): status New In Progress
2017-01-15 03:45:18 Steve Langasek powerpc-utils (Ubuntu Xenial): assignee Steve Langasek (vorlon)
2017-01-18 23:15:26 Brian Murray powerpc-utils (Ubuntu Xenial): status In Progress Fix Committed
2017-01-18 23:15:29 Brian Murray bug added subscriber Ubuntu Stable Release Updates Team
2017-01-18 23:15:31 Brian Murray bug added subscriber SRU Verification
2017-01-18 23:15:46 Brian Murray tags architecture-ppc64le bugnameltc-143197 severity-critical targetmilestone-inin16042 architecture-ppc64le bugnameltc-143197 severity-critical targetmilestone-inin16042 verification-needed
2017-01-23 12:03:39 Breno Leitão tags architecture-ppc64le bugnameltc-143197 severity-critical targetmilestone-inin16042 verification-needed architecture-ppc64le bugnameltc-143197 severity-critical targetmilestone-inin16042 verification-done
2017-01-26 17:40:50 Brian Murray tags architecture-ppc64le bugnameltc-143197 severity-critical targetmilestone-inin16042 verification-done architecture-ppc64le bugnameltc-143197 severity-critical targetmilestone-inin16042 verification-needed
2017-02-09 07:32:06 Launchpad Janitor powerpc-utils (Ubuntu Xenial): status Fix Committed Fix Released