Looks at CPUID leaves it should not

Question

Looks at CPUID leaves it should not

DemiMarie opened this issue 2 years ago · 5 comments

A few months ago, @fosslinux reported that this code package looped forever under Xen. This has since been fixed, but it turns out that this code does not check if TOPOEXT is exposed before using it. Feel free to close if this has since been fixed.

Answer 1 · 2022-08-24T21:56:38.000Z

I see that the loop was fixed in 2a5efe8 but this code should not even try to query the cache values unless TOPOEXT is enabled.

Answer 2 · 2022-08-25T06:56:59.000Z

@DemiMarie Do you happen to have any link to documentation for that?

Answer 3 · 2022-08-25T23:35:35.000Z

@klauspost Not offhand, but it would be in the AMD programmer manuals.

Answer 4 · 2022-08-26T11:10:26.000Z

Added & Released

Answer 5 · 2022-08-26T19:06:04.000Z

Thanks @klauspost!