standard processor-specific block structure does not match SMBIOS spec

[kimonberlin]

The "standard processor-specific block structure" described in section 2.2 does not match the "processor-specific block" defined in section 7.45.1 of SMBIOS 3.5.0.
The fields should be "block length", "processor type", and "processor-specific data".

[changab]

Thanks for catching this, I will correct it in riscv-smbios spec.

[jiegec]

Ping @changab

[changab]

Wow, this ticket was opened in 2022 Jan. I am no longer working on RISC-V as I joined AMD. Could you please help to send the PR?

[jiegec]

There seemed to be a newer (and incompatible) definition at https://github.com/riscv-non-isa/riscv-brs/blob/main/smbios.adoc, which one should be used?

[changab]

Hi @jiegec, I am not quite clear. Which definition do you refer? Could you please point me to that definition?

[jiegec]

Hi @jiegec, I am not quite clear. Which definition do you refer? Could you please point me to that definition?

The SMBIOS Type 44 Processor-Specific Data definition seemed different.

[changab]

Are you saying the definition of SMBIOS type 44 is different than RISC-V SMBIOS Type44 processor-specific data?
The one defined in SMBIOS spec is the SMBIOS type 44 structure (the header). The one defined here is the processor-specific data which should be located at the offset 02h in table 130 in SMBIOS section 7.45.1.

[jiegec]

RISC-V Processor-specific Block Structure in this repo has revision v0.10, but in riscv-brs, the revision is v1.00.

[changab]

Ah I see. I think https://github.com/riscv-non-isa/riscv-brs/blob/main/smbios.adoc is newer. I think they created riscv-brs later.

[jiegec]

However, the SMBIOS spec still refers to this repo, which is the older one.

[kimonberlin]

What is the new URL to put in the SMBIOS spec?

[jiegec]

I think we can archive this repo and add links to the new specification.

[changab]

However, the SMBIOS spec still refers to this repo, which is the older one.

ok, I will send the ECR to SMBIOS WG to update this change.

[changab]

What is the new URL to put in the SMBIOS spec?

@kimonberlin, this is the one: https://github.com/riscv-non-isa/riscv-brs/blob/main/smbios.adoc.
Please help to update this link on SMBIOS spec 7.45.2.1.

Thank you
Abner