Project

General

Profile

Actions
Copy link

Feature #16071

closed

update JEDEC vendors to JEP106BH

Added by Robert Mustacchi 11 days ago. Updated 1 day ago.

Status:
Closed
Priority:
Normal
Assignee:
Robert Mustacchi
Category:
lib - userland libraries
Start date:
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:
Gerrit CR:
3147
External Bug:

Description

This updates the libjedec vendor data to the latest JEDEC release from September 2023.

Actions
Copy link
 #1

Updated by Electric Monk 11 days ago

  • Gerrit CR set to 3147
Actions
Copy link
 #2

Updated by Robert Mustacchi 1 day ago

To test this I reran the various libjedec tests:

rm@romulus ~ $ /opt/util-tests/tests/libjedec_spd
TEST PASSED: Invalid SPD Data (zero length)
TEST PASSED: Unsupported SPD type (DDR SGRAM)
TEST PASSED: Unknown SPD type (0x42)
TEST PASSED: Bad DDR4 Revision (0x00)
TEST PASSED: Bad DDR4 Revision (0x54)
TEST PASSED: Bad DDR4 Revision (0x00)
TEST PASSED: Bad DDR5 Revision (0xb2)
TEST PASSED: Bad DDR5 Common Revision (0x00)
TEST PASSED: Catch incomplete errors
TEST PASSED: Non-fatal parsing errors
rm@romulus ~ $ echo $?
0
rm@romulus ~ $ /opt/util-tests/tests/libjedec_temp 
TEST PASSED: Operating temperature A2T
TEST PASSED: Operating temperature RT
TEST PASSED: Ambient temperature CT
TEST PASSED: Ambient temperature IOT
TEST PASSED: Ambient temperature A01T
TEST PASSED: Storage temperature ST
TEST PASSED: invalid temperature (42)
TEST PASSED: invalid temperature (INT32_MAX)
TEST PASSED: invalid temperature (UINT32_MAX)
rm@romulus ~ $ echo $?
0
rm@romulus ~ $ /opt/util-tests/tests/libjedec_test 
rm@romulus ~ $ echo $?
0

After that I sanity checked the SMBIOS consumer data:

rm@romulus ~ $ smbios -t17 | grep -i manuf
  Manufacturer: Samsung
  Module Manufacturer ID: Bank: 0x1 Vendor: 0xce (Samsung)
  Manufacturer: Samsung
  Module Manufacturer ID: Bank: 0x1 Vendor: 0xce (Samsung)
  Manufacturer: Samsung
  Module Manufacturer ID: Bank: 0x1 Vendor: 0xce (Samsung)
  Manufacturer: Samsung
  Module Manufacturer ID: Bank: 0x1 Vendor: 0xce (Samsung)
  Manufacturer: Samsung
  Module Manufacturer ID: Bank: 0x1 Vendor: 0xce (Samsung)
  Manufacturer: Samsung
  Module Manufacturer ID: Bank: 0x1 Vendor: 0xce (Samsung)
  Manufacturer: Samsung
  Module Manufacturer ID: Bank: 0x1 Vendor: 0xce (Samsung)
  Manufacturer: Samsung
  Module Manufacturer ID: Bank: 0x1 Vendor: 0xce (Samsung)
  Manufacturer: Samsung
  Module Manufacturer ID: Bank: 0x1 Vendor: 0xce (Samsung)
  Manufacturer: Samsung
  Module Manufacturer ID: Bank: 0x1 Vendor: 0xce (Samsung)
  Manufacturer: Samsung
  Module Manufacturer ID: Bank: 0x1 Vendor: 0xce (Samsung)
  Manufacturer: Samsung
  Module Manufacturer ID: Bank: 0x1 Vendor: 0xce (Samsung)
  Manufacturer: Samsung
  Module Manufacturer ID: Bank: 0x1 Vendor: 0xce (Samsung)
  Manufacturer: Samsung
  Module Manufacturer ID: Bank: 0x1 Vendor: 0xce (Samsung)
  Manufacturer: Samsung
  Module Manufacturer ID: Bank: 0x1 Vendor: 0xce (Samsung)
  Manufacturer: Samsung
  Module Manufacturer ID: Bank: 0x1 Vendor: 0xce (Samsung)

The module manufacturer encodes the JEDEC ID and matches the top-level manufacturer.

Actions
Copy link
 #3

Updated by Electric Monk 1 day ago

  • Status changed from New to Closed
  • % Done changed from 0 to 100

git commit b3783300013fa93b98278c901b855062f538f7e2

commit  b3783300013fa93b98278c901b855062f538f7e2
Author: Robert Mustacchi <rm@fingolfin.org>
Date:   2023-11-30T01:05:45.000Z

    16072 Want manuals for libjedec
    16071 update JEDEC vendors to JEP106BH
    Reviewed by: Andy Fiddaman <illumos@fiddaman.net>
    Approved by: Gordon Ross <gordon.w.ross@gmail.com>

Actions
Copy link

Also available in: Atom PDF