Clover General discussion

[Jief_Machak]

 

You can compile yourself if you switch to the branch "0.7.3branch".

[Matgen84]

29 minutes ago, Jief_Machak said:

Here is my first version with an update of the OpenCore component to 0.7.3.

 

Could you try ?CloverX64-2021-09-23-15-42-13-d8755c9-jief.zip

 

From USB key and HDD: stuck at "Automatic Boot in 0 seconds"

 

Note: when you have few minutes, please remove serdeluik's commits from master branch in CloverHackyColor/OpencorePKG. Thanks.

2021-09-23_13-08_CloverX64-2021-09-23-15-42-13-d8755c9-jief.efi.log

[Jief_Machak]

Forgot to say, if it seems to work, this will become the new master branch.

[Matgen84]

32 minutes ago, Jief_Machak said:

Forgot to say, if it seems to work, this will become the new master branch.

 

This new Clover.efi need Openruntime r12. I git clone 0.73 branch, compile clover and openruntime. Otherwise the tests are false. See my previous post, please.

CAN'T BUILD with XCODE8 or GCC53  (Buildme)

 

build.py...
/Users/mathieu/src/cloverbootloader/Clover.dsc(294): error 000E: File/directory not found in workspace
	/Users/mathieu/src/cloverbootloader/OpenCorePkg/Platform/OpenCore/OpenCoreLib.inf

 

Edited September 23, 2021 by Matgen84

[Matgen84]

@Jief_Machak

I try another method to build 0.7.3 branch (Clover/OpencorePkg) : no success 

 

[DLINK] AppleKeyAggregator
/Users/mathieu/src/CloverBootloader/OpenCorePkg/Library/OcMainLib/OpenCoreMisc.c: In function 'OcMiscEarlyInit':
/Users/mathieu/src/CloverBootloader/OpenCorePkg/Library/OcMainLib/OpenCoreMisc.c:448:29: error: unused variable 'ConfigDataSize' [-Werror=unused-variable]
  448 |   UINT32                    ConfigDataSize;
      |                             ^~~~~~~~~~~~~~
/Users/mathieu/src/CloverBootloader/OpenCorePkg/Library/OcMainLib/OpenCoreMisc.c:447:30: error: unused variable 'ConfigData' [-Werror=unused-variable]
  447 |   CHAR8                     *ConfigData;
      |                              ^~~~~~~~~~
/Users/mathieu/src/CloverBootloader/OpenCorePkg/Library/OcMainLib/OpenCoreMisc.c:446:29: error: unused variable 'Status' [-Werror=unused-variable]
  446 |   EFI_STATUS                Status;
      |                             ^~~~~~
[SLINK] OcCpuLib
[CC] MemMap
Building ... /Users/mathieu/src/CloverBootloader/MdeModulePkg/Universal/Metronome/Metronome.inf [X64]
[CC] file
[CC] BootEntryManagement
[CC] AutoGen
[SLINK] UefiBootManagerLib
/Users/mathieu/src/CloverBootloader/OpenCorePkg/Library/OcMainLib/OpenCoreMisc.c: At top level:
cc1: note: unrecognized command-line option '-Wno-incompatible-ms-struct' may have been intended to silence earlier diagnostics
cc1: all warnings being treated as errors
make: *** [/Users/mathieu/src/CloverBootloader/Build/Clover/RELEASE_GCC53/X64/OpenCorePkg/Library/OcMainLib/OcMainLibClover/OUTPUT/OpenCoreMisc.obj] Error 1


build.py...
 : error 7000: Failed to execute command
	make tbuild [/Users/mathieu/src/CloverBootloader/Build/Clover/RELEASE_GCC53/X64/OpenCorePkg/Library/OcMainLib/OcMainLibClover]


build.py...
 : error F002: Failed to build module
	/Users/mathieu/src/CloverBootloader/OpenCorePkg/Library/OcMainLib/OcMainLibClover.inf [X64, GCC53, RELEASE]

 

[Jief_Machak]

2 hours ago, Matgen84 said:

 

From USB key and HDD: stuck at "Automatic Boot in 0 seconds"

 

Note: when you have few minutes, please remove serdeluik's commits from master branch in CloverHackyColor/OpencorePKG. Thanks.

2021-09-23_13-08_CloverX64-2021-09-23-15-42-13-d8755c9-jief.efi.log 35.98 kB · 1 download

Ah yes !

You have to update openruntime.efi

 

[Jief_Machak]

15 minutes ago, Matgen84 said:

@Jief_Machak

I try another method to build 0.7.3 branch (Clover/OpencorePkg) : no success 

 

[DLINK] AppleKeyAggregator
/Users/mathieu/src/CloverBootloader/OpenCorePkg/Library/OcMainLib/OpenCoreMisc.c: In function 'OcMiscEarlyInit':
/Users/mathieu/src/CloverBootloader/OpenCorePkg/Library/OcMainLib/OpenCoreMisc.c:448:29: error: unused variable 'ConfigDataSize' [-Werror=unused-variable]
  448 |   UINT32                    ConfigDataSize;
      |                             ^~~~~~~~~~~~~~
/Users/mathieu/src/CloverBootloader/OpenCorePkg/Library/OcMainLib/OpenCoreMisc.c:447:30: error: unused variable 'ConfigData' [-Werror=unused-variable]
  447 |   CHAR8                     *ConfigData;
      |                              ^~~~~~~~~~
/Users/mathieu/src/CloverBootloader/OpenCorePkg/Library/OcMainLib/OpenCoreMisc.c:446:29: error: unused variable 'Status' [-Werror=unused-variable]
  446 |   EFI_STATUS                Status;
      |                             ^~~~~~
[SLINK] OcCpuLib
[CC] MemMap
Building ... /Users/mathieu/src/CloverBootloader/MdeModulePkg/Universal/Metronome/Metronome.inf [X64]
[CC] file
[CC] BootEntryManagement
[CC] AutoGen
[SLINK] UefiBootManagerLib
/Users/mathieu/src/CloverBootloader/OpenCorePkg/Library/OcMainLib/OpenCoreMisc.c: At top level:
cc1: note: unrecognized command-line option '-Wno-incompatible-ms-struct' may have been intended to silence earlier diagnostics
cc1: all warnings being treated as errors
make: *** [/Users/mathieu/src/CloverBootloader/Build/Clover/RELEASE_GCC53/X64/OpenCorePkg/Library/OcMainLib/OcMainLibClover/OUTPUT/OpenCoreMisc.obj] Error 1


build.py...
 : error 7000: Failed to execute command
	make tbuild [/Users/mathieu/src/CloverBootloader/Build/Clover/RELEASE_GCC53/X64/OpenCorePkg/Library/OcMainLib/OcMainLibClover]


build.py...
 : error F002: Failed to build module
	/Users/mathieu/src/CloverBootloader/OpenCorePkg/Library/OcMainLib/OcMainLibClover.inf [X64, GCC53, RELEASE]

 

I must have a difference in my build options because these are warnings, not error, with my build script.

Committed.

[Jief_Machak]

For those who won't compile, the runtime v12 : OpenRuntime-v12.efi

You keep the name like this and rename your previous OpenRuntime-v11.efi.

Tomorrow, I'll post a modification so you can keep both in the driver folder. For tonight, only keep one.

[Matgen84]

28 minutes ago, Jief_Machak said:

For those who won't compile, the runtime v12 : OpenRuntime-v12.efi

You keep the name like this and rename your previous OpenRuntime-v11.efi.

Tomorrow, I'll post a modification so you can keep both in the driver folder. For tonight, only keep one.

 

Thanks.
 

I can't always build... 0.7.3Branch. No big deal for the moment

/Users/mathieu/src/cloverbootloader/Clover.dsc(294): error 000E: File/directory not found in workspace
	/Users/mathieu/src/cloverbootloader/OpenCorePkg/Platform/OpenCore/OpenCoreLib.inf

 

[Matgen84]

5 minutes ago, MifJpnAlphaPlus said:

 

...EXITBS:START...

+++++++++++++++++++++++++++++++++++++

 

and STOP

 

2021-09-23_16-27_CloverX64-2021-09-23-15-42-13-d8755c9-jief.efi.log.zip 21.32 kB · 0 downloads

 

Thank you

 

 

 

@Jief_Machak

Same issue.

 

2021-09-23_16-36_CloverX64-2021-09-23-15-42-13-d8755c9-jief.efi.log

[yapan4]

 

@Jief_Machak

My Supermicro X11SRA-F based system boots OK!

yapan4-logs.zip

Edited September 23, 2021 by yapan4

[Slice]

@Jief_Machak

I had to make this change

- Status = OcStorageInitFromFs(&mOpenCoreStorage, FileSystem, self.getCloverDirFullPath().wc_str(), NULL);
+ Status = OcStorageInitFromFs(&mOpenCoreStorage, FileSystem, NULL, NULL, self.getCloverDirFullPath().wc_str(), NULL);

For the Clover compatibility with new OC. But I afraid it is not correct. See please.

[Slice]

This version works somehow. Not sure better or not.

CLOVERX64.efi.zip

@Jief_Machak

I changed config-sample.plist so I think ccpv should be upgraded

[yapan4]

30 minutes ago, Slice said:

This version works somehow. Not sure better or not.

CLOVERX64.efi.zip 783.45 kB · 2 downloads

@Jief_Machak

I changed config-sample.plist so I think ccpv should be upgraded

Boot ok only with old OpenRuntime.efi

yapan4_logs_for_Slice.zip

Edited September 23, 2021 by yapan4

[Jief_Machak]

18 minutes ago, Slice said:

Status = OcStorageInitFromFs(&mOpenCoreStorage, FileSystem, self.getCloverDirFullPath().wc_str(), NULL);

This was line 869 of main.cpp and it's already done in the repo but in the branch "0.7.3branch". You probably switch branch for OpenCorePkg but not for Clover. That's to be avoided because you're committing to master changes that are needed for the 0.7.3branch.

 

 

Why did you add "FW_FEATURE_SUPPORTS_LARGE_BASESYSTEM". it's not used anywhere so far ?

And gEfiAudioDecodeProtocolGuid ? Here it's compiling fine without it...

 

Looks like you remerge the commits I cancelled about the upgrade to 0.6.5 that wasn't complete. I guess you didn't re-clone or did the git reset --hard.

 

I've deleted these commits, so we can be back to the master branch how it was before I messed things up with the OpenCore upgrade.

 

If you got a message saying you should merge your local changes, please don't. Re-clone instead.

 

The commit "implement ExtendedFirmwareFeatures" is lost, sorry. But if we can wait a bit before re-doing it, it'll be simpler.

[Jief_Machak]

@Slice There is one thing I didn't import yet in the 0.7.3branch : the cpuid you added in OC (file OpenCorePkg/Library/OcAppleKernelLib/CommonPatches.c and file OpenCoreKernelPatch.c)

I didn't import because the file structure has changed and I don't know where to put it now. It looks like there is something in OC about that. Don't know if it's new in OC.

 

To commit in the branch, you have to be sure that OpenCorePkg is on branch "0.7.3branch2" (do a git checkout 0.7.3branch2 in Clover/OpenCorePkg folder) and that Clover is on branch "0.7.3branch" (do a git checkout 0.7.3branch in Clover folder)

You easily check which branch you are on by issuing "git status".

Here is a version with a panic message instead of a freeze if OpenRuntime is not right.

CloverX64-2021-09-23-22-55-36-8f3b2f0-jief.zip

[Slice]

Oops...

I'll have my own local repo until I will have a possibility reimplement my changes into new master branch especially CPUID and ExtendedFeatures.

 

I have to report that I successfully updated Monterey to beta 7 using the version from previous post and old Openruntime.efi.

Update method is follow:

1. Boot by new Clover and old Openruntime, old config.plist.

2. SystemPreferences->Update->OK

3. Rest until finish.

4. After return I see OS version 21A5522h

[Slice]

1 hour ago, eSaF said:

I like this 3rd step, yes we all needed plenty of this because of this Beta 7. 

Moreover I was going to sleep and returned to the computer only at morning.

 

But I forgot to say that I already have the model iMac17,1 so I am not sure if all users must change it.

[Slice]

10 hours ago, Jief_Machak said:

This was line 869 of main.cpp and it's already done in the repo but in the branch "0.7.3branch". You probably switch branch for OpenCorePkg but not for Clover. That's to be avoided because you're committing to master changes that are needed for the 0.7.3branch.

 

 

Why did you add "FW_FEATURE_SUPPORTS_LARGE_BASESYSTEM". it's not used anywhere so far ?

And gEfiAudioDecodeProtocolGuid ? Here it's compiling fine without it...

 

Looks like you remerge the commits I cancelled about the upgrade to 0.6.5 that wasn't complete. I guess you didn't re-clone or did the git reset --hard.

 

I've deleted these commits, so we can be back to the master branch how it was before I messed things up with the OpenCore upgrade.

 

If you got a message saying you should merge your local changes, please don't. Re-clone instead.

 

The commit "implement ExtendedFirmwareFeatures" is lost, sorry. But if we can wait a bit before re-doing it, it'll be simpler.

FW_FEATURE_SUPPORTS_LARGE_BASESYSTEM is a definition that is not used directly as many other definitions somehow discovered by acidanthera. It is proposed to be used by users in their config.plists.

 

gEfiAudioDecodeProtocolGuid - I just wonder why you and acidanthera didn't see this mistake. May be because you are using clang? I compiled it with gcc and got linker error "undefined symbol in OpenCoreLib".

 

I will not work with a branch staying at master branch. I will wait while you finish your work and return to master.

 

[Jief_Machak]

1 hour ago, Slice said:

FW_FEATURE_SUPPORTS_LARGE_BASESYSTEM is a definition that is not used directly as many other definitions somehow discovered by acidanthera. It is proposed to be used by users in their config.plists.

 

If it's not used in code, better not to import it now. This will be imported automatically with the next upgrade to the OpenCorePkg component. My plan is to keep up with OpenCore version, as it should now be easy to upgrade using rebase git technique. But that would only work if there is not conflict.

 

1 hour ago, Slice said:

gEfiAudioDecodeProtocolGuid - I just wonder why you and acidanthera didn't see this mistake. May be because you are using clang? I compiled it with gcc and got linker error "undefined symbol in OpenCoreLib".

 

I'm using GCC all the time and it's compiling here.

I'm not saying it's wrong to have that here, it probably should, but I try to keep OC modification to a bare minimum so there is less chance of conflict for the next OC upgrade. 

Here it's compiling both Gcc and xcode.

 

1 hour ago, Slice said:

I will wait while you finish your work and return to master.

I'm doing a test with MatGen. If it works, I'll make the 0.7.3branch the new master.

4 hours ago, Slice said:

I have to report that I successfully updated Monterey to beta 7 using the version from previous post and old Openruntime.efi.

I'm not sure what version, but if you use an Openruntime v11, it means it was tu current version of Clover, not the one that integrates OC 0.7.3. Am I right ?

[Slice]

In my EFI folder I changed the file CloverX64.efi and no more. So Openruntime.efi remains june version.

[Jief_Machak]

6 hours ago, Slice said:

In my EFI folder I changed the file CloverX64.efi and no more. So Openruntime.efi remains june version.

Yes understood that. But if that works, it means you are NOT using the Clover I've sent that use OC 0.7.3. OC check OpenRuntime version and stop if version is not right with a message "incompatible version". Because OC stop (CpuDeadLoop), Clover GUI is frozen.

If implemented a check before that gives a panic message, so at least people know.

New Clover that uses OC 0.7.3 cannot use that same openruntime as before when it was OC 0.6.1. It is just impossible.

 

It seems to work, so I'll convert that branch to master, keeping the old master as something like "0.6.1branch" in case we need to access it sometime.

[PG7]

1 hour ago, Jief_Machak said:

Yes understood that. But if that works, it means you are NOT using the Clover I've sent that use OC 0.7.3. OC check OpenRuntime version and stop if version is not right with a message "incompatible version". Because OC stop (CpuDeadLoop), Clover GUI is frozen.

If implemented a check before that gives a panic message, so at least people know.

New Clover that uses OC 0.7.3 cannot use that same openruntime as before when it was OC 0.6.1. It is just impossible.

 

It seems to work, so I'll convert that branch to master, keeping the old master as something like "0.6.1branch" in case we need to access it sometime.

 

tested here on All In One Asus Vivo AiO V241IC-R

it worked perfect with the Cloverx64 and OpenRuntime v12.efi set posted by Jief

thanks

 

Spoiler

 

[Jief_Machak]

I've re-assigned master, for both Clover and OpenCorePkg.

It means now that master is using 0.7.3 version of OpenCore : update your OpenRuntime.efi. And please, call it OpenRuntime-v12.efi. Reason for that is that, when we'll have a v13, the v12 will be automatically ignored and you'll be able to have OpenRuntime-v12.efi AND OpenRuntime-v13.efi in the same driver folder and Clover will pick the right one : no more panic when you'll switch version.

 

NOTE : please re-clone, or make sure that Clover AND OpenCorePkg pointing to the new master. I've tested with buildme and buildme XCODE8 from scratch, so if it doesn't compile, it's because you got a wrong version (old or deleted commit).

15 hours ago, MifJpnAlphaPlus said:

Hello.
% git pull --recurse-submodules
% ./buildme
And buildall.

% git rev-parse HEAD
583e2dd5dacc278c34a67652521a628ed13d4e3d

I adapted the package to USBStick and tried booting.

 

 ... EXIT BS: START ...
+++++++++++++++++++++++++++++++++++++

 

 and STOP

 

2021-09-24_14-29_BOOTX64.EFI.log.zip
What should I do?

Maybe USB/FixOwnership is wrong ?

[Matgen84]

@Jief_Machak I think it will be interesting to build Openruntime with his incremental number version ( OpenRuntime-v12.efi), to identify it immediately. Just an opinion, but I don't know if it's possible. 

I can build Clover successfully. Thanks.

Please, can you re-add @Slice commit for macOS Monterey Beta 7. When you'll have time

Edited September 25, 2021 by Matgen84