processor information is missing in json reports

Bug #1839667 reported by Sheila Miguez
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Checkbox Provider - Base
Fix Released
Critical
Sylvain Pineau

Bug Description

This was discovered in lp:1839638.

The old-style xml reports parsed dmi info into a "processor" element as well as created a "PROCESSOR" device element in the list of devices. The PROCESSOR device entries in the old submissions contained information such as

      {
        "category": "PROCESSOR",
        "subproduct_id": null,
        "product": "Hi1616",
        "vendor": "Hisilicon",
        "product_id": null,
        "bus": "dmi",
        "path": "/devices/virtual/dmi/id/processor",
        "vendor_id": null,
        "driver": null,
        "subvendor_id": null
      },

C3 needs to have the vendor and product information to display to users. That information is not available in the "architecture" element in the old or new submission formats.

Let's agree on a new format that contains all of the information we need. Can you add the vendor and product and bus information to the "architecture" element?

Related branches

Jeff Lane  (bladernr)
Changed in checkbox-ng:
importance: Undecided → High
status: New → Confirmed
Revision history for this message
Sheila Miguez (codersquid) wrote :
Revision history for this message
Sheila Miguez (codersquid) wrote :

From Jeff's bug on hexr,

https://certification.canonical.com/hardware/201907-27213/submission/147411/

dmidecode has:

Processor Information
 Socket Designation: CPU01
 Type: Central Processor
 Family: ARM
 Manufacturer: Hisilicon
 ID: 10 D0 1F 48 00 00 00 00
 Signature: Implementor 0x48, Variant 0x1, Architecture 15, Part 0xd01, Revision 0
 Version: Hisilicon 1620
 Voltage: 0.9 V
 External Clock: 100 MHz
 Max Speed: 2600 MHz
 Current Speed: 2600 MHz
 Status: Populated, Enabled
 Upgrade: Unknown
 L1 Cache Handle: 0x0023
 L2 Cache Handle: 0x0024
 L3 Cache Handle: 0x0025
 Serial Number: To be filled by O.E.M.
 Asset Tag: To be filled by O.E.M.
 Part Number: To be filled by O.E.M.
 Core Count: 64
 Core Enabled: 64
 Thread Count: 64
 Characteristics:
  64-bit capable
  Multi-Core
  Execute Protection
  Enhanced Virtualization
  Power/Performance Control

submission.json has

    "processor": {
        "bogomips": "200",
        "cache": "-1",
        "count": "128",
        "model": "aarch64",
        "model_number": "0x1",
        "model_revision": "0",
        "model_version": "8",
        "other": "fp asimd evtstrm aes pmull sha1 sha2 crc32 atomics fphp asimdhp cpuid asimdrdm jscvt fcma dcpop asimddp",
        "platform": "aarch64",
        "speed": "2600",
        "type": "aarch64"
    },

Jeff linked to an old submission https://certification.canonical.com/hardware/201812-26732/submission/135998/ that has the data from dmidecode in the parsed json. We generated the json from the old checkbox library. In the json that was generated from the old checkbox support tools, it has both the processor element and an element in the devices list that is categorized as PROCESSOR. I pasted the old example up above when I opened hte bug.

basically we need the version information from dmidecode with the processor information, i.e. Version: Hisilicon 1620.

Revision history for this message
Sheila Miguez (codersquid) wrote :

Here is a dump of data from dmidecode.DmidecodeParser using the DmiResult class from there. I would like if this data could be included in the submission.json, though I am uncertain where to go to add it.

(I do not mean for the data to be shaped the same way, the dump is just an example of the content that the parser can generate that includes processor info.)

Revision history for this message
Sheila Miguez (codersquid) wrote :

Considering that the submission.json has the raw dmidecode attachment, I could have c3 use the dmidecode parser from checkbox_support since we have that as a dependency already.

Changed in checkbox-ng:
importance: High → Critical
Changed in checkbox-ng:
assignee: nobody → Sylvain Pineau (sylvain-pineau)
milestone: none → 1.6.0
Changed in checkbox-ng:
status: Confirmed → In Progress
affects: checkbox-ng → plainbox-provider-checkbox
Changed in plainbox-provider-checkbox:
milestone: 1.6.0 → none
status: In Progress → Fix Committed
milestone: none → 0.49.0
Changed in plainbox-provider-checkbox:
status: Fix Committed → Fix Released
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.