From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists1p.gnu.org (lists1p.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id A6B03CD5BD1 for ; Thu, 28 May 2026 09:59:42 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists1p.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1wSXWI-0007FC-A8; Thu, 28 May 2026 05:59:18 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists1p.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1wSXWG-0007EZ-Pd for qemu-arm@nongnu.org; Thu, 28 May 2026 05:59:16 -0400 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1wSXWE-0000Xr-96 for qemu-arm@nongnu.org; Thu, 28 May 2026 05:59:16 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1779962352; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=ovRx+sev4bTK56alCnYUaSQDqAPK3uVNpV0CBk53D2M=; b=NEEoa0SKGa4i/7ccdX2il/wKZoDGA5Wb/Qy4nUO3Nx3jZXb6qJGxjYa6UYSu5DmcHimqre sYHaD5bHYueJH3wnX9kmdB/JqwMfTg/kuYGK85Bbac4hr3dwvKOTlNg48v/Tx5zy9MHL4L QiQcTZOIKguM5Pgw2VgdIDA0CwhWC8A= Received: from mail-wr1-f69.google.com (mail-wr1-f69.google.com [209.85.221.69]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-503-8CzBCWibMRmr4713gARUhg-1; Thu, 28 May 2026 05:59:11 -0400 X-MC-Unique: 8CzBCWibMRmr4713gARUhg-1 X-Mimecast-MFC-AGG-ID: 8CzBCWibMRmr4713gARUhg_1779962350 Received: by mail-wr1-f69.google.com with SMTP id ffacd0b85a97d-44696b11265so15301319f8f.0 for ; Thu, 28 May 2026 02:59:10 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1779962350; x=1780567150; h=content-transfer-encoding:in-reply-to:references:cc:to:from :content-language:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=ovRx+sev4bTK56alCnYUaSQDqAPK3uVNpV0CBk53D2M=; b=ojZoosqobty0HYCr6VoByQ0A5NZnq7NCBScUSOm2OmTgv9uuf8iRwa8KbfqEytVlSz LgSC7qr+dsve5cyn1w3BHC8Ac/0jlZanCfIWuWwn6j0+qQJIoKuiSrBv9Iuz9kZojH2c /jEtYENmOOt83Y6EA9KVX7dk7rsnXHy+q1ZsWtt/KQbNwzA35GQGIysyQVFcLYOJJO/A OvMRdvht/z0rvMCf1r6wt3T+uPS1OC6LC4Ss+Ha6vdIIYqjDheDLVXAyj9RyE7ucKP3z J4xBM0i486HUOohrfgk3OUkxml7JW72hEXChrrUWNHdHc08fMAw42N6o+iPENeeSUUkx fWBw== X-Forwarded-Encrypted: i=1; AFNElJ/+t10TZsXT3FJ2hozuE841+bu0Deqb3HPCZkoZRGoBfYqpNtXKE5HgKkOVhy4CHD/+WAd+IODI+A==@nongnu.org X-Gm-Message-State: AOJu0YyCVBn4khI1WjIvhCGA1tFZWGnRo01eU4vH1zBKe09A9nf8Kw8s eGiPPMZXjjBRpe8AGIP7ZtY6whAUadRB++k7zUcSwVI3B0fkJquxEvrTbRkTMxJHg46FpqJsYwo TzwJFdKtoFDjoaanMSjbBX8PfW+CiIyPwWdV34WJn4aX7MSCRgAm1WQ== X-Gm-Gg: Acq92OFrS4dnP+ZK+zDqOZazEjnVhrr8gswq+FSxW+cqnaugNrxcqPPEenM9Sfoy7Df fBR4OF1tJdG+hGWzf+zC+zljOJSwSxVGPWQxEYV5dp7YM/iPuHDhnCJTKugJcvUNvx4WGOTt6+N UvOXMzpiMvrzzJ8426/g6EWQzIaxI2UbBkXqohDfqzu0l3JM4vgVBIC4sE1jH7Dku5NLbsCtGcO 41A4lSxnDVQsNTGsw5GqeruCZTNWWTToGAAgeWs1/JAASLaLEDXkmSrCfXkKW5owdvDTLgYrWBI I17y8xsAKwrZ7F97p88KVWZPch+OBdQ9TtEaWlmyXS2/uxcREzB9go6NJLZMdId6OEYewbd7L+g Z9JveQ5rYYCotsKiPt9tg/OMES69eWRjU2AUPwoMzvcznKKpLB2D/iKELdwOuyPRMf7Ba9LLYZp M= X-Received: by 2002:a05:600c:a49:b0:490:6889:202 with SMTP id 5b1f17b1804b1-4906889026emr269463775e9.29.1779962349772; Thu, 28 May 2026 02:59:09 -0700 (PDT) X-Received: by 2002:a05:600c:a49:b0:490:6889:202 with SMTP id 5b1f17b1804b1-4906889026emr269463305e9.29.1779962349303; Thu, 28 May 2026 02:59:09 -0700 (PDT) Received: from ?IPV6:2a01:e0a:f0e:9070:527b:9dff:feef:3874? ([2a01:e0a:f0e:9070:527b:9dff:feef:3874]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-4908ecb5238sm13682765e9.24.2026.05.28.02.59.07 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 28 May 2026 02:59:08 -0700 (PDT) Message-ID: Date: Thu, 28 May 2026 11:59:06 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 09/21] arm: virt: create sbsa_gwdt watchdog From: Eric Auger To: Igor Mammedov Cc: qemu-devel@nongnu.org, mst@redhat.com, anisinha@redhat.com, pbonzini@redhat.com, peter.maydell@linaro.org, shannon.zhaosl@gmail.com, philmd@linaro.org, zhao1.liu@intel.com, rad@semihalf.com, leif.lindholm@oss.qualcomm.com, qemu-arm@nongnu.org References: <20260303092532.2410177-1-imammedo@redhat.com> <20260303092532.2410177-10-imammedo@redhat.com> <4cdd38f4-4efc-48c5-a236-a40449ad8e85@redhat.com> <20260527144955.64eff309@imammedo> In-Reply-To: X-Mimecast-Spam-Score: 0 X-Mimecast-MFC-PROC-ID: fQ4ww_fFgdMSYhfIUpnAR8QHPf5_5MkUJJGXF814RA8_1779962350 X-Mimecast-Originator: redhat.com Content-Language: en-US Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Received-SPF: pass client-ip=170.10.129.124; envelope-from=eauger@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -24 X-Spam_score: -2.5 X-Spam_bar: -- X-Spam_report: (-2.5 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.445, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H4=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-arm@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-arm-bounces+qemu-arm=archiver.kernel.org@nongnu.org Sender: qemu-arm-bounces+qemu-arm=archiver.kernel.org@nongnu.org Hi, On 5/28/26 11:04 AM, Eric Auger wrote: > Hi Igor, > > On 5/27/26 2:49 PM, Igor Mammedov wrote: >> On Tue, 26 May 2026 15:58:08 +0200 >> Eric Auger wrote: >> >>> Hi Igor, >>> >>> On 3/3/26 10:25 AM, Igor Mammedov wrote: >>>> Allow to use SBSA generic watchdog with virt machine type. >>>> (includes conditional generation of corresponding FDT and >>>> ACPI GTDT descriptors) >>>> >>>> Use '-device sbsa_gwdt' to command line to enable it. >>> I thank that in general Peter is not a big fan of dynamic instantiation >>> of sysbus devices. Don't you have means to instantiate it statically in >>> arm virt memory map? >> >> /me neither >> >> in previous version there was a machine option to enable the feature >> to that was rejected in favor of device. >> >> in this case for platform -device of sysbus based device, I don't think >> we can do better that dynamic sysbus. > > sorry I may have missed some previous discussion details. As far as I > can read, I understand you could instantiate the wdt in a free MMIO > slot, unconfitionnaly just as it is done in hw/arm/sbsa-ref.c. Maybe add > a compat to avoid having it by default on older machine types. Then > about the type of the integration gtdt or wat, can't you have a device > compat? After further reading of the previous version thread, I wonder if you shouldn't always instantiate the watchdog in the machine instead. Add a device property allowing to switch between GWDT or WDAT version. In arm virt, 1) if you discover the GWDT version is selected (default) you generate both dt and GTDT ACPI table 2) if you discover the WDAT version is selected, you only generate WDAT ACPI description and you output an error in case of DT boot. You may add a machine no_wdt flag to handle compats for older machine types if needed. Would that make sense? Otherwise, can you point me the msgid where the -device was prefered? Thanks Eric > > Thanks > > Eric > >> >> >>> >>> Eric >>>> >>>> Tested with Fedora 43: >>>> FDT: -M virt,acpi=off -device sbsa_gwdt >>>> ACPI: -M virt -device sbsa_gwdt >>>> >>>> Signed-off-by: Igor Mammedov >>>> --- >>>> hw/arm/Kconfig | 1 + >>>> hw/arm/virt-acpi-build.c | 33 +++++++++++++++++++++++++++++++-- >>>> hw/arm/virt.c | 2 ++ >>>> hw/core/sysbus-fdt.c | 32 ++++++++++++++++++++++++++++++++ >>>> hw/watchdog/sbsa_gwdt.c | 1 + >>>> 5 files changed, 67 insertions(+), 2 deletions(-) >>>> >>>> diff --git a/hw/arm/Kconfig b/hw/arm/Kconfig >>>> index c66c452737..1222efadd1 100644 >>>> --- a/hw/arm/Kconfig >>>> +++ b/hw/arm/Kconfig >>>> @@ -36,6 +36,7 @@ config ARM_VIRT >>>> select VIRTIO_MEM_SUPPORTED >>>> select ACPI_CXL >>>> select ACPI_HMAT >>>> + select WDT_SBSA >>>> >>>> config CUBIEBOARD >>>> bool >>>> diff --git a/hw/arm/virt-acpi-build.c b/hw/arm/virt-acpi-build.c >>>> index 719d2f994e..9c14b1d4d5 100644 >>>> --- a/hw/arm/virt-acpi-build.c >>>> +++ b/hw/arm/virt-acpi-build.c >>>> @@ -64,6 +64,7 @@ >>>> #include "hw/virtio/virtio-acpi.h" >>>> #include "target/arm/cpu.h" >>>> #include "target/arm/multiprocessing.h" >>>> +#include "hw/watchdog/sbsa_gwdt.h" >>>> >>>> #define ARM_SPI_BASE 32 >>>> >>>> @@ -863,6 +864,8 @@ build_gtdt(GArray *table_data, BIOSLinker *linker, VirtMachineState *vms) >>>> const uint32_t irqflags = 0; /* Interrupt is Level triggered */ >>>> AcpiTable table = { .sig = "GTDT", .rev = 3, .oem_id = vms->oem_id, >>>> .oem_table_id = vms->oem_table_id }; >>>> + uint32_t gtdt_start = table_data->len; >>>> + Object *wdt = object_resolve_type_unambiguous(TYPE_WDT_SBSA, NULL); >>>> >>>> acpi_table_begin(&table, table_data); >>>> >>>> @@ -893,10 +896,15 @@ build_gtdt(GArray *table_data, BIOSLinker *linker, VirtMachineState *vms) >>>> build_append_int_noprefix(table_data, irqflags, 4); >>>> /* CntReadBase Physical address */ >>>> build_append_int_noprefix(table_data, 0xFFFFFFFFFFFFFFFF, 8); >>>> + >>>> /* Platform Timer Count */ >>>> - build_append_int_noprefix(table_data, 0, 4); >>>> + build_append_int_noprefix(table_data, wdt ? 1 : 0, 4); >>>> /* Platform Timer Offset */ >>>> - build_append_int_noprefix(table_data, 0, 4); >>>> + build_append_int_noprefix(table_data, >>>> + wdt ? (table_data->len - gtdt_start) + >>>> + 4 + 4 + 4 /* len of this & following 2 fields to skip */ >>>> + : 0, 4); >>>> + >>>> if (vms->ns_el2_virt_timer_irq) { >>>> /* Virtual EL2 Timer GSIV */ >>>> build_append_int_noprefix(table_data, ARCH_TIMER_NS_EL2_VIRT_IRQ, 4); >>>> @@ -906,6 +914,27 @@ build_gtdt(GArray *table_data, BIOSLinker *linker, VirtMachineState *vms) >>>> build_append_int_noprefix(table_data, 0, 4); >>>> build_append_int_noprefix(table_data, 0, 4); >>>> } >>>> + >>>> + /* ACPI 6.5 spec: 5.2.25.2 ARM Generic Watchdog Structure (Table 5-124) */ >>>> + if (wdt) { >>>> + PlatformBusDevice *pbus = PLATFORM_BUS_DEVICE(vms->platform_bus_dev); >>>> + SysBusDevice *sd = SYS_BUS_DEVICE(wdt); >>>> + hwaddr mmio_base = vms->memmap[VIRT_PLATFORM_BUS].base; >>>> + hwaddr rbase = mmio_base + platform_bus_get_mmio_addr(pbus, sd, 0); >>>> + hwaddr cbase = mmio_base + platform_bus_get_mmio_addr(pbus, sd, 1); >>>> + int irq = ARM_SPI_BASE + vms->irqmap[VIRT_PLATFORM_BUS] + >>>> + platform_bus_get_irqn(pbus, sd, 0); >>>> + >>>> + build_append_int_noprefix(table_data, 1 /* Type: Watchdog GT */, 1); >>>> + build_append_int_noprefix(table_data, 28 /* Length */, 2); >>>> + build_append_int_noprefix(table_data, 0, 1); /* Reserved */ >>>> + /* RefreshFrame Physical Address */ >>>> + build_append_int_noprefix(table_data, rbase, 8); >>>> + /* WatchdogControlFrame Physical Address */ >>>> + build_append_int_noprefix(table_data, cbase, 8); >>>> + build_append_int_noprefix(table_data, irq, 4); /* Watchdog Timer GSIV */ >>>> + build_append_int_noprefix(table_data, 0, 4); /* Watchdog Timer Flags */ >>>> + } >>>> acpi_table_end(linker, &table); >>>> } >>>> >>>> diff --git a/hw/arm/virt.c b/hw/arm/virt.c >>>> index cab2e21e8a..a87991b591 100644 >>>> --- a/hw/arm/virt.c >>>> +++ b/hw/arm/virt.c >>>> @@ -95,6 +95,7 @@ >>>> #include "hw/cxl/cxl.h" >>>> #include "hw/cxl/cxl_host.h" >>>> #include "qemu/guest-random.h" >>>> +#include "hw/watchdog/sbsa_gwdt.h" >>>> >>>> static GlobalProperty arm_virt_compat_defaults[] = { >>>> { TYPE_VIRTIO_IOMMU_PCI, "aw-bits", "48" }, >>>> @@ -3474,6 +3475,7 @@ static void virt_machine_class_init(ObjectClass *oc, const void *data) >>>> machine_class_allow_dynamic_sysbus_dev(mc, TYPE_RAMFB_DEVICE); >>>> machine_class_allow_dynamic_sysbus_dev(mc, TYPE_UEFI_VARS_SYSBUS); >>>> machine_class_allow_dynamic_sysbus_dev(mc, TYPE_ARM_SMMUV3); >>>> + machine_class_allow_dynamic_sysbus_dev(mc, TYPE_WDT_SBSA); >>>> #ifdef CONFIG_TPM >>>> machine_class_allow_dynamic_sysbus_dev(mc, TYPE_TPM_TIS_SYSBUS); >>>> #endif >>>> diff --git a/hw/core/sysbus-fdt.c b/hw/core/sysbus-fdt.c >>>> index 89d0c46445..76aa1bc579 100644 >>>> --- a/hw/core/sysbus-fdt.c >>>> +++ b/hw/core/sysbus-fdt.c >>>> @@ -36,6 +36,7 @@ >>>> #include "hw/display/ramfb.h" >>>> #include "hw/uefi/var-service-api.h" >>>> #include "hw/arm/fdt.h" >>>> +#include "hw/watchdog/sbsa_gwdt.h" >>>> >>>> /* >>>> * internal struct that contains the information to create dynamic >>>> @@ -118,6 +119,36 @@ static int add_uefi_vars_node(SysBusDevice *sbdev, void *opaque) >>>> return 0; >>>> } >>>> >>>> +static int add_sbsa_gwdt_node(SysBusDevice *sbdev, void *opaque) >>>> +{ >>>> + PlatformBusFDTData *data = opaque; >>>> + PlatformBusDevice *pbus = data->pbus; >>>> + const char *parent_node = data->pbus_node_name; >>>> + void *fdt = data->fdt; >>>> + uint64_t cbase, rbase; >>>> + char *nodename; >>>> + int irq; >>>> + >>>> + cbase = platform_bus_get_mmio_addr(pbus, sbdev, 1); /* control frame */ >>>> + rbase = platform_bus_get_mmio_addr(pbus, sbdev, 0); /* refresh frame */ >>>> + irq = platform_bus_get_irqn(pbus, sbdev, 0); >>>> + >>>> + nodename = g_strdup_printf("%s/watchdog@%" PRIx64, parent_node, cbase); >>>> + qemu_fdt_add_subnode(fdt, nodename); >>>> + >>>> + qemu_fdt_setprop_string(fdt, nodename, "compatible", "arm,sbsa-gwdt"); >>>> + qemu_fdt_setprop_cells(fdt, nodename, "reg", >>>> + cbase, SBSA_GWDT_CMMIO_SIZE, >>>> + rbase, SBSA_GWDT_RMMIO_SIZE); >>>> + qemu_fdt_setprop_cells(fdt, nodename, "interrupts", >>>> + GIC_FDT_IRQ_TYPE_SPI, data->irq_start + irq, >>>> + GIC_FDT_IRQ_FLAGS_LEVEL_HI); >>>> + qemu_fdt_setprop_cell(fdt, nodename, "timeout-sec", 30); >>>> + >>>> + g_free(nodename); >>>> + return 0; >>>> +} >>>> + >>>> static int no_fdt_node(SysBusDevice *sbdev, void *opaque) >>>> { >>>> return 0; >>>> @@ -140,6 +171,7 @@ static const BindingEntry bindings[] = { >>>> TYPE_BINDING(TYPE_ARM_SMMUV3, no_fdt_node), >>>> TYPE_BINDING(TYPE_RAMFB_DEVICE, no_fdt_node), >>>> TYPE_BINDING(TYPE_UEFI_VARS_SYSBUS, add_uefi_vars_node), >>>> + TYPE_BINDING(TYPE_WDT_SBSA, add_sbsa_gwdt_node), >>>> TYPE_BINDING("", NULL), /* last element */ >>>> }; >>>> >>>> diff --git a/hw/watchdog/sbsa_gwdt.c b/hw/watchdog/sbsa_gwdt.c >>>> index acb970e8b3..c4dd8005b7 100644 >>>> --- a/hw/watchdog/sbsa_gwdt.c >>>> +++ b/hw/watchdog/sbsa_gwdt.c >>>> @@ -285,6 +285,7 @@ static void wdt_sbsa_gwdt_class_init(ObjectClass *klass, const void *data) >>>> dc->realize = wdt_sbsa_gwdt_realize; >>>> device_class_set_legacy_reset(dc, wdt_sbsa_gwdt_reset); >>>> dc->hotpluggable = false; >>>> + dc->user_creatable = true; >>>> set_bit(DEVICE_CATEGORY_WATCHDOG, dc->categories); >>>> dc->vmsd = &vmstate_sbsa_gwdt; >>>> dc->desc = "SBSA-compliant generic watchdog device"; >>> >> >