[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 13/13] net: Introduce e1000e device emulation
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCH v2 13/13] net: Introduce e1000e device emulation |
Date: |
Wed, 6 Apr 2016 16:23:27 +0300 |
On Wed, Apr 06, 2016 at 11:22:24AM +0300, Dmitry Fleytman wrote:
> + MemoryRegion flash;
>
>
> Looks there's no real implementation for flash. So is this really needed
> (e.g did WHQL or other test check for this?).
>
>
> From our experience some drivers may verify that device exposes all memory
> regions as required by HW specification, so we added this dummy memory region
> to be on the safe side.
But then, why use an io region?
Also, pls add some comments that explains this.
--
MSt