On 22.03.19 10:04, Burak Atalay wrote:
Hello Mr. Kiszka,I have confirmed that we are on commit f743631213a1 without modifications. I went into the source code in the file "/usr/local/libexec/jailhouse/jailhouse-hardware-check", inside the "elif cpu_vendor == 'AuthenticAMD':" branch, in the last loop I see that there are two variables named "n", one in the outer loop and one in the inner loop. Changing the name of the variable "n" in the inner loop fixes the error in the "jailhouse hardware check" command. So the following lines: for n in range(num_filter_regs): smi_freg = mmio.read64(0x60 + (n << 3)) becomes for nn in range(num_filter_regs): smi_freg = mmio.read64(0x60 + (nn << 3)) After this modification, "jailhouse hardware check" command gives the following output:
Good catch! Would you like to write a patch so that this gets fixed for everyone? Thanks in advance, Jan -- Siemens AG, Corporate Technology, CT RDA IOT SES-DE Corporate Competence Center Embedded Linux -- You received this message because you are subscribed to the Google Groups "Jailhouse" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/d/optout.
