RE: [PATCH 03/04] qemu-kvm: Remove the dependency for phys_ram_base

"Zhang, Xiantao" <[email protected]>
Newsgroups org.kernel.vger.kvm-ia64,org.kernel.vger.kvm
Message-ID <706158FABBBA044BAD4FE898A02E4BC236A2C04D@pdsmsx503.ccr.corp.intel.com>
Jes Sorensen wrote:
> Zhang, Xiantao wrote:
>> Avi Kivity wrote:
>>> I haven't pushed this out yet, so I can apply a replacement patch.
>> 
>> We don't need flush_icache_range here, because I believe it is
>> called in cpu_physical_memory_write. Xiantao
> 
> Hi Xiantao,
> 
> Good point, I hadn't spotted that, and worse I mangled
> flush_icache_range() in the process, trying to be smart :-(
> 
> Here's a version without cache flush changes, it should do the right
> thing<tm>. We still need to look at the nvram stuff, but that is
> broken in all setups currently.
> 
> What do you think of this one?

Hi, Jes
    Except nvram stuff, I don't see the difference with my patch. Could you provide an incremental patch to fix nvram stuff ? :)
Xiantao
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.