[Qemu-devel] [PATCH v2 0/3] Update AppleSMC for OS X Sierra 10.12.4 guests

Gabriel L. Somlo posted 3 patches 7 years ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/1491325264-14502-1-git-send-email-gsomlo@gmail.com
Test checkpatch passed
Test docker passed
Test s390x passed
hw/misc/applesmc.c | 219 +++++++++++++++++++++++++++++++++++++----------------
1 file changed, 155 insertions(+), 64 deletions(-)
[Qemu-devel] [PATCH v2 0/3] Update AppleSMC for OS X Sierra 10.12.4 guests
Posted by Gabriel L. Somlo 7 years ago
As of 10.12.4 (currently the latest Sierra update), OS X refuses to boot
unless the AppleSMC supports a third I/O port, which provides the current
error status when read.

New since v1:

	- 1/3: don't touch the default OSK string, as it's unnecessary
		at this time

	- 2/3: don't consolidate I/O regions, leave as-is for data
		and cmd; This patch now implements the error-code state
		machine, AND adds an i/o region dedicaded to the error
		status port, complete with read/write access methods.

	- 3/3: optional patch setting access width to 1-byte on data and
		cmd i/o regions. Tested on OS X versions 10.[6..12].

> This series consists of three patches:
>
>	- 1/3: indentation/whitespace cleanup for applesmc.c to the point
>		where it now passes scripts/checkpatc.pl, and allows
>		subsequent changes to look nice in diff-patch format :)
>
>	- 2/3: consolidate Port I/O into a single region, and invoke
>		appropriate read/write methods based on the offset being
>		accessed
>
>	- 3/3: implement read-only error/status port, and update
>		data and command read/write methods to correctly
>		maintain the state machine for keeping the status_1e
>		value up to date.

Gabriel L. Somlo (3):
  applesmc: cosmetic whitespace and indentation cleanup
  applesmc: implement error status port
  applesmc: fix port i/o access width

 hw/misc/applesmc.c | 219 +++++++++++++++++++++++++++++++++++++----------------
 1 file changed, 155 insertions(+), 64 deletions(-)

-- 
2.7.4


Re: [Qemu-devel] [PATCH v2 0/3] Update AppleSMC for OS X Sierra 10.12.4 guests
Posted by Alexander Graf 7 years ago
On 04/04/2017 07:01 PM, Gabriel L. Somlo wrote:
> As of 10.12.4 (currently the latest Sierra update), OS X refuses to boot
> unless the AppleSMC supports a third I/O port, which provides the current
> error status when read.

Looks much nicer after this series :). Thanks a lot!

Reviewed-by: Alexander Graf <agraf@suse.de>


Alex


Re: [Qemu-devel] [PATCH v2 0/3] Update AppleSMC for OS X Sierra 10.12.4 guests
Posted by Gabriel L. Somlo 7 years ago
On Tue, Apr 04, 2017 at 09:35:00PM +0200, Alexander Graf wrote:
> On 04/04/2017 07:01 PM, Gabriel L. Somlo wrote:
> > As of 10.12.4 (currently the latest Sierra update), OS X refuses to boot
> > unless the AppleSMC supports a third I/O port, which provides the current
> > error status when read.
> 
> Looks much nicer after this series :). Thanks a lot!
> 
> Reviewed-by: Alexander Graf <agraf@suse.de>

Thanks! Thinking about it, we should probably drop the last (3/3)
patch, once again to minimize useless churn -- Doing that part (or
not, as the case may turn out to be) should be part of a future
series implementing more accurate emulation, including key write
support...

Thanks,
--Gabriel