From: Finn Thain <fthain@linux-m68k.org>
To: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Cc: aleksandar.rikalo@syrmia.com, jasowang@redhat.com,
qemu-devel@nongnu.org, f4bug@amsat.org, hpoussin@reactos.org,
aurelien@aurel32.net, laurent@vivier.eu
Subject: Re: [PATCH v2 00/10] dp8393x: fixes for MacOS toolbox ROM
Date: Sat, 26 Jun 2021 18:55:18 +1000 (AEST) [thread overview]
Message-ID: <7bd0c5f7-ce26-51dd-a238-104157636620@nippy.intranet> (raw)
In-Reply-To: <20210625065401.30170-1-mark.cave-ayland@ilande.co.uk>
On Fri, 25 Jun 2021, Mark Cave-Ayland wrote:
> Here is the next set of patches from my attempts to boot MacOS under
> QEMU's Q800 machine related to the Sonic network adapter.
>
> Patches 1 and 2 sort out checkpatch and convert from DPRINTF macros to
> trace-events.
>
> The discussion for the v1 patchset concluded that the dp8393x device
> does NOT have its own NVRAM (there is no mention of it on the datasheet)
> and so patches 3 to 5 move the generation of the PROM to the q800 and
> jazz boards separately to allow the formats to diverge.
>
> Patch 6 adds an implementation of bitrev8 to bitops.h in preparation for
> changing the q800 PROM storage format, whilst patch 7 updates the MAC
> address storage and checksum for the q800 machine to match the format
> expected by the MacOS toolbox ROM.
>
> Patch 8 ensures that the CPU loads/stores are correctly converted to
> 16-bit accesses for the network card and patch 9 fixes a bug when
> selecting the index specified for CAM entries.
>
> Finally since the MIPS magnum machine exists for both big-endian
> (mips64) and little-endian (mips64el) configurations, patch 10 sets the
> dp8393x big_endian property accordingly using a similar technique
> already used for the MIPS malta machines.
>
> Migration notes: the changes to the dp8393x PROM are a migration break,
> but we don't care about this for now since a) the q800 machine will have
> more breaking migration changes as further MacOS toolbox ROM support is
> upstreamed and b) the magnum machine migration is currently broken (and
> has been for quite some time).
>
> Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
>
Tested-by: Finn Thain <fthain@linux-m68k.org>
My testing was much the same as last time: 'qemu-system-mips64el -M
magnum' with Linux/mips and NetBSD/arc, and 'qemu-system-m68k -M q800'
with Linux/m68k. In each case I checked the MAC address against the '-nic'
option. The host is little-endian.
I have not tested 'qemu-system-mips64 -M magnum'. It appears MIPS RISC/os
is needed for that but I don't have it.
>
> v2:
> - Move PROM generation from dp8393x to q800 and magnum machines and remove
> the existing code from the device itself
> - Add bitrev8 implementation to bitops.h so it can be used elsewhere in
> future. Use a shift/merge technique rather than a massive table lookup
> as we don't care about speed
> - Add patch to set the big_endian property correctly depending upon whether
> a big-endian or little-endian configuration is being used
>
>
> Mark Cave-Ayland (10):
> dp8393x: checkpatch fixes
> dp8393x: convert to trace-events
> hw/mips/jazz: move PROM and checksum calculation from dp8393x device
> to board
> hw/m68k/q800: move PROM and checksum calculation from dp8393x device
> to board
> dp8393x: remove onboard PROM containing MAC address and checksum
> qemu/bitops.h: add bitrev8 implementation
> hw/m68k/q800: fix PROM checksum and MAC address storage
> dp8393x: don't force 32-bit register access
> dp8393x: fix CAM descriptor entry index
> hw/mips/jazz: specify correct endian for dp8393x device
>
> hw/m68k/q800.c | 21 ++-
> hw/mips/jazz.c | 32 ++++-
> hw/net/dp8393x.c | 313 +++++++++++++++++++-----------------------
> hw/net/trace-events | 17 +++
> include/qemu/bitops.h | 22 +++
> 5 files changed, 231 insertions(+), 174 deletions(-)
>
>
next prev parent reply other threads:[~2021-06-26 13:15 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-25 6:53 [PATCH v2 00/10] dp8393x: fixes for MacOS toolbox ROM Mark Cave-Ayland
2021-06-25 6:53 ` [PATCH v2 01/10] dp8393x: checkpatch fixes Mark Cave-Ayland
2021-06-25 8:45 ` Philippe Mathieu-Daudé
2021-06-25 6:53 ` [PATCH v2 02/10] dp8393x: convert to trace-events Mark Cave-Ayland
2021-06-25 8:47 ` Philippe Mathieu-Daudé
2021-06-25 6:53 ` [PATCH v2 03/10] hw/mips/jazz: move PROM and checksum calculation from dp8393x device to board Mark Cave-Ayland
2021-07-01 21:43 ` Philippe Mathieu-Daudé
2021-06-25 6:53 ` [PATCH v2 04/10] hw/m68k/q800: " Mark Cave-Ayland
2021-07-01 21:43 ` Philippe Mathieu-Daudé
2021-06-25 6:53 ` [PATCH v2 05/10] dp8393x: remove onboard PROM containing MAC address and checksum Mark Cave-Ayland
2021-07-01 21:43 ` Philippe Mathieu-Daudé
2021-06-25 6:53 ` [PATCH v2 06/10] qemu/bitops.h: add bitrev8 implementation Mark Cave-Ayland
2021-07-01 21:46 ` Philippe Mathieu-Daudé
2021-06-25 6:53 ` [PATCH v2 07/10] hw/m68k/q800: fix PROM checksum and MAC address storage Mark Cave-Ayland
2021-06-25 6:53 ` [PATCH v2 08/10] dp8393x: don't force 32-bit register access Mark Cave-Ayland
2021-07-01 21:34 ` Philippe Mathieu-Daudé
2021-07-02 4:36 ` Finn Thain
2021-07-03 6:21 ` Mark Cave-Ayland
2021-07-03 8:52 ` Philippe Mathieu-Daudé
2021-07-03 12:04 ` Mark Cave-Ayland
2021-07-03 13:10 ` Philippe Mathieu-Daudé
2021-07-03 14:16 ` Mark Cave-Ayland
2021-07-03 14:22 ` Philippe Mathieu-Daudé
2021-06-25 6:54 ` [PATCH v2 09/10] dp8393x: fix CAM descriptor entry index Mark Cave-Ayland
2021-07-03 12:59 ` Philippe Mathieu-Daudé
2021-07-05 19:13 ` Philippe Mathieu-Daudé
2021-06-25 6:54 ` [PATCH v2 10/10] hw/mips/jazz: specify correct endian for dp8393x device Mark Cave-Ayland
2021-06-25 8:51 ` Philippe Mathieu-Daudé
2021-06-25 12:01 ` Mark Cave-Ayland
2021-07-01 21:45 ` Philippe Mathieu-Daudé
2021-06-26 8:55 ` Finn Thain [this message]
2021-07-02 13:03 ` [PATCH v2 00/10] dp8393x: fixes for MacOS toolbox ROM Philippe Mathieu-Daudé
2021-07-03 6:32 ` Mark Cave-Ayland
2021-07-03 8:48 ` Philippe Mathieu-Daudé
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=7bd0c5f7-ce26-51dd-a238-104157636620@nippy.intranet \
--to=fthain@linux-m68k.org \
--cc=aleksandar.rikalo@syrmia.com \
--cc=aurelien@aurel32.net \
--cc=f4bug@amsat.org \
--cc=hpoussin@reactos.org \
--cc=jasowang@redhat.com \
--cc=laurent@vivier.eu \
--cc=mark.cave-ayland@ilande.co.uk \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).