Re: [PATCH v2] efi: libstub: add support for the apple_set_os protocol

Lukas Wunner
2024-07-01T13:45:03Z

On Mon, Jul 01, 2024 at 11:14:52PM +1000, Orlando Chamberlain wrote:
> > FWIW, there would be only 6 models to match if this needs to be
> > constrained to ones with dual GPUs:
> >
> > MacBookPro11,3
> > MacBookPro11,5
> > MacBookPro13,3
> > MacBookPro14,3
> > MacBookPro15,1
> > MacBookPro16,1
>
> I know that at least these two also need it:
> MacBookPro16,4
> MacBookPro15,3

Ah okay, thanks. I went by the list in Wikipedia, disappointing
that it apparently wasn't updated with those final T2 models:

https://en.wikipedia.org/wiki/MacBook_Pro_(Intel-based)

> I think there could be more older ones too. Apple has a list of
> MacBookPros [0], but I don't know how many of the older models that
> list 2 gpus on their "Tech Specs" pages need apple-set-os.

No, Bruno Bierbaumer's page lists the MacBookPro11,3 and 11,5
as the first models that need apple_set_os:

https://github.com/0xbb/gpu-switch

I'm still using a MacBookPro9,1 (last pre-retina) on a daily basis
and apple_set_os isn't needed there. Neither on the MacBookPro10,1
(first retina). Both were introduced mid 2012.

> The original apple-set-os code was posted to the GRUB mailing list in
> December 2013 [1] so maybe it was in 2013 that new dual GPU Macbooks
> started needing apple-set-os?

Yes, Haswell generation introduced Oct 2013 was the first one that
needed it.

Thanks,

Lukas