How to boost the OS X boot process...

[GrootWitBaas]

something is not right on getting the dynamic cpu data ...sure on static all works fine, but look at this

 

 

Yes I was all taken from the same boot

 

PS can you post a FULL release Pretty Please, Just to make sure I have all the edits done correctly.

 

every thing else works GR8 as always (I think, but I am Know to be wrong some times)

[DB1]

Hi blackosx,

 

Perfect.

You don't. I guess I mixed the two.

Huh? Wait. Let me check the files... Just what I expected. I was in a hurry for dinner and somehow attached the wrong files. Sorry.

 

Yeah that messed with my head for while!

 

Got compile error with these:

In file included from smbios.c:62:
smbios/dynamic_data.h: In function ‘smbiosRealRun’:
smbios/dynamic_data.h:572: error: expected expression before ‘uint32_t’
smbios/dynamic_data.h:572: error: called object ‘newsmbios->dmi.tableAddress’ is not a function
smbios/dynamic_data.h:572: error: expected ‘;’ before ‘AllocateKernelMemory’
make[2]: *** [smbios.o] Error 1
make[1]: *** [all] Error 2
make: *** [all] Error 2

 

Fixed with:

 

newsmbios->dmi.tableAddress

= (uint32_t)AllocateKernelMemory(newsmbios->dmi.tableLength);

 

Testing now, update shortly.

 

This worked ok for dynamic on both but when I tried static cpu I got compile error:

cpu/dynamic_data.h: In function ‘initCPUStruct’:
cpu/dynamic_data.h:212: warning: passing argument 2 of ‘strstr’ makes pointer from integer without a cast
cpu/dynamic_data.h:213: error: expected ‘)’ before ‘{’ token
cpu/dynamic_data.h:247: error: expected expression before ‘}’ token
cpu/dynamic_data.h:205: warning: unused variable ‘s’
make[2]: *** [cpu.o] Error 1
make[1]: *** [all] Error 2
make: *** [all] Error 2

 

Fixed it with:

 

if (strstr((char *)reg, "Atom") != NULL || strstr((char *)reg, "mobile" != NULL))

 

You were in a rush! LOL

 

Ok similar to blackosx in next post I have 160 Mhz cpu, core 2 solo when set dynamic both.

EDIT Found to many 0's:

 

static int getCPUFrequency(void)

{

return gPlatform.CPU.CPUFrequency / 1000000;

}

 

Good news though the memory patching works

 

Getting closer, keep it up

 

Only problem now for me is still not recognising mobile cpu, and core 2 solo

 

Soooo......Close

[blackosx]

You don't. I guess I mixed the two.

Lol.. I'm glad you confirmed that. I thought I was going do-lally..

 

Huh? Wait. Let me check the files... Just what I expected. I was in a hurry for dinner and somehow attached the wrong files. Sorry.

Thanks.

 

I had a compile error for cpu/dynamic_data.h

cpu/dynamic_data.h: In function ‘initCPUStruct’:
cpu/dynamic_data.h:212: warning: passing argument 2 of ‘strstr’ makes pointer from integer without a cast
cpu/dynamic_data.h:213: error: expected ‘)’ before ‘{’ token
cpu/dynamic_data.h:247: error: expected expression before ‘}’ token
cpu/dynamic_data.h:205: warning: unused variable ‘s’
make[2]: *** [cpu.o] Error 1
make[1]: *** [all] Error 2
make: *** [all] Error 2

which I fixed by adding a missing )

 

EDIT: (I see you found that one one too DB1 )

 

and also had the same error in smbios/dynamic_data.h as DB1 reported and used his fix. (thanks DB1)

 

This boots successfully, however, using dynamic data my Processor Speed is now reported as 267Mhz and not 2.67Ghz. Can we shift it along one more?

 

Nearly there DHP

[DB1]

Lol.. I'm glad you confirmed that. I thought I was going do-lally..

 

 

Thanks.

 

I had a compile error for cpu/dynamic_data.h

cpu/dynamic_data.h: In function ‘initCPUStruct’:
cpu/dynamic_data.h:212: warning: passing argument 2 of ‘strstr’ makes pointer from integer without a cast
cpu/dynamic_data.h:213: error: expected ‘)’ before ‘{’ token
cpu/dynamic_data.h:247: error: expected expression before ‘}’ token
cpu/dynamic_data.h:205: warning: unused variable ‘s’
make[2]: *** [cpu.o] Error 1
make[1]: *** [all] Error 2
make: *** [all] Error 2

which I fixed by adding a missing )

 

EDIT: (I see you found that one one too DB1 )

 

and also had the same error in smbios/dynamic_data.h as DB1 reported and used his fix. (thanks DB1)

 

This boots successfully, however, using dynamic data my Processor Speed is now reported as 267Mhz and not 2.67Ghz. Can we shift it along one more?

 

Nearly there DHP

 

To many 0's in smbios/dynamic_data.h here:

 

//==============================================================================

 

static int getCPUFrequency(void)

{

return gPlatform.CPU.CPUFrequency / 1000000;

}

[blackosx]

To many 0's in smbios/dynamic_data.h here:

Good spot DB1. That fixed it - Thanks mate.

 

Good news though the memory patching works

I've tested that too and yep - works for me also.

[blackosx]

Hi STLVNUB

 

Nice one.

 

I've downloaded it and will take a look either tonight or more likely in the morning as it's late now and bed time for me very shortly.. I'll report back when I can.

[GrootWitBaas]

This worked ok for dynamic on both but when I tried static cpu I got compile error:

cpu/dynamic_data.h: In function ‘initCPUStruct’:
cpu/dynamic_data.h:212: warning: passing argument 2 of ‘strstr’ makes pointer from integer without a cast
cpu/dynamic_data.h:213: error: expected ‘)’ before ‘{’ token
cpu/dynamic_data.h:247: error: expected expression before ‘}’ token
cpu/dynamic_data.h:205: warning: unused variable ‘s’
make[2]: *** [cpu.o] Error 1
make[1]: *** [all] Error 2
make: *** [all] Error 2

 

Fixed it with:

 

if (strstr((char *)reg, "Atom") != NULL || strstr((char *)reg, "mobile" != NULL))

 

You were in a rush! LOL

With all the fixes it works, but there was still a warning during compile, so I fixed it with changing the above to :

 

if (strstr((char *)reg, "Atom") != NULL || strstr((char *)reg,"Mobile") != NULL)

I am not sure if the "Mobile" should be with a Capital or not. We could just search for "obile" like it was and it should work, I guess this is what DHP had in mind when putting "obile" there

 

Now to get rid off the last warning (acpi/essentials.h:272: warning: ‘staticTables’ defined but not used)

 

Every thing else works

@STLVNUB will test this and give feedback soon thanks

[GrootWitBaas]

From your tool how do I choose "Prep" as there is no choice for it. See below

A ok I see now, I needed a usb disk called RevBooter then I have this option. nice, testing it now - worked fine.

[blackosx]

Hi STLVNUB

 

First of all - Wow.. You've been a busy bee as that's a big script with a lot of code.

Second - I have used it successfully to create a bootable USB.

 

EDIT: and out of interest, I've burnt the .iso and it also boots perfectly.

 

So well done

 

 

I've started to make a constructive report based on my trial which I hope you take positively.

 

1 - When first running RevStart.command maybe change the screen readout so it reads with the option numbers beside the Modes rather than below. like this?

 

1) Format USB/HD and Prep it.

2) Compile FULL Then Make Revolution USB/HD/Iso

3) Compile Revolution boot file

4) Clean Revolution

5) Delete ISO, ACPI and STATIC Folders And UserFlags

6) Refresh The Menu To Detect Your USB

7) Name For USB/HD/ISO

8) exit

 

2 - It's wasn't obvious I needed to rename by USB flash drive to the default of 'RevBooter', or that I could choose option 6 to tell the script to use a name of my choice.

 

On screen I read:

RevBooter IS OFFLINE

INSERT USB AND SELECT MENU

 

Maybe change it to read something like:

Volume RevBooter can't be found:

Insert a USB stick named RevBooter and select option 6 to refresh.

If it's not named RevBooter then select option 7 to tell me the name.

 

3 - Choosing option 1 to 'Prep' then runs an Applescript for me to select a volume. Why? as surely this option should use the volume that is already shown above as the 'Prep' option is not available until this is set.

 

Note: exiting the script here means the Applescript keeps running even though I click cancel

 

EDIT: I see now from reading the previous post that I can forget the 'Prep' option and that once the script is satisfied with the USB volume name, I can then just press option 2 for MakeMedia and let it run. Cool.

 

There is more but lets try to do this one step at a time..

 

 

This tool is mainly for Dynamic use.

The STATICS are broken in the Revstart tools, the Source code and this tool.

By Revstart tools I mean the general layout of the Source code has Improved so the tools need updating (IMHO).

By Source code I mean the placing of STATIC Folder outside the Source and just including the various files.

By this tool I mean a combination of the other two and then some.

 

STATICS should work, just not with this tool, YET!!

I agree that more work is needed with auto-generating the private_data file and I am happy to work on that as some of the hard work was already done before with regard to grabbing the required info, but we just have to be aware that this structure of the file will probably change in time as DHP continues the development.

[GrootWitBaas]

enjoy the sleep

 

So I guessed correctly about "obile" and looks like reading "Sams Teach Yourself C++ in 21 days" does help to place the closing ) on the right place, I just wish I knew more (and studied more when I was younger, cause at my age it does not take 21 days, more like 21 weeks)

[DB1]

I just returned from the sweet 16 party. What a madness.

 

Let's start with the "obile" versus "Mobile". This was intentional because then it will match with both "mobile" and "Mobile". No idea if we need this, but that was the idea yes. Giving us the following line:

if (strstr((char *)reg, "Atom") != NULL || strstr((char *)reg, "obile") != NULL)

 

Now look at the following line:

gPlatform.CPU.CPUFrequency	= (cpuFrequency / 10); // <= This might be wrong / limited to Core i CPU maybe?

And having read comments about the extra zero. Well. This basically means that it is Intel Core in only. Let's restore it to:

gPlatform.CPU.CPUFrequency	= cpuFrequency;

And let me move the line to a new place for the next release.

 

Ok made these changes and still get core 2 solo.

 

Oh to be 16 again - and know what I know now!

 

And without further ado.

edit : Revised Script DOH!!!

Treat this as beta,

Iso creation is just a fad.Probably no real use.

When first run it will ask a few questions, just do what it says.

e.g Revolution Revision, Put Extra/Extensions Here etc.

 

Typing this from a Newly created USB using this tool.

It should also do HD.Not really Tested and probably not needed.

Select Prep: Use The Revised Script

Select your USB ***WARNING WILL FORMAT USB/HD TAKE CARE***

It Will FULLY Prep it and Make it bootable.

Compile and place the necessary files on it.

You are now ready to try it

 

ACPI Folder contains ALL ACPI TABLES for future use.

STATIC Folder contains some STATIC structures for future use.

 

Also contains fdisk and bless

This bless command is a cut down version of something I have been working on for while, that version

does practically everything bar wash the dishes.

 

Bugs are bound to be there (couple found already), anybody wanting to fix or improve are most welcome.I'm no script wizard.

 

This tool is mainly for Dynamic use.

The STATICS are broken in the Revstart tools, the Source code and this tool.

By Revstart tools I mean the general layout of the Source code has Improved so the tools need updating (IMHO).

By Source code I mean the placing of STATIC Folder outside the Source and just including the various files.

By this tool I mean a combination of the other two and then some.

 

STATICS should work, just not with this tool, YET!!

 

All three need to work in unison and when that is done a couple of key presses should do it all.

 

Enjoy

 

 

I'll be out for the rest of the day.(Runs and hides....Bye)

 

Nice work STLVNUB, just a couple of observations:

 

I did not see an option for an alternative kernel which Atom users and other than core 2 users may want.

 

The mkext did not build correctly for me - complained about the kextx not being authentic so an alternative option kextbuild command probably needed (I, 32 bit only)

 

Oh, and an option to put on EFI partition if you want to go the whole hog.

 

Other than that great stuff.

 

 

 

 

 

enjoy the sleep

 

So I guessed correctly about "obile" and looks like reading "Sams Teach Yourself C++ in 21 days" does help to place the closing ) on the right place, I just wish I knew more (and studied more when I was younger, cause at my age it does not take 21 days, more like 21 weeks)

 

Don't think i could learn it in 21 years - if I have that left LOL

 

DHP attached sysctl - might help.

cpu_sysctl.txt

[humph]

Hi all. My internet died, so in a coffee shop to catch up latest!!

 

STLVNUB,

Nice tool. Am looking forward to trying it out in a bit!

 

DHP, DB1,

Had to do some reading but 0x0201 was used for old Mac Mini's and thus we might want to make the following change:

-		else if (gPlatform.CPU.NoCores == 1)
+		else if (gPlatform.CPU.NoCores == 1 && gPlatform.CPU.NoThreads == 1)
	{
		gPlatform.CPU.Type = 0x0201;	// Intel Core Solo (old Mac Mini's).
	}

 

I was sort of following the CPU identification thing, but not overly concerned (perhaps I should have been!) with getting Core 2 Solo name for my ATOM. Just wondering about this, since Atom has hyperthreading, 1 core, 2 threads (or 2 cores and 4 threads on Atom 330 desktop). Are we trying to get Atom seen as Core Solo or ??

 

I also noted that in the (new) CPU Dynamic Data.h:

_CPU_DEBUG_DUMP("CPU: CPUFreq			  : %dMHz\n",			gPlatform.CPU.CPUFrequency / 100000);

Should this actually have an extra 0 also? TSC, FSB have the extra zero there.

 

P.S. Still working on DSDT reduction, down to 11kB now from 25kB, thanks to insights from scrax's and DB1s earlier posts. Although not seeing any appreciable difference in boot times. I guess I just have to accept that the majority of the time it's kernel stuff and that's taking 40-50s for the Atom NB, so a second or two improvement at the front end is less relevant. Although it'll make big percentage impact when I get Revo stuff onto my P55 thing, currently in storage.

[blackosx]

Now look at the following line:

gPlatform.CPU.CPUFrequency	= (cpuFrequency / 10); // <= This might be wrong / limited to Core i CPU maybe?

And having read comments about the extra zero. Well. This basically means that it is Intel Core in only. Let's restore it to:

gPlatform.CPU.CPUFrequency	= cpuFrequency;

And let me move the line to a new place for the next release.

Just to confirm. Making this change and re-adding the zero we removed in post #953 allows using dynamic data to show my Processor Speed as 2.67Ghz.

[DB1]

Thanks. I will. Right after I fixed two bugs

 

 

Yes. You was right. And isn't it the combination of different people/minds here that makes us strong? I think so.

 

Yes, absolutely!

Had to do some reading but 0x0201 was used for old Mac Mini's and thus we might want to make the following change:

-		else if (gPlatform.CPU.NoCores == 1)
+		else if (gPlatform.CPU.NoCores == 1 && gPlatform.CPU.NoThreads == 1)
	{
		gPlatform.CPU.Type = 0x0201;	// Intel Core Solo (old Mac Mini's).
	}

That should take care of it.

 

So. Now it is time for me to get a few hours of sleep. Back later...

 

Ok so smbios dynamic & cpu dynamic still shows core 2 solo, but with smbios static & cpu dynamic it shows core solo as it should be so there's some interaction problem when both dynamic. Might help point you toward the problem I hope.

 

Once this sorted I will grab a new smbios as switching off smbios to get cpu right prevents capture of the memory stuff and I would like to get both correct for new static.

 

Saw NoThreads == 1) Atom has 2 threads so will change and try that now.

 

EDIT

 

Change to 2 ok, hypethreading working. hump if you don't already know the prefpane for hyperthreading is in excode/Extras/Prefpanes

 

EDIT 2

 

Tried static both after the changes and get compile error:

In file included from cpu.c:23:
cpu/dynamic_data.h: In function ‘initCPUStruct’:
cpu/dynamic_data.h:385: error: invalid operands to binary && (have ‘int’ and ‘struct CPU’)
cpu/dynamic_data.h:385: error: ‘NoThreads’ undeclared (first use in this function)
cpu/dynamic_data.h:385: error: (Each undeclared identifier is reported only once
cpu/dynamic_data.h:385: error: for each function it appears in.)
cpu/dynamic_data.h:385: warning: left-hand operand of comma expression has no effect
make[2]: *** [cpu.o] Error 1
make[1]: *** [all] Error 2
make: *** [all] Error 2

Enjoy the sleep catch up.

[DB1]

Thanks.

 

Is The kernel for Atom users named differently??

Could put option for different kernel I suppose.

 

Thanks again

 

The lotto predictor is the one we really want!

 

I keep my modded Atom kernel on the RevBooter and don't rename it, but it can be renamed and put in RevBooter or on hdd root and selected via a call from c.a.B.plist. Either which way an option during the process would be good but it's no real pain dropping it onto the RevBooter after it's made.

[blackosx]

The menu SELECTION is built DYNAMICALLY , but I have implemented some of your suggestions in the next Version.

Like I said ANY changes are welcome, still haven't done the lotto predictor YET.

Okay. Great.

 

Maybe need two p_d.h like private_data.h and static_private_data.h.

I've already started revising function DoPrivateDataH to build a private_data.h file to match the latest style using some of the static data we have collected and I've revised the output from aml2struct_mod.pl to better fit our needs. Next job is to look at revising efidp2struct to tweak it's output too.

[GrootWitBaas]

I Think having a Private_Table.h and including it in Private_Data.h might be the right way to go. This way the private file can be changed and the tables will still be on the right place. The alternative it to have the tables all at the end of Private_Data.h

[DB1]

Hi DB1,

 

First. Static SMBIOS data won't make use of your dynamically gathered CPU data.

 

And I am getting a little confused. I mean your debug output in post #944 clearly shows type as being 513 (0x201) and now I wonder what this gives you. Is that "Core 2 Solo" or "Core Solo" in About This Mac/System Profiler?

 

I wonder why you want a "Core Solo" because that's the successor of the Pentium M... with a single core/thread. Which is what early mac mini's used though these days not anymore of course. Now they use 769 (0x301).

 

 

That is the way to go yes.

 

 

Yes. And because of that it shouldn't use 0x201 but stick to the default aka 0x301.

 

#944 debug does show 513 and gave me Core 2 solo in about this mac/System Profiler. "I wonder why you want a core solo" well that's because that's what my cpu is (Atom n270) , and if the idea of dynamic detection is to get the correct values then there was something not quite right.

 

I do now have core solo with the latest tweaks with smbios in private data h static and cpu dynamic but if I try both static I cannot compile, the error given in previous post and below.

private_data.h.zip cpu_dynamic_data.h.zip

[DB1]

And that is what it should be since the Intel Core 2 Solo has one core and supports 2 threads. Just like you Atom. We're not talking about the Core 2 Duo after all.

 

 

No sir. The Intel Core Solo not only has a single core, like your Atom, but support a single thread only. Unlike your Atom which supports two (with hyper-threading).

 

 

It is yes. But I think that one of us is confused.

 

 

That has to be some typo/error because I double checked it. Compiles fine, and it works.

 

I take it that you copied the line from the code snippet in my previous post? If yes, then please check for strange invisible characters (think spaces).

 

I'll work a bit on the new templates now. Should be fairly easy. Updates on the fly...

 

1.) I renamed/replaced i386/private_data.h to/with i386/private_config.h (see attachments).

2.) I copied the private_data.h template and stripped all data blocks out of it (see attachments).

3.) I copied the private_data.h template and stripped all settings (directives) out of it (see attachments).

4.) I renamed the filename in i386/libsa/libsa.h so that it reads: #include "../private-config.h"

5.) I renamed the filename in i386/libsaio/biosfn.c so that it reads: #include "../private-config.h"

6.) I renamed the filename in i386/libsaio/efi/essentials.h so that it reads: #include "../private-config.h"

 

Compiling. Done. Booting. Err. No HPET's available. Guess I forgot to load my DSDT.aml Checking... Yup Boots fine. Testing static and dynamic CPU data gathering. Both fine. Now testing static and dynamic SMBIOS data gathering. Fine (after removing some new/experimental code).

 

Still two bugs to fix (the same I was working on). Getting closer to a new release by the day, but not today. Maybe tomorrow. Good night.

 

I'm not confused on this one Atom n270 is not core 2: Quick refrence

 

Think I will wait for next release my files have been changed so many times since last release there very well may be typo's.

 

Did you win the hockey?

 

EDIT

 

Could not resist trying the new files and took me a while to work it out:

Settings dynamic cpu & static smbios in revo-config.h boots ok and shows core solo in profiler (great)

 

When setting cpu & smbios dynamic in revo-config.h (so i could get the benefit of patching the ram settings) get compile error:

 

platform.c: In function ‘initPlatform’:
platform.c:171: error: ‘SMB_MEM_TYPE_DDR2’ undeclared (first use in this function)
platform.c:171: error: (Each undeclared identifier is reported only once
platform.c:171: error: for each function it appears in.)
make[2]: *** [platform.o] Error 1
make[1]: *** [all] Error 2
make: *** [all] Error 2

 

Note i changed DDR3 to 2 in revo-config.h because that's what i have. Must have a buggy set of files if everything works for you. need the release to be sure I not messed up some where.

 

Had enough for today, up for work in a few hours so will be tomorrow night before I catchup.

[blackosx]

I'll work a bit on the new templates now. Should be fairly easy. Updates on the fly...

Thanks DHP - I'll test now.

 

Good to see what you can come up with

I had something going but I wanted to wait to see what DHP came up with. Now I know.

 

Anyone else getting this:

IPS Driver Error

Yep. Most annoying. Sometimes it doesn't happen for a week then I see it frequently.. then it's fixed again. So it's not just you.

[humph]

I'm not confused on this one Atom n270 is not core 2: Quick refrence

Indeed, Atom is not a Core/Core2 CPU. It's Atom!

I personally think it might make sense to call it same as a proc with 2 threads though, and for cosmetic reasons nice if was used on a real laptop product.

Which kind of means not Core2Solo (as most are 1C/1T), or CoreSolo (1C/1T) but one of:

 

Core Duo (2C/2T) T2300/T2400

Core2Duo (2C/2T) T5500/T5600

 

Unless doing so somehow messes up something important! Like hyperthreading.

 

P.S. Never needed the Hyperthreading/Processor Prefpane to get 2 threads on the Atom, just works up to now.

[GrootWitBaas]

We won one with a pretty massive lead so we tried something new – no defense players – in the second match and lost it on one point. No problem, because we're pool champion already, but walking / sliding out of the goal against three players is no fun. BTW my brother drove like a mad man to get home in time for a pizza. Didn't get his pizza but I was home long before I should have been.

 

I can imagine that. I need I fix the two bugs that is holding me off from releasing Revolution v643 but I am home already so who knows.

Congratz on that.

 

Hope you have a fabulous day (Feb 14 is always for me special, my sister has her birthday and that makes me want the whole wide world have a fabulous day:D) )

 

Now those to bugs .....what are they ?? as you yourself said before many minds make this work, and maybe just maybe one of us could help or point to something that get them squashed to bits

[GrootWitBaas]

looking nice asking for usb is good. Only thing is I get two blank files in STATIC folder, is this correct ?

 

My mistake got it fixed now and yes looks good

[scrax]

Hi STLVNUB,

 

Yeah that sucks but what can we do? Not much I guess they have to get it sorted. Must be busy times.

 

 

Including separate files won't work I'm afraid. Sorry. What I did for the attached release of Revolution-643 was to change a few things, to make it easier for you. Now look at the new files and directories:

 

i386/config

i386/config/data.h

i386/config/settings.h

i386/config/acpi/data.h

i386/config/efi/data.h

i386/config/smbios/data.h

 

Templates included. Hockey training now so back later...

I was waiting it! and it looks great the config dir. Now let's try...

[DB1]

Including separate files won't work I'm afraid. Sorry. What I did for the attached release of Revolution-643 was to change a few things, to make it easier for you. Now look at the new files and directories:

 

i386/config

i386/config/data.h

i386/config/settings.h

i386/config/acpi/data.h

i386/config/efi/data.h

i386/config/smbios/data.h

 

Templates included. Hockey training now so back later...

 

All working fine for me on SDcard and will try on EFI shortly - core 2 solo dynamic, core solo static. Understand now where you coming from re cpu, but there seems no detrimental effect with either setting (or any so far) both threads show operating in processor palette.

 

You left out memory size fix, can I just added back into settings.h or is there other stuff to do?

 

I may have asked this before but cannot remember (either the fix or if I asked-maybe loosing the plot!): I want to grab a new smbios (with memory adjustments) with MBP6.1 identifier. If i do dynamic simbios current it gives me MB4.1. What do I need to do to force identifier MBP6.1 (I have them set in settings.h efi.c and platform.c)