| /openbmc/phosphor-logging/extensions/openpower-pels/registry/schema/ |
| H A D | registry_example.json | 479b6927 Tue Aug 17 16:34:59 CDT 2021 Matt Spinler <spinler@us.ibm.com> PEL: Remove 'no_vpd_for_fru' maintenance procedure
This procedure was being used in a callout when the location code could
not be found on an inventory item. The code was changed to just not add
a callout in this case.
This was done for the following reasons:
1) There's no expected reason that the inventory would be missing a
location code for a valid inventory path.
2) There wasn't a way to give a hint to the end user about what the
callout should be in that case. Usually maintenance procedures should
have steps one can take to still do the appropriate replacement.
3) Because of 1), this case indicates a bad inventory path passed in by
the caller. Theoretically there is a way to have a 'bmc_code'
callout, but that wouldn't be appropriate to add in this log since
the intent of this one is for another problem.
3a) There is no way to create a new error log from inside a PEL section,
and I don't think this one case warrants coming up with a way to do
so.
4) The inventory path in question is still being added into a UserData
section so that development can debug it.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I0757080f04942807cc34029d8667387db6b959fe
|
| H A D | schema.json | 479b6927 Tue Aug 17 16:34:59 CDT 2021 Matt Spinler <spinler@us.ibm.com> PEL: Remove 'no_vpd_for_fru' maintenance procedure
This procedure was being used in a callout when the location code could
not be found on an inventory item. The code was changed to just not add
a callout in this case.
This was done for the following reasons:
1) There's no expected reason that the inventory would be missing a
location code for a valid inventory path.
2) There wasn't a way to give a hint to the end user about what the
callout should be in that case. Usually maintenance procedures should
have steps one can take to still do the appropriate replacement.
3) Because of 1), this case indicates a bad inventory path passed in by
the caller. Theoretically there is a way to have a 'bmc_code'
callout, but that wouldn't be appropriate to add in this log since
the intent of this one is for another problem.
3a) There is no way to create a new error log from inside a PEL section,
and I don't think this one case warrants coming up with a way to do
so.
4) The inventory path in question is still being added into a UserData
section so that development can debug it.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I0757080f04942807cc34029d8667387db6b959fe
|
| /openbmc/phosphor-logging/test/openpower-pels/ |
| H A D | fru_identity_test.cpp | 479b6927 Tue Aug 17 16:34:59 CDT 2021 Matt Spinler <spinler@us.ibm.com> PEL: Remove 'no_vpd_for_fru' maintenance procedure
This procedure was being used in a callout when the location code could
not be found on an inventory item. The code was changed to just not add
a callout in this case.
This was done for the following reasons:
1) There's no expected reason that the inventory would be missing a
location code for a valid inventory path.
2) There wasn't a way to give a hint to the end user about what the
callout should be in that case. Usually maintenance procedures should
have steps one can take to still do the appropriate replacement.
3) Because of 1), this case indicates a bad inventory path passed in by
the caller. Theoretically there is a way to have a 'bmc_code'
callout, but that wouldn't be appropriate to add in this log since
the intent of this one is for another problem.
3a) There is no way to create a new error log from inside a PEL section,
and I don't think this one case warrants coming up with a way to do
so.
4) The inventory path in question is still being added into a UserData
section so that development can debug it.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I0757080f04942807cc34029d8667387db6b959fe
|
| H A D | src_callout_test.cpp | 479b6927 Tue Aug 17 16:34:59 CDT 2021 Matt Spinler <spinler@us.ibm.com> PEL: Remove 'no_vpd_for_fru' maintenance procedure
This procedure was being used in a callout when the location code could
not be found on an inventory item. The code was changed to just not add
a callout in this case.
This was done for the following reasons:
1) There's no expected reason that the inventory would be missing a
location code for a valid inventory path.
2) There wasn't a way to give a hint to the end user about what the
callout should be in that case. Usually maintenance procedures should
have steps one can take to still do the appropriate replacement.
3) Because of 1), this case indicates a bad inventory path passed in by
the caller. Theoretically there is a way to have a 'bmc_code'
callout, but that wouldn't be appropriate to add in this log since
the intent of this one is for another problem.
3a) There is no way to create a new error log from inside a PEL section,
and I don't think this one case warrants coming up with a way to do
so.
4) The inventory path in question is still being added into a UserData
section so that development can debug it.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I0757080f04942807cc34029d8667387db6b959fe
|
| H A D | registry_test.cpp | 479b6927 Tue Aug 17 16:34:59 CDT 2021 Matt Spinler <spinler@us.ibm.com> PEL: Remove 'no_vpd_for_fru' maintenance procedure
This procedure was being used in a callout when the location code could
not be found on an inventory item. The code was changed to just not add
a callout in this case.
This was done for the following reasons:
1) There's no expected reason that the inventory would be missing a
location code for a valid inventory path.
2) There wasn't a way to give a hint to the end user about what the
callout should be in that case. Usually maintenance procedures should
have steps one can take to still do the appropriate replacement.
3) Because of 1), this case indicates a bad inventory path passed in by
the caller. Theoretically there is a way to have a 'bmc_code'
callout, but that wouldn't be appropriate to add in this log since
the intent of this one is for another problem.
3a) There is no way to create a new error log from inside a PEL section,
and I don't think this one case warrants coming up with a way to do
so.
4) The inventory path in question is still being added into a UserData
section so that development can debug it.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I0757080f04942807cc34029d8667387db6b959fe
|
| H A D | src_test.cpp | 479b6927 Tue Aug 17 16:34:59 CDT 2021 Matt Spinler <spinler@us.ibm.com> PEL: Remove 'no_vpd_for_fru' maintenance procedure
This procedure was being used in a callout when the location code could
not be found on an inventory item. The code was changed to just not add
a callout in this case.
This was done for the following reasons:
1) There's no expected reason that the inventory would be missing a
location code for a valid inventory path.
2) There wasn't a way to give a hint to the end user about what the
callout should be in that case. Usually maintenance procedures should
have steps one can take to still do the appropriate replacement.
3) Because of 1), this case indicates a bad inventory path passed in by
the caller. Theoretically there is a way to have a 'bmc_code'
callout, but that wouldn't be appropriate to add in this log since
the intent of this one is for another problem.
3a) There is no way to create a new error log from inside a PEL section,
and I don't think this one case warrants coming up with a way to do
so.
4) The inventory path in question is still being added into a UserData
section so that development can debug it.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I0757080f04942807cc34029d8667387db6b959fe
|
| H A D | pel_manager_test.cpp | 479b6927 Tue Aug 17 16:34:59 CDT 2021 Matt Spinler <spinler@us.ibm.com> PEL: Remove 'no_vpd_for_fru' maintenance procedure
This procedure was being used in a callout when the location code could
not be found on an inventory item. The code was changed to just not add
a callout in this case.
This was done for the following reasons:
1) There's no expected reason that the inventory would be missing a
location code for a valid inventory path.
2) There wasn't a way to give a hint to the end user about what the
callout should be in that case. Usually maintenance procedures should
have steps one can take to still do the appropriate replacement.
3) Because of 1), this case indicates a bad inventory path passed in by
the caller. Theoretically there is a way to have a 'bmc_code'
callout, but that wouldn't be appropriate to add in this log since
the intent of this one is for another problem.
3a) There is no way to create a new error log from inside a PEL section,
and I don't think this one case warrants coming up with a way to do
so.
4) The inventory path in question is still being added into a UserData
section so that development can debug it.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I0757080f04942807cc34029d8667387db6b959fe
|
| /openbmc/phosphor-logging/extensions/openpower-pels/ |
| H A D | pel_values.cpp | 479b6927 Tue Aug 17 16:34:59 CDT 2021 Matt Spinler <spinler@us.ibm.com> PEL: Remove 'no_vpd_for_fru' maintenance procedure
This procedure was being used in a callout when the location code could
not be found on an inventory item. The code was changed to just not add
a callout in this case.
This was done for the following reasons:
1) There's no expected reason that the inventory would be missing a
location code for a valid inventory path.
2) There wasn't a way to give a hint to the end user about what the
callout should be in that case. Usually maintenance procedures should
have steps one can take to still do the appropriate replacement.
3) Because of 1), this case indicates a bad inventory path passed in by
the caller. Theoretically there is a way to have a 'bmc_code'
callout, but that wouldn't be appropriate to add in this log since
the intent of this one is for another problem.
3a) There is no way to create a new error log from inside a PEL section,
and I don't think this one case warrants coming up with a way to do
so.
4) The inventory path in question is still being added into a UserData
section so that development can debug it.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I0757080f04942807cc34029d8667387db6b959fe
|
| H A D | src.cpp | 479b6927 Tue Aug 17 16:34:59 CDT 2021 Matt Spinler <spinler@us.ibm.com> PEL: Remove 'no_vpd_for_fru' maintenance procedure
This procedure was being used in a callout when the location code could
not be found on an inventory item. The code was changed to just not add
a callout in this case.
This was done for the following reasons:
1) There's no expected reason that the inventory would be missing a
location code for a valid inventory path.
2) There wasn't a way to give a hint to the end user about what the
callout should be in that case. Usually maintenance procedures should
have steps one can take to still do the appropriate replacement.
3) Because of 1), this case indicates a bad inventory path passed in by
the caller. Theoretically there is a way to have a 'bmc_code'
callout, but that wouldn't be appropriate to add in this log since
the intent of this one is for another problem.
3a) There is no way to create a new error log from inside a PEL section,
and I don't think this one case warrants coming up with a way to do
so.
4) The inventory path in question is still being added into a UserData
section so that development can debug it.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I0757080f04942807cc34029d8667387db6b959fe
|