Re: [PATCH] virsysinfo: Try reading DMI table

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello,

On Mon, Mar 18, 2024 at 11:30 PM <brett.holman@xxxxxxxxxxxxx> wrote:
From: Brett Holman <brett.holman@xxxxxxxxxxxxx>

The SMBIOS specification[1] includes RISC-V and mips, and some systems have
SMBIOS info. Attempt to read dmidecode and fall back to old behavior if that
fails.

[1] https://www.dmtf.org/sites/default/files/standards/documents/DSP0134_3.4.0.pdf

thank you for the patch, but I do not fully understand one thing - in the pdf you included there is only risc-v as a supported processor architecture (section 1.1, page 15) and not mips.

As I do not have knowledge in this - did you possibly include mips, because the newest specification also includes LoongArch?
If yes - could you amend the commit message to include the link to  the newest version [2] and to explain the relationship between mips x LoongArch? 




Signed-off-by: Brett Holman <brett.holman@xxxxxxxxxxxxx>
---
 src/util/virsysinfo.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/util/virsysinfo.c b/src/util/virsysinfo.c
index 36a861c53f..d8d660d694 100644
--- a/src/util/virsysinfo.c
+++ b/src/util/virsysinfo.c
@@ -1248,6 +1248,8 @@ virSysinfoRead(void)
 #elif !defined(WIN32) && \
     (defined(__x86_64__) || \
      defined(__i386__) || \
+     defined(__mips__) || \
+     defined(__riscv__) || \
      defined(__amd64__))
     return virSysinfoReadDMI();
 #else /* WIN32 || not supported arch */
--
2.43.0
_______________________________________________
Devel mailing list -- devel@xxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxx


Regards,
Kristina
_______________________________________________
Devel mailing list -- devel@xxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxx

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux