Re: [Qemu-devel] [PATCH 1/1] net: Add SunGEM device emulation as found on Apple UniNorth

[Mark Cave-Ayland]

On 06/09/17 04:16, David Gibson wrote:
> On Tue, Sep 05, 2017 at 11:13:43AM +1000, David Gibson wrote:
>> On Mon, Sep 04, 2017 at 07:39:38PM +0100, Mark Cave-Ayland wrote:
>>> From: Benjamin Herrenschmidt <address@hidden>
>>>
>>> This adds a simplistic emulation of the Sun GEM ethernet controller
>>> found in Apple ASICs.
>>>
>>> Currently we only support the Apple UniNorth 1.x variant, but the
>>> other Apple or Sun variants should mostly be a matter of adding
>>> PCI IDs options.
>>>
>>> We have a very primitive emulation of a single Broadcom 5201 PHY
>>> which is supported by the MacOS driver.
>>>
>>> This model brings out-of-the-box networking to MacOS 9, and all
>>> versions of OS X I tried with the mac99 platform.
>>>
>>> Further improvements from Mark:
>>> - Remove sungem.h file, moving constants into sungem.c as required
>>> - Switch to using tracepoints for debugging
>>> - Split register blocks into separate memory regions
>>> - Use arrays in SunGEMState to hold register values
>>> - Add state-saving support
>>>
>>> Signed-off-by: Benjamin Herrenschmidt <address@hidden>
>>> Signed-off-by: Mark Cave-Ayland <address@hidden>
>>
>> Applied to ppc-for-2.11.
> 
> Until I discovered that it breaks compile with
> --enable-trace-backend=ust.   So I've removed it again.

Oh that's interesting. I've had --enable-trace-backend=simple as part of
my default build for a long while now, so I'm quite surprised that the
trace-events are backend sensitive.

Any clue as to what the error might be before I go and start installing
the relevant libraries?


ATB,

Mark.