From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([140.186.70.92]:37818)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <avi@redhat.com>) id 1Qwfe2-0003oj-Ru
	for qemu-devel@nongnu.org; Thu, 25 Aug 2011 15:31:20 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <avi@redhat.com>) id 1Qwfe1-0006Ak-Ek
	for qemu-devel@nongnu.org; Thu, 25 Aug 2011 15:31:18 -0400
Received: from mx1.redhat.com ([209.132.183.28]:21769)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <avi@redhat.com>) id 1Qwfe1-0006Ag-57
	for qemu-devel@nongnu.org; Thu, 25 Aug 2011 15:31:17 -0400
Message-ID: <4E56A300.8020805@redhat.com>
Date: Thu, 25 Aug 2011 22:31:12 +0300
From: Avi Kivity <avi@redhat.com>
MIME-Version: 1.0
References: <CAAu8pHvE1a95q4LnH4UGK6aQt3mywMWhA22OvgLSt3xo6Garng@mail.gmail.com>
	<4E54C730.1010602@redhat.com>
	<CAAu8pHtUTvw+HxO79mGy-MbKYMJZSFP3R2D1fH6AptZso8MvPg@mail.gmail.com>
	<4E55465B.2000708@redhat.com>
	<CAAu8pHu19oWGpAkrjhcOYccDejp6QzusNp82oMVYAesTvdPRkQ@mail.gmail.com>
	<4E55583C.5040407@redhat.com>
	<CAAu8pHtppQskPhW0GwhGPucxCtxJM1AvBb7JObSTcQhZOv2how@mail.gmail.com>
	<4E55D30D.4080501@redhat.com>
	<CAAu8pHs+TXZirHqH1rYq3XD63b+pB6M1-YjdzHthvK1Uakxfvw@mail.gmail.com>
In-Reply-To: <CAAu8pHs+TXZirHqH1rYq3XD63b+pB6M1-YjdzHthvK1Uakxfvw@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] PPC* and Sparc32 crash
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Blue Swirl <blauwirbel@gmail.com>
Cc: qemu-devel <qemu-devel@nongnu.org>

On 08/25/2011 08:33 PM, Blue Swirl wrote:
> (gdb) bt
> #0  0x00007ffff3da2165 in *__GI_raise (sig=<value optimized out>)
>      at ../nptl/sysdeps/unix/sysv/linux/raise.c:64
> #1  0x00007ffff3da4f70 in *__GI_abort () at abort.c:92
> #2  0x00007ffff3d9b2b1 in *__GI___assert_fail (
>      assertion=0x5c7390 "info->qdev.size>= sizeof(SysBusDevice)",
>      file=<value optimized out>, line=0x9c,
>      function=0x5c73c0 "sysbus_register_withprop") at assert.c:81
> #3  0x000000000052e000 in sysbus_register_withprop (info=<value optimized out>)
>      at /src/qemu/hw/sysbus.c:156
> #4  0x000000000041d9c3 in module_call_init (type=<value optimized out>)
>      at /src/qemu/module.c:78
> #5  0x0000000000518b64 in main (argc=<value optimized out>,
>      argv=<value optimized out>, envp=<value optimized out>)
>      at /src/qemu/vl.c:3241
> (gdb) up 3
> #3  0x000000000052e000 in sysbus_register_withprop (info=<value optimized out>)
>      at /src/qemu/hw/sysbus.c:156
> 156         assert(info->qdev.size>= sizeof(SysBusDevice));
> (gdb) p *info
> Cannot access memory at address 0x0
> (gdb) p info
> $1 =<value optimized out>

-O1 fixes that.

> (gdb) up
> #4  0x000000000041d9c3 in module_call_init (type=<value optimized out>)
>      at /src/qemu/module.c:78
> 78              e->init();
> (gdb) p *e
> $2 = {type = MODULE_INIT_BLOCK,
>    init = 0x53cf50<sun4c_intctl_register_devices>, node = {
>      tqe_next = 0x1051130, tqe_prev = 0x10510b0}}

static SysBusDeviceInfo sun4c_intctl_info = {
     .init = sun4c_intctl_init1,
     .qdev.name  = "sun4c_intctl",
     .qdev.size  = sizeof(Sun4c_INTCTLState),
     .qdev.vmsd  = &vmstate_sun4c_intctl,
     .qdev.reset = sun4c_intctl_reset,
};

typedef struct Sun4c_INTCTLState {
     SysBusDevice busdev;
     ...
}

so the code looks fine.  Can you dig deeper?  Did something stomp on 
qdev.size?

-- 
I have a truly marvellous patch that fixes the bug which this
signature is too narrow to contain.