Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incorrect display of the order of lines with memory slots in the profiler. #660

Open
ghost opened this issue Jan 9, 2024 · 19 comments
Open

Comments

@ghost
Copy link

ghost commented Jan 9, 2024

The order of the memory slot lines in the profiler does not match the actual one.
Instead of:
Channel1-Dimm1 - empty
Channel1-Dimm0 - 32 GB
Channel0-Dimm1 - empty
Channel0-Dimm0 - 32 GB
displayed:
Channel1-Dimm1 - empty
Channel1-Dimm0 - empty
Channel0-Dimm1 - 32 GB
Channel0-Dimm0 - 32 GB
Using the bdmesg command, everything is determined correctly.
When I using OpenCore, the order memory slot lines are displayed in the profiler is also correct.
2024-01-09_10-16_CLOVERX64.EFI.log

@ghost
Copy link
Author

ghost commented Jan 10, 2024

Hardware configuration:
CPU: i7-12700kf Alder Lake
MB: Asrock Z690 PG Reptide
RAM: 2x32Gb DDR4 3200 Crucial Technology
GPU: Radeon RX5700 Sapphire Nitro+ 8Gb
SSD: 1Tb WD_BLACK SN850X NVME M2 (PCI-e 4.0) - Mac OS 14.2.1/Windos 11
SSD: 521gb Samsung 860 EVO (SATA-6 Gbit) -
WIFI/BT: bcm943602CS in FV-AC1200 PCI-E adapter
BIOS: v16.01
Display: ViewSonic 4K 60 Hz HDR via Display Port (VX3211-4K)

@jief666
Copy link
Contributor

jief666 commented Jan 10, 2024

Have you tried an older Clover to see if the order is correct ?

@SergeySlice
Copy link
Collaborator

Set SMBIOS->Trust=true

@SergeySlice
Copy link
Collaborator

Also set
SMBIOS->Memory->Channels=2
By default this value = 1

@ghost
Copy link
Author

ghost commented Jan 11, 2024

I can’t try Old Clover, since I don’t have it saved, and I couldn’t load either Ventura or Sonoma.
I also doubt that I would be able to compile it now.
Bat I changed my config file like this:

	<dict>
		<key>Memory</key>
		<dict>
			<key>Channels</key>
			<integer>2</integer>
		</dict>
		<key>Trust</key>
		<true/>
	</dict>

and nothing has changed:
2024-01-1121_04_06 jpg 038c5e5d36623c053ee9ed42e19f5629
2024-01-1121_05_27 jpg 195cf00d34ec28aa844da0511db12987
[preboot.log](https://github.com/CloverHackyColor/CloverBootloader/files/13907648/preboot.log
And this is from OpenCore:
Снимок экрана 2024-01-11 в 23 27 20
Снимок экрана 2024-01-11 в 23 29 06

@SergeySlice
Copy link
Collaborator

Test please this version
CLOVERX64.efi.zip
SMBIOS->Trust=true

@ghost
Copy link
Author

ghost commented Jan 13, 2024

Test please this version CLOVERX64.efi.zip SMBIOS->Trust=true
Again no change. Here are the results.
Снимок экрана 2024-01-13 в 14 55 54
Снимок экрана
[preboot.log](https://github.com/CloverHackyColor/CloverBootloader/files/13928611/preboot.log
And this is a SMBIOS dump before the patches.
smbios.txt.zip

@ghost
Copy link
Author

ghost commented Jan 14, 2024

I found my old preboot log, when the order of the dims was determined correctly still to Clover revision: 5119 (HEAD, commit a0948b2).
Motherboard Asus Z77-a Memory Samsung 4096MB 1333MHz
preboot.log

@jief666
Copy link
Contributor

jief666 commented Jan 14, 2024

It's not the same computer, so it's hard to compare and see what is now different in Clover (therefore find the bug).
Can you try to boot a Clover 5119 on your new computer ?

@ghost
Copy link
Author

ghost commented Jan 14, 2024

It's not the same computer, so it's hard to compare and see what is now different in Clover (therefore find the bug). Can you try to boot a Clover 5119 on your new computer ?

Yes, this is my old computer, but later versions of Clover displayed the same memory error as on the new one. Therefore, I did not update Clover on it, considering that the Mac OS would not be updated either. Now my son plays games previous years on it, since it is still quite powerful for this.

@jief666
Copy link
Contributor

jief666 commented Jan 15, 2024

Slice did some modification regarding that, Try this clover : CloverX64-2024-01-15-18-47-28-8697e93-jief.zip.

@ghost
Copy link
Author

ghost commented Jan 15, 2024

Slice did some modification regarding that, Try this clover : CloverX64-2024-01-15-18-47-28-8697e93-jief.zip.

Without changes.
preboot.log
Снимок экрана 2024-01-15 в 22 18 58
Снимок экрана 2024-01-15 в 22 20 09

@YBronst
Copy link

YBronst commented Jan 24, 2024

Due to glitches, I was forced to re-create my profile. I apologize for any inconvenience!

@jief666
Copy link
Contributor

jief666 commented Jan 27, 2024

@YBronst Sorry, I but I think this fix is beyond my current skill... Let's see what Slice will do.

@YBronst
Copy link

YBronst commented Feb 21, 2024

Slice writes that he cannot fix what he cannot reproduce.

@jief666
Copy link
Contributor

jief666 commented Feb 21, 2024

It's possible by asking to do a lot of test and generates a lot of debug log. But since it's only cosmetic, I'm guessing it's a lot of time for something we almost never look at (system information).
Sorry.

@YBronst
Copy link

YBronst commented Feb 22, 2024

It's possible by asking to do a lot of test and generates a lot of debug log. But since it's only cosmetic, I'm guessing it's a lot of time for something we almost never look at (system information). Sorry.

Yes, this is a cosmetic problem. I would just like to bring Clover to the level of OpenСore.

@jief666
Copy link
Contributor

jief666 commented Feb 23, 2024

Yes, me too. But the quick look I took made me realize I know almost nothing about SMBIOS etc. So, In the time I have, I can't do it sorry.

@YBronst
Copy link

YBronst commented Feb 24, 2024

Yes, me too. But the quick look I took made me realize I know almost nothing about SMBIOS etc. So, In the time I have, I can't do it sorry.

Could you get Slice to fix it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants