qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [Bug] System emulation with flash broken?
@ 2012-01-06 18:58 Stefan Weil
  2012-01-07 18:03 ` Gerhard Wiesinger
  0 siblings, 1 reply; 6+ messages in thread
From: Stefan Weil @ 2012-01-06 18:58 UTC (permalink / raw)
  To: Avi Kivity; +Cc: QEMU Developers

Hello Avi,

latest QEMU seems to be broken when a system emulation with flash is 
started.

I did not run git bisect, but it looks like the problem started with 
this change:

c5705a7728b4a6bc9e4f2d35911adbaf28042b25
Author: Avi Kivity <avi@redhat.com>  2011-12-20 14:59:12
vmstate, memory: decouple vmstate from memory API

I first noticed the problem with my AR7 emulation (not part
of official QEMU), but also got it with MIPS Malta, so I assume
that all systems with flash are broken and fail with this assertion:

/qemu/exec.c:2785: qemu_ram_set_idstr: Assertion `new_block' failed.

The function is searching for address 0x10000009 which is 0x10000000
plus an index. Without the index, the address would have been found.

Regards,

Stefan

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [Qemu-devel] [Bug] System emulation with flash broken?
  2012-01-06 18:58 [Qemu-devel] [Bug] System emulation with flash broken? Stefan Weil
@ 2012-01-07 18:03 ` Gerhard Wiesinger
  2012-01-08 11:12   ` Avi Kivity
  0 siblings, 1 reply; 6+ messages in thread
From: Gerhard Wiesinger @ 2012-01-07 18:03 UTC (permalink / raw)
  To: Avi Kivity; +Cc: Stefan Weil, QEMU Developers

On Fri, 6 Jan 2012, Stefan Weil wrote:

> Hello Avi,
>
> latest QEMU seems to be broken when a system emulation with flash is started.
>
> I did not run git bisect, but it looks like the problem started with this 
> change:
>
> c5705a7728b4a6bc9e4f2d35911adbaf28042b25
> Author: Avi Kivity <avi@redhat.com>  2011-12-20 14:59:12
> vmstate, memory: decouple vmstate from memory API
>
> I first noticed the problem with my AR7 emulation (not part
> of official QEMU), but also got it with MIPS Malta, so I assume
> that all systems with flash are broken and fail with this assertion:
>
> /qemu/exec.c:2785: qemu_ram_set_idstr: Assertion `new_block' failed.
>
> The function is searching for address 0x10000009 which is 0x10000000
> plus an index. Without the index, the address would have been found.

Hello Avi,

I'm having also problems with this patch:
# VGA bank switching problems under DOS
git checkout 682a3c07f0e28d2532c911a44a9b6142d6299cc2

# OK
git checkout aad3b517a1b83561f2755dc4451596a421399c19

And commit 682a3c07f0e28d2532c911a44a9b6142d6299cc2 is the Merge commit 
'c5705a7728b4a6bc9e4f2d35911adbaf28042b25' into upstream-merge.

Thnx.

Ciao,
Gerhard

--
http://www.wiesinger.com/

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [Qemu-devel] [Bug] System emulation with flash broken?
  2012-01-07 18:03 ` Gerhard Wiesinger
@ 2012-01-08 11:12   ` Avi Kivity
  2012-01-08 12:23     ` Gerhard Wiesinger
  0 siblings, 1 reply; 6+ messages in thread
From: Avi Kivity @ 2012-01-08 11:12 UTC (permalink / raw)
  To: Gerhard Wiesinger; +Cc: Stefan Weil, QEMU Developers

On 01/07/2012 08:03 PM, Gerhard Wiesinger wrote:
> On Fri, 6 Jan 2012, Stefan Weil wrote:
>
>> Hello Avi,
>>
>> latest QEMU seems to be broken when a system emulation with flash is
>> started.
>>
>> I did not run git bisect, but it looks like the problem started with
>> this change:
>>
>> c5705a7728b4a6bc9e4f2d35911adbaf28042b25
>> Author: Avi Kivity <avi@redhat.com>  2011-12-20 14:59:12
>> vmstate, memory: decouple vmstate from memory API
>>
>> I first noticed the problem with my AR7 emulation (not part
>> of official QEMU), but also got it with MIPS Malta, so I assume
>> that all systems with flash are broken and fail with this assertion:
>>
>> /qemu/exec.c:2785: qemu_ram_set_idstr: Assertion `new_block' failed.
>>
>> The function is searching for address 0x10000009 which is 0x10000000
>> plus an index. Without the index, the address would have been found.
>
> Hello Avi,
>
> I'm having also problems with this patch:
> # VGA bank switching problems under DOS
> git checkout 682a3c07f0e28d2532c911a44a9b6142d6299cc2
>
> # OK
> git checkout aad3b517a1b83561f2755dc4451596a421399c19
>
> And commit 682a3c07f0e28d2532c911a44a9b6142d6299cc2 is the Merge
> commit 'c5705a7728b4a6bc9e4f2d35911adbaf28042b25' into upstream-merge.
>

A patch for this has already been posted; hopefully to be merged soon.

-- 
error compiling committee.c: too many arguments to function

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [Qemu-devel] [Bug] System emulation with flash broken?
  2012-01-08 11:12   ` Avi Kivity
@ 2012-01-08 12:23     ` Gerhard Wiesinger
  2012-01-08 12:33       ` Avi Kivity
  0 siblings, 1 reply; 6+ messages in thread
From: Gerhard Wiesinger @ 2012-01-08 12:23 UTC (permalink / raw)
  To: Avi Kivity; +Cc: Stefan Weil, QEMU Developers

On Sun, 8 Jan 2012, Avi Kivity wrote:

> On 01/07/2012 08:03 PM, Gerhard Wiesinger wrote:
>> On Fri, 6 Jan 2012, Stefan Weil wrote:
>>
>>> Hello Avi,
>>>
>>> latest QEMU seems to be broken when a system emulation with flash is
>>> started.
>>>
>>> I did not run git bisect, but it looks like the problem started with
>>> this change:
>>>
>>> c5705a7728b4a6bc9e4f2d35911adbaf28042b25
>>> Author: Avi Kivity <avi@redhat.com>  2011-12-20 14:59:12
>>> vmstate, memory: decouple vmstate from memory API
>>>
>>> I first noticed the problem with my AR7 emulation (not part
>>> of official QEMU), but also got it with MIPS Malta, so I assume
>>> that all systems with flash are broken and fail with this assertion:
>>>
>>> /qemu/exec.c:2785: qemu_ram_set_idstr: Assertion `new_block' failed.
>>>
>>> The function is searching for address 0x10000009 which is 0x10000000
>>> plus an index. Without the index, the address would have been found.
>>
>> Hello Avi,
>>
>> I'm having also problems with this patch:
>> # VGA bank switching problems under DOS
>> git checkout 682a3c07f0e28d2532c911a44a9b6142d6299cc2
>>
>> # OK
>> git checkout aad3b517a1b83561f2755dc4451596a421399c19
>>
>> And commit 682a3c07f0e28d2532c911a44a9b6142d6299cc2 is the Merge
>> commit 'c5705a7728b4a6bc9e4f2d35911adbaf28042b25' into upstream-merge.
>>
>
> A patch for this has already been posted; hopefully to be merged soon.

To test: Which patch do you exactly mean?

Thnx.

Ciao,
Gerhard

--
http://www.wiesinger.com/

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [Qemu-devel] [Bug] System emulation with flash broken?
  2012-01-08 12:23     ` Gerhard Wiesinger
@ 2012-01-08 12:33       ` Avi Kivity
  2012-01-12 20:55         ` Gerhard Wiesinger
  0 siblings, 1 reply; 6+ messages in thread
From: Avi Kivity @ 2012-01-08 12:33 UTC (permalink / raw)
  To: Gerhard Wiesinger; +Cc: Stefan Weil, QEMU Developers

On 01/08/2012 02:23 PM, Gerhard Wiesinger wrote:
> On Sun, 8 Jan 2012, Avi Kivity wrote:
>
>> On 01/07/2012 08:03 PM, Gerhard Wiesinger wrote:
>>> On Fri, 6 Jan 2012, Stefan Weil wrote:
>>>
>>>> Hello Avi,
>>>>
>>>> latest QEMU seems to be broken when a system emulation with flash is
>>>> started.
>>>>
>>>> I did not run git bisect, but it looks like the problem started with
>>>> this change:
>>>>
>>>> c5705a7728b4a6bc9e4f2d35911adbaf28042b25
>>>> Author: Avi Kivity <avi@redhat.com>  2011-12-20 14:59:12
>>>> vmstate, memory: decouple vmstate from memory API
>>>>
>>>> I first noticed the problem with my AR7 emulation (not part
>>>> of official QEMU), but also got it with MIPS Malta, so I assume
>>>> that all systems with flash are broken and fail with this assertion:
>>>>
>>>> /qemu/exec.c:2785: qemu_ram_set_idstr: Assertion `new_block' failed.
>>>>
>>>> The function is searching for address 0x10000009 which is 0x10000000
>>>> plus an index. Without the index, the address would have been found.
>>>
>>> Hello Avi,
>>>
>>> I'm having also problems with this patch:
>>> # VGA bank switching problems under DOS
>>> git checkout 682a3c07f0e28d2532c911a44a9b6142d6299cc2
>>>
>>> # OK
>>> git checkout aad3b517a1b83561f2755dc4451596a421399c19
>>>
>>> And commit 682a3c07f0e28d2532c911a44a9b6142d6299cc2 is the Merge
>>> commit 'c5705a7728b4a6bc9e4f2d35911adbaf28042b25' into upstream-merge.
>>>
>>
>> A patch for this has already been posted; hopefully to be merged soon.
>
> To test: Which patch do you exactly mean?
>

Original patch:

  http://article.gmane.org/gmane.comp.emulators.qemu/131082

My alternative:

  http://article.gmane.org/gmane.comp.emulators.qemu/131164

-- 
error compiling committee.c: too many arguments to function

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [Qemu-devel] [Bug] System emulation with flash broken?
  2012-01-08 12:33       ` Avi Kivity
@ 2012-01-12 20:55         ` Gerhard Wiesinger
  0 siblings, 0 replies; 6+ messages in thread
From: Gerhard Wiesinger @ 2012-01-12 20:55 UTC (permalink / raw)
  To: Avi Kivity; +Cc: Stefan Weil, QEMU Developers

On Sun, 8 Jan 2012, Avi Kivity wrote:

> On 01/08/2012 02:23 PM, Gerhard Wiesinger wrote:
>> On Sun, 8 Jan 2012, Avi Kivity wrote:
>>
>>> On 01/07/2012 08:03 PM, Gerhard Wiesinger wrote:
>>>> On Fri, 6 Jan 2012, Stefan Weil wrote:
>>>>
>>>>> Hello Avi,
>>>>>
>>>>> latest QEMU seems to be broken when a system emulation with flash is
>>>>> started.
>>>>>
>>>>> I did not run git bisect, but it looks like the problem started with
>>>>> this change:
>>>>>
>>>>> c5705a7728b4a6bc9e4f2d35911adbaf28042b25
>>>>> Author: Avi Kivity <avi@redhat.com>  2011-12-20 14:59:12
>>>>> vmstate, memory: decouple vmstate from memory API
>>>>>
>>>>> I first noticed the problem with my AR7 emulation (not part
>>>>> of official QEMU), but also got it with MIPS Malta, so I assume
>>>>> that all systems with flash are broken and fail with this assertion:
>>>>>
>>>>> /qemu/exec.c:2785: qemu_ram_set_idstr: Assertion `new_block' failed.
>>>>>
>>>>> The function is searching for address 0x10000009 which is 0x10000000
>>>>> plus an index. Without the index, the address would have been found.
>>>>
>>>> Hello Avi,
>>>>
>>>> I'm having also problems with this patch:
>>>> # VGA bank switching problems under DOS
>>>> git checkout 682a3c07f0e28d2532c911a44a9b6142d6299cc2
>>>>
>>>> # OK
>>>> git checkout aad3b517a1b83561f2755dc4451596a421399c19
>>>>
>>>> And commit 682a3c07f0e28d2532c911a44a9b6142d6299cc2 is the Merge
>>>> commit 'c5705a7728b4a6bc9e4f2d35911adbaf28042b25' into upstream-merge.
>>>>
>>>
>>> A patch for this has already been posted; hopefully to be merged soon.
>>
>> To test: Which patch do you exactly mean?
>>
>
> Original patch:
>
>  http://article.gmane.org/gmane.comp.emulators.qemu/131082
>
> My alternative:
>
>  http://article.gmane.org/gmane.comp.emulators.qemu/131164

Helo Avi,

Can NOT confirm that one of these or both changes work well. So 
something else is (still) wrong.

Ciao,
Gerhard

--
http://www.wiesinger.com/

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2012-01-12 20:56 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-01-06 18:58 [Qemu-devel] [Bug] System emulation with flash broken? Stefan Weil
2012-01-07 18:03 ` Gerhard Wiesinger
2012-01-08 11:12   ` Avi Kivity
2012-01-08 12:23     ` Gerhard Wiesinger
2012-01-08 12:33       ` Avi Kivity
2012-01-12 20:55         ` Gerhard Wiesinger

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).