[SRU] Add Xilinx support patch to enable latest Xilinx tools

Bug #2019147 reported by Talha Can Havadar
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
freeipmi (Ubuntu)
Fix Released
Undecided
Unassigned
Jammy
Fix Released
Undecided
Unassigned
Kinetic
Fix Released
Undecided
Unassigned
Lunar
Fix Released
Undecided
Unassigned

Bug Description

[ Impact ]

 * It prevents the users of Xilinx boards to use latest versions of Xilinx SDK tools.

 * Since Ubuntu is officially supports Xilinx Kria boards, we need to add Xilinx OEM support into freeipmi to enable latest Xilinx SDK tools on jammy, kinetic and lunar.

[ Test Plan ]

 * Xilinx OEM data parsing is not supported on current version of freeipmi. So when you try to run `sudo ipmi-fru --fru-file /sys/bus/i2c/devices/1-0050/eeprom --interpret-oem-data` on a Xilinx Kria board you wont be able to get the oem specific data.

 * When the patch is applied, you must be able to see the Xilinx OEM data parsed successfully as in following output.
```
ubuntu@kria:~$ sudo ipmi-fru --fru-file /sys/bus/i2c/devices/1-0050/eeprom --interpret-oem-data
FRU Inventory From File: /sys/bus/i2c/devices/1-0050/eeprom

FRU Board Manufacturing Date/Time: 12/20/22 - 12:51:00
FRU Board Manufacturer: XILINX
FRU Board Product Name: SMK-K24-XCL2G
FRU Board Serial Number: 51281A22C82S
FRU Board Part Number: 05128-01
FRU FRU File ID: 00h
FRU Board Custom Info: A
FRU Board Custom Info: 10h EEh 00h 00h 00h 00h 00h 00h
FRU Board Custom Info: 66h 27h 20h 4Bh 4Eh 48h EFh 9Bh 35h 47h 1Ah 02h A2h 63h 66h B5h

FRU DC Load Output Number: 1
FRU DC Load Nominal Voltage: 5000 mV
FRU DC Load Spec'd Minimum Voltage: 4500 mV
FRU DC Load Spec'd Maximum Voltage: 5500 mV
FRU DC Load Spec'd Ripple and Noise pk-pk: 100 mV
FRU DC Load Minimum Current Load: 0 mA
FRU DC Load Maximum Current Load: 2000 mA

FRU OEM Manufacturer ID: Xilinx, Inc. (10DAh)
FRU OEM MAC Version: DUT - MAC (31h)
FRU OEM MAC ID 0: 00:0a:35:18:76:08

FRU OEM Manufacturer ID: Xilinx, Inc. (10DAh)
FRU OEM Memory: QSPI:512Mb
FRU OEM Memory: eMMC:32GB
FRU OEM Memory: PSDDR4:2GB
FRU OEM Memory: PLDDR4:None
```

[ Where problems could occur ]

 * Since the change is only on OEM data parsing part of `ipmi-fru` tool it could only break `--interpret-oem-data` option for Xilinx boards.

 * This change must not introduce a regression since the Xilinx OEM support will be newly added via this patch.

 * Plus, patch is already in stable branch of the upstream project so it is accepted by the maintainer as a stable patch.

 * [racb] The common code being changed looks for a new manufacturer_id of IPMI_IANA_ENTERPRISE_ID_XILINX (defined to be 4314) and follows a different path in that case, including a (slightly) early exit that skips the output of generic hex data. Presumably this deliberately changes behaviour to provide richer output. Anyone relying on the previous hex output behaviour will be broken, but this seems deliberate and acceptable as part of hardware enablement.

 * [racb] If some other board identifies the same way, then behaviour on that board will chage, too.

[ Other Info ]

 * This patch request is created after evaluating LP #2015825.

Revision history for this message
Talha Can Havadar (tchavadar) wrote :

Since kinetic and jammy has the same version of freeipmi (1.6.9-2), this debdiff should be enough to apply this patch on both of these series.

Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "1-1.6.9-3.debdiff" seems to be a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. If the attachment isn't a patch, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are member of the ~ubuntu-sponsors, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issue please contact him.]

Revision history for this message
Talha Can Havadar (tchavadar) wrote :

Patch is already landed on 1.6.10-2 on mantic-proposed thanks to fantonifabio.

Revision history for this message
Lucas Kanashiro (lucaskanashiro) wrote :

Hi Talha,

Since you are proposing stable updates you need to update your bug description accordingly, following the SRU template: https://wiki.ubuntu.com/StableReleaseUpdates#SRU_Bug_Template . Could you please do that?

Changed in freeipmi (Ubuntu):
status: New → Fix Released
Revision history for this message
Talha Can Havadar (tchavadar) wrote (last edit ):

Hi Lucas,

Of course, I will update the description according to template. (Updated)

description: updated
Revision history for this message
Talha Can Havadar (tchavadar) wrote :

Hi Lucas, is there any update about the process? I see that the patch is already landed on mantic-release.

Revision history for this message
Athos Ribeiro (athos-ribeiro) wrote :

Hi Talha,

I am raising the work priority on this one so we can sponsor these patches soon. I suppose you will need SRUs for lunar, kinetic, and jammy, right? If so, would you mind having all the debdiffs ready (or filing 3 different MPs for this package's git repository)?

Thanks

Changed in freeipmi (Ubuntu Jammy):
status: New → Triaged
Changed in freeipmi (Ubuntu Kinetic):
status: New → Triaged
Changed in freeipmi (Ubuntu Lunar):
status: New → Triaged
tags: added: bitesize server-todo
Revision history for this message
Talha Can Havadar (tchavadar) wrote :

Hi Athos,

Thank you for your comment. Since the versions in kinetic and jammy are same 1.6.9-2 (no change in between), we can use the debdiff I provide named as jammy_and_kinetic-1.debdiff.

For Lunar, we can use lunar-1.debdiff

Best Regards

Revision history for this message
Fantu (fantonifabio) wrote :

version for jammy and kinetic should be 1.6.9-2ubuntu1 instead 1.6.9-3 and lunar should be 1.6.10-1ubuntu1 instead 1.6.10-1xlnx1. even if probably can be simply changed by the sponsor before upload
I can't sponsor them because I don't have upload permission on ubuntu repo

Revision history for this message
Talha Can Havadar (tchavadar) wrote :

Hi Fantu, I have just modified versions in changelog file in debdiffs. I hope somebody from ubuntu can help this.

Revision history for this message
Athos Ribeiro (athos-ribeiro) wrote (last edit ):

Hi Talha,

I can sponsor this one.

- Please update the maintainer field in d/control. You can do that by running the "update-maintaner" tool from the ubuntu-dev-tools package.

- Please, either break the changelog line or abbreviate the file name to "d/p/add-initial-support-for-xilinx.patch" so lintian won't throw a warning due to the long line.

- Please provide 3 debdiffs (lunar, kinetic, jammy) with correct series names and versions in d/changelog.

- It would be nice to include DEP3 headers to the patch file for context: https://dep-team.pages.debian.net/deps/dep3/ (e.g, upstream bug, ubuntu bug, origin).

- You can refer to https://wiki.ubuntu.com/SecurityTeam/UpdatePreparation#Update_the_packaging to set proper versions in d/changelog

Revision history for this message
Talha Can Havadar (tchavadar) wrote :
Revision history for this message
Talha Can Havadar (tchavadar) wrote :
Revision history for this message
Talha Can Havadar (tchavadar) wrote :
Revision history for this message
Talha Can Havadar (tchavadar) wrote :

Hi Athos,

I have just uploaded the patches for 3 different release. Can you check if they are ok or not?

Revision history for this message
Athos Ribeiro (athos-ribeiro) wrote (last edit ):

Hi Talha, thanks for addressing the requested changes.

The version field still needs fixing. Check the examples in https://wiki.ubuntu.com/SecurityTeam/UpdatePreparation#Update_the_packaging. The idea here is to avoid upgrade path issues. For instance, by keeping the same versions in jammy and kinetic, a user upgrading from jammy to kinetic would not get his freeipmi package upgraded since the version in the new series would not be greater.

You want to change those string to the following versions:

lunar: 1.6.10-1ubuntu0.1
kinetic: 1.6.9-2ubuntu0.22.10.1
jammy: 1.6.9-2ubuntu0.22.04.1

It would also be nice to adjust the changelog message to set it to an imperative form to match other messages in the changelog.

* d/p/add-initial-support-for-xilinx.patch: support Xilinx (LP: #2015825)

Still for the changelog, I just realized that the reference in there points to a different (private) bug. Instead, you want to refer to this bug, with the SRU template.

* d/p/add-initial-support-for-xilinx.patch: support Xilinx (LP: #2019147)

In https://wiki.ubuntu.com/StableReleaseUpdates, you will find:

"Also make sure that [...] There is a reference to the SRU bug number in the changelog, using the 'LP: #NNNNNN' convention. Only public bugs should be referenced in the changelog."

Finally, you could have kept the commit message in the patch files, which would add more context to the patches. Then, you could add the remaining DE3 headers after that description. This is just a nitpick, no need to change that though.

Overall, the changes LGTM.

I also pushed them to a PPA to ensure the builds are sane in all supported platforms: https://launchpad.net/~athos-ribeiro/+archive/ubuntu/freeipmi-xilinx/+packages.

Next time, it would be nice to do the same to show the sponsors you checked those builds.

Thank you!

Revision history for this message
Talha Can Havadar (tchavadar) wrote :
Revision history for this message
Talha Can Havadar (tchavadar) wrote :
Revision history for this message
Talha Can Havadar (tchavadar) wrote :
Revision history for this message
Talha Can Havadar (tchavadar) wrote (last edit ):

Hi Athos,

Thank you very much for detailed answer. I updated the patches, I have also added the original commit information before DEP3 header, But I am not sure if this is the correct way or not.

I have also published them in my private ppa https://launchpad.net/~tchavadar/+archive/ubuntu/junk/+packages

Still struggling with versions as you see. Thanks again for your time!

Revision history for this message
Athos Ribeiro (athos-ribeiro) wrote :

Thanks for addressing those requests, Talha!

I did adjust the changelog entry to fit in a single line since it was a single entry.

I also removed the Description and Author fields from the DEP3 headers, they were redundant due to Subject and From, respectively.

Uploaded. Please refer to https://wiki.ubuntu.com/StableReleaseUpdates to continue with the SRU process. Thanks for the patches :)

Robie Basak (racb)
description: updated
Revision history for this message
Robie Basak (racb) wrote :

I added some notes to "Where problems could occur".

Please note that the new output format will be "set in stone" by this SRU, and changing it again would probably tip the balance in regression risk to that SRU being refused. Please make sure the output is exactly what you want, and ensure that you won't need to change it again, before marking this SRU as verified.

description: updated
Changed in freeipmi (Ubuntu Lunar):
status: Triaged → Fix Committed
tags: added: verification-needed verification-needed-lunar
Revision history for this message
Robie Basak (racb) wrote : Please test proposed package

Hello Talha, or anyone else affected,

Accepted freeipmi into lunar-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/freeipmi/1.6.10-1ubuntu0.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-lunar to verification-done-lunar. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-lunar. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Revision history for this message
Robie Basak (racb) wrote :

Unsubscribing ~ubuntu-sponsors as there is nothing left to sponsor.

Changed in freeipmi (Ubuntu Kinetic):
status: Triaged → Fix Committed
tags: added: verification-needed-kinetic
Revision history for this message
Robie Basak (racb) wrote :

Hello Talha, or anyone else affected,

Accepted freeipmi into kinetic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/freeipmi/1.6.9-2ubuntu0.22.10.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-kinetic to verification-done-kinetic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-kinetic. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in freeipmi (Ubuntu Jammy):
status: Triaged → Fix Committed
tags: added: verification-needed-jammy
Revision history for this message
Robie Basak (racb) wrote :

Hello Talha, or anyone else affected,

Accepted freeipmi into jammy-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/freeipmi/1.6.9-2ubuntu0.22.04.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-jammy to verification-done-jammy. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-jammy. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Revision history for this message
Ubuntu SRU Bot (ubuntu-sru-bot) wrote : Autopkgtest regression report (freeipmi/1.6.10-1ubuntu0.1)

All autopkgtests for the newly accepted freeipmi (1.6.10-1ubuntu0.1) for lunar have finished running.
The following regressions have been reported in tests triggered by the package:

nut/2.8.0-7 (amd64)

Please visit the excuses page listed below and investigate the failures, proceeding afterwards as per the StableReleaseUpdates policy regarding autopkgtest regressions [1].

https://people.canonical.com/~ubuntu-archive/proposed-migration/lunar/update_excuses.html#freeipmi

[1] https://wiki.ubuntu.com/StableReleaseUpdates#Autopkgtest_Regressions

Thank you!

Revision history for this message
Talha Can Havadar (tchavadar) wrote :

Hi Robie, thank you for your comments. I have informed the vendor about the situation. They will also test the package to see if this exactly what they want. I have also tested the one on jammy and it looks ok but I will wait for them.

About this regression failure on lunar, I think it is not related to package but I dont have knowledge about `nut`

Here is the failure from the logs:
```
AssertionError: False is not true : Could not find pidfile '/run/nut/dummy-ups-dummy-dev1.pid'
```

I will investigate this to get more information.

Revision history for this message
Athos Ribeiro (athos-ribeiro) wrote :

Hi Talha,

the test failure seemed to be a flaky test. I re-triggered it and it is now green.

Once you (and the vendor) are happy with testing the package in -proposed and it passes your [test plan] described above, please change the tags in this bug as described above so the SRU process can continue its course (the package will not be released to -updates if the verification tag is not updated here).

tags: added: verification-done-jammy
removed: verification-needed-jammy
Revision history for this message
Talha Can Havadar (tchavadar) wrote :
Download full text (8.0 KiB)

Tested on jammy, package version was 1.6.9-2ubuntu0.22.04.1

Following command is executed on a Xilinx Kria KV260 Device

```
sudo ipmi-fru --fru-file=/sys/devices/platform/axi/ff030000.i2c/i2c-1/1-0050/eeprom --interpret-oem-data --debug
```

Output as following:
```
FRU Inventory From File: /sys/devices/platform/axi/ff030000.i2c/i2c-1/1-0050/eeprom
=====================================================
Common Header
=====================================================
[ HEX DUMP ..... ] = HEX[ 8B]
[ 01h 00h 00h 01h 00h 0Dh 00h F1h ]
=====================================================
Common Header
=====================================================
[ 1h] = format_version[ 4b]
[ 0h] = reserved[ 4b]
[ 0h] = internal_use_area_starting_offset[ 8b]
[ 0h] = chassis_info_area_starting_offset[ 8b]
[ 1h] = board_info_area_starting_offset[ 8b]
[ 0h] = product_info_area_starting_offset[ 8b]
[ Dh] = multirecord_area_starting_offset[ 8b]
[ 0h] = pad[ 8b]
[ F1h] = checksum[ 8b]
=====================================================
Board Info Header
=====================================================
[ 1h] = format_version[ 4b]
[ 0h] = reserved[ 4b]
[ Ch] = info_area_length[ 8b]
=====================================================
Board Info Area
=====================================================
[ HEX DUMP ..... ] = HEX[96B]
[ 01h 0Ch 00h 66h C3h D3h C6h 58h ]
[ 49h 4Ch 49h 4Eh 58h D0h 53h 4Dh ]
[ 4Bh 2Dh 4Bh 32h 36h 2Dh 58h 43h ]
[ 4Ch 32h 47h 00h 00h 00h D0h 58h ]
[ 46h 4Ch 31h 54h 33h 31h 48h 31h ]
[ 46h 49h 43h 00h 00h 00h 00h C9h ]
[ 35h 30h 35h 37h 2Dh 30h 31h 20h ]
[ 20h 01h 00h C8h 31h 00h 00h 00h ]
[ 00h 00h 00h 00h 08h 10h EEh 00h ]
[ 00h 00h 00h 00h 00h 10h D8h 89h ]
[ DDh EEh 30h FDh 44h F3h B1h C0h ]
[ 5Eh D7h 0Ch 6Ah E3h 44h C1h 33h ]

  FRU Board Manufacturing Date/Time: 05/21/22 - 13:58:00
=====================================================
Type/Length Field Header
=====================================================
[ HEX DUMP ..... ] = HEX[ 1B]
[ C6h ]
  FRU Board Manufacturer: XILINX
=====================================================
Type/Length Field Header
=====================================================
[ HEX DUMP ..... ] = HEX[ 1B]
[ D0h ]
  FRU Board Product Name: SMK-K26-XCL2G
=====================================================
Type/Length Field Header
=====================================================
[ HEX DUMP ..... ] = HEX[ 1B]
[ D0h ]
  FRU Board Serial Number: XFL1T31H1FIC
=====================================================
Type/Length Field Header
=====================================================
[ HEX DUMP ..... ] = HEX[ 1B]
[ C9h ]
  FRU Board Part Number: 5057-01
=====================================================
Type/Length Field Header
=====================================================
[ HEX DUMP ..... ] = HEX[ 1B]
[ 01h ]
  FRU FRU File ID: 00h
=====================================================
Type/Length Field Header
=====================================================
[ HEX DUMP ..... ] = HEX[ 1B]
[ C8h ]
  FRU Board Custom Info: 1
=====================================================
Type/Length Field Header
=====================================================
[ HEX DUM...

Read more...

Revision history for this message
Talha Can Havadar (tchavadar) wrote :
Download full text (8.9 KiB)

1.6.9-2ubuntu0.22.10.1 on kinetic-proposed verified. Result is the same as jammy which is expected.

```
ubuntu@kria:~$ apt-cache policy freeipmi
freeipmi:
  Installed: 1.6.9-2ubuntu0.22.10.1
  Candidate: 1.6.9-2ubuntu0.22.10.1
  Version table:
 *** 1.6.9-2ubuntu0.22.10.1 500
        500 http://ports.ubuntu.com/ubuntu-ports kinetic-proposed/universe arm64 Packages
        100 /var/lib/dpkg/status
     1.6.9-2 500
        500 http://ports.ubuntu.com/ubuntu-ports jammy/universe arm64 Packages
ubuntu@kria:~$ sudo ipmi-fru --fru-file=/sys/devices/platform/axi/ff030000.i2c/i2c-1/1-0050/eeprom --interpret-oem-data --debug
FRU Inventory From File: /sys/devices/platform/axi/ff030000.i2c/i2c-1/1-0050/eeprom
=====================================================
Common Header
=====================================================
[ HEX DUMP ..... ] = HEX[ 8B]
[ 01h 00h 00h 01h 00h 0Dh 00h F1h ]
=====================================================
Common Header
=====================================================
[ 1h] = format_version[ 4b]
[ 0h] = reserved[ 4b]
[ 0h] = internal_use_area_starting_offset[ 8b]
[ 0h] = chassis_info_area_starting_offset[ 8b]
[ 1h] = board_info_area_starting_offset[ 8b]
[ 0h] = product_info_area_starting_offset[ 8b]
[ Dh] = multirecord_area_starting_offset[ 8b]
[ 0h] = pad[ 8b]
[ F1h] = checksum[ 8b]
=====================================================
Board Info Header
=====================================================
[ 1h] = format_version[ 4b]
[ 0h] = reserved[ 4b]
[ Ch] = info_area_length[ 8b]
=====================================================
Board Info Area
=====================================================
[ HEX DUMP ..... ] = HEX[96B]
[ 01h 0Ch 00h 66h C3h D3h C6h 58h ]
[ 49h 4Ch 49h 4Eh 58h D0h 53h 4Dh ]
[ 4Bh 2Dh 4Bh 32h 36h 2Dh 58h 43h ]
[ 4Ch 32h 47h 00h 00h 00h D0h 58h ]
[ 46h 4Ch 31h 54h 33h 31h 48h 31h ]
[ 46h 49h 43h 00h 00h 00h 00h C9h ]
[ 35h 30h 35h 37h 2Dh 30h 31h 20h ]
[ 20h 01h 00h C8h 31h 00h 00h 00h ]
[ 00h 00h 00h 00h 08h 10h EEh 00h ]
[ 00h 00h 00h 00h 00h 10h D8h 89h ]
[ DDh EEh 30h FDh 44h F3h B1h C0h ]
[ 5Eh D7h 0Ch 6Ah E3h 44h C1h 33h ]

  FRU Board Manufacturing Date/Time: 05/21/22 - 13:58:00
=====================================================
Type/Length Field Header
=====================================================
[ HEX DUMP ..... ] = HEX[ 1B]
[ C6h ]
  FRU Board Manufacturer: XILINX
=====================================================
Type/Length Field Header
=====================================================
[ HEX DUMP ..... ] = HEX[ 1B]
[ D0h ]
  FRU Board Product Name: SMK-K26-XCL2G
=====================================================
Type/Length Field Header
=====================================================
[ HEX DUMP ..... ] = HEX[ 1B]
[ D0h ]
  FRU Board Serial Number: XFL1T31H1FIC
=====================================================
Type/Length Field Header
=====================================================
[ HEX DUMP ..... ] = HEX[ 1B]
[ C9h ]
  FRU Board Part Number: 5057-01...

Read more...

tags: added: verification-done-kinetic
removed: verification-needed-kinetic
Revision history for this message
Talha Can Havadar (tchavadar) wrote :
Download full text (8.9 KiB)

1.6.10-1ubuntu0.1 on lunar-proposed verified. Result is the same as jammy and kinetic which is expected.

```
ubuntu@kria:~$ apt-cache policy freeipmi
freeipmi:
  Installed: 1.6.10-1ubuntu0.1
  Candidate: 1.6.10-1ubuntu0.1
  Version table:
 *** 1.6.10-1ubuntu0.1 100
        100 http://ports.ubuntu.com/ubuntu-ports lunar-proposed/universe arm64 Packages
        100 /var/lib/dpkg/status
     1.6.9-2 500
        500 http://ports.ubuntu.com/ubuntu-ports jammy/universe arm64 Packages
ubuntu@kria:~$ sudo ipmi-fru --fru-file=/sys/devices/platform/axi/ff030000.i2c/i2c-1/1-0050/eeprom --interpret-oem-data --debug
FRU Inventory From File: /sys/devices/platform/axi/ff030000.i2c/i2c-1/1-0050/eeprom
=====================================================
Common Header
=====================================================
[ HEX DUMP ..... ] = HEX[ 8B]
[ 01h 00h 00h 01h 00h 0Dh 00h F1h ]
=====================================================
Common Header
=====================================================
[ 1h] = format_version[ 4b]
[ 0h] = reserved[ 4b]
[ 0h] = internal_use_area_starting_offset[ 8b]
[ 0h] = chassis_info_area_starting_offset[ 8b]
[ 1h] = board_info_area_starting_offset[ 8b]
[ 0h] = product_info_area_starting_offset[ 8b]
[ Dh] = multirecord_area_starting_offset[ 8b]
[ 0h] = pad[ 8b]
[ F1h] = checksum[ 8b]
=====================================================
Board Info Header
=====================================================
[ 1h] = format_version[ 4b]
[ 0h] = reserved[ 4b]
[ Ch] = info_area_length[ 8b]
=====================================================
Board Info Area
=====================================================
[ HEX DUMP ..... ] = HEX[96B]
[ 01h 0Ch 00h 66h C3h D3h C6h 58h ]
[ 49h 4Ch 49h 4Eh 58h D0h 53h 4Dh ]
[ 4Bh 2Dh 4Bh 32h 36h 2Dh 58h 43h ]
[ 4Ch 32h 47h 00h 00h 00h D0h 58h ]
[ 46h 4Ch 31h 54h 33h 31h 48h 31h ]
[ 46h 49h 43h 00h 00h 00h 00h C9h ]
[ 35h 30h 35h 37h 2Dh 30h 31h 20h ]
[ 20h 01h 00h C8h 31h 00h 00h 00h ]
[ 00h 00h 00h 00h 08h 10h EEh 00h ]
[ 00h 00h 00h 00h 00h 10h D8h 89h ]
[ DDh EEh 30h FDh 44h F3h B1h C0h ]
[ 5Eh D7h 0Ch 6Ah E3h 44h C1h 33h ]

  FRU Board Manufacturing Date/Time: 05/21/22 - 13:58:00
=====================================================
Type/Length Field Header
=====================================================
[ HEX DUMP ..... ] = HEX[ 1B]
[ C6h ]
  FRU Board Manufacturer: XILINX
=====================================================
Type/Length Field Header
=====================================================
[ HEX DUMP ..... ] = HEX[ 1B]
[ D0h ]
  FRU Board Product Name: SMK-K26-XCL2G
=====================================================
Type/Length Field Header
=====================================================
[ HEX DUMP ..... ] = HEX[ 1B]
[ D0h ]
  FRU Board Serial Number: XFL1T31H1FIC
=====================================================
Type/Length Field Header
=====================================================
[ HEX DUMP ..... ] = HEX[ 1B]
[ C9h ]
  FRU Board Part Number: 5057-01
=========...

Read more...

tags: added: verification-done verification-done-lunar
removed: verification-needed verification-needed-lunar
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package freeipmi - 1.6.10-1ubuntu0.1

---------------
freeipmi (1.6.10-1ubuntu0.1) lunar; urgency=medium

  * d/p/add-initial-support-for-xilinx.patch: enable Xilinx OEM data parsing
    in ipmi-fru (LP: #2019147)

 -- Talha Can Havadar <email address hidden> Tue, 30 May 2023 10:35:37 +0200

Changed in freeipmi (Ubuntu Lunar):
status: Fix Committed → Fix Released
Revision history for this message
Robie Basak (racb) wrote : Update Released

The verification of the Stable Release Update for freeipmi has completed successfully and the package is now being released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package freeipmi - 1.6.9-2ubuntu0.22.10.1

---------------
freeipmi (1.6.9-2ubuntu0.22.10.1) kinetic; urgency=medium

  * d/p/add-initial-support-for-xilinx.patch: enable Xilinx OEM data parsing
    in ipmi-fru (LP: #2019147)

 -- Talha Can Havadar <email address hidden> Tue, 30 May 2023 10:28:30 +0200

Changed in freeipmi (Ubuntu Kinetic):
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package freeipmi - 1.6.9-2ubuntu0.22.04.1

---------------
freeipmi (1.6.9-2ubuntu0.22.04.1) jammy; urgency=medium

  * d/p/add-initial-support-for-xilinx.patch: enable Xilinx OEM data parsing
    in ipmi-fru (LP: #2019147)

 -- Talha Can Havadar <email address hidden> Tue, 30 May 2023 10:11:32 +0200

Changed in freeipmi (Ubuntu Jammy):
status: Fix Committed → Fix Released
Revision history for this message
Fantu (fantonifabio) wrote (last edit ):

as reported in https://bugs.launchpad.net/ubuntu/+source/freeipmi/+bug/2025177 this patch (https://github.com/chu11/freeipmi-mirror/commit/aa699dbdb6b7bf3efada662b8126132e36a3bd05) should be added also to backports for missed header
1.6.11-1 include it

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.