[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-2.4] watchdog/diag288: correctly register fo
From: |
Christian Borntraeger |
Subject: |
Re: [Qemu-devel] [PATCH for-2.4] watchdog/diag288: correctly register for system reset requests |
Date: |
Tue, 07 Jul 2015 16:22:02 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 |
Am 07.07.2015 um 12:51 schrieb Peter Crosthwaite:
>> qemu_log_mask(CPU_LOG_RESET, "Watchdog timer expired.\n");
>> @@ -80,6 +87,7 @@ static void wdt_diag288_realize(DeviceState *dev, Error
>> **errp)
>> {
>> DIAG288State *diag288 = DIAG288(dev);
>>
>> + qemu_register_reset(diag288_reset, diag288);
>
> Doesn't seem right. Even if it is not a SBD it should still sit in the
> QOM tree in a place where the reset is reached. Where is this device
> in the QOM tree?
Hmm, to me it seems that qemu_devices_reset does only work for devices
on a bus. This is a busless-device so the reset function gets not triggered.
>
> I.E. What string do you get with an object_get_canonical_path() of the
> obj after machine init?
path /machine/peripheral/watchdog0
- [Qemu-devel] [PATCH for-2.4] s390x: diag288 reset fix, Cornelia Huck, 2015/07/07
- [Qemu-devel] [PATCH for-2.4] watchdog/diag288: correctly register for system reset requests, Cornelia Huck, 2015/07/07
- Re: [Qemu-devel] [PATCH for-2.4] watchdog/diag288: correctly register for system reset requests, Christian Borntraeger, 2015/07/07
- Re: [Qemu-devel] [PATCH for-2.4] watchdog/diag288: correctly register for system reset requests, Peter Crosthwaite, 2015/07/07
- Re: [Qemu-devel] [PATCH for-2.4] watchdog/diag288: correctly register for system reset requests,
Christian Borntraeger <=
- Re: [Qemu-devel] [PATCH for-2.4] watchdog/diag288: correctly register for system reset requests, Peter Crosthwaite, 2015/07/07
- Re: [Qemu-devel] [PATCH for-2.4] watchdog/diag288: correctly register for system reset requests, Cornelia Huck, 2015/07/08
- Re: [Qemu-devel] [PATCH for-2.4] watchdog/diag288: correctly register for system reset requests, Cornelia Huck, 2015/07/08
- Re: [Qemu-devel] [PATCH for-2.4] watchdog/diag288: correctly register for system reset requests, Cornelia Huck, 2015/07/09
- Re: [Qemu-devel] [PATCH for-2.4] watchdog/diag288: correctly register for system reset requests, Peter Crosthwaite, 2015/07/09
- Re: [Qemu-devel] [PATCH for-2.4] watchdog/diag288: correctly register for system reset requests, Peter Maydell, 2015/07/14
- Re: [Qemu-devel] [PATCH for-2.4] watchdog/diag288: correctly register for system reset requests, Cornelia Huck, 2015/07/07
- Re: [Qemu-devel] [PATCH for-2.4] watchdog/diag288: correctly register for system reset requests, Paolo Bonzini, 2015/07/08
- Re: [Qemu-devel] [PATCH for-2.4] watchdog/diag288: correctly register for system reset requests, Christian Borntraeger, 2015/07/08
- Re: [Qemu-devel] [PATCH for-2.4] watchdog/diag288: correctly register for system reset requests, Paolo Bonzini, 2015/07/08