From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by mx.groups.io with SMTP id smtpd.web10.51656.1643734631224588905 for ; Tue, 01 Feb 2022 08:57:11 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=jKx7JqQO; spf=pass (domain: kernel.org, ip: 139.178.84.217, mailfrom: ardb@kernel.org) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 894F960AFF for ; Tue, 1 Feb 2022 16:57:10 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id F09B2C340EE for ; Tue, 1 Feb 2022 16:57:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1643734630; bh=anUVMgvPB+zZclWSdIxgXKoyeThiQpe0BLUivMExd24=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=jKx7JqQOfESF/NkW85V9QKJ62Q1NoMwA1o1IYB7E1veh3Z5Q3Kq5bcz2FENrMn73u 9UhWRJ6z1/fDpjFyX8rFYX+TKj70jSMlrFiouW0J9EAYkx7p+cx3sfzhul61fEZYdt BuDYmIgZTw+ke7Fg5i4W8OXj5X4QcJWxgBemUMjL4WEvDfQXKLnPYElXv60Eco59F7 vPj7zowUKmDtTHywi+preWyhTVa0E2hah8x0yHcVzL3NZ6z9Yii/CIGglGpWIdn2yy IIyJMotg5noGu+DGCieyg4wdSexMee2s++YCH0lnjInw9JPhxrXDvFAKWDfUuOIVPR GHRdrIvNS2i9w== Received: by mail-wr1-f49.google.com with SMTP id v13so33175023wrv.10 for ; Tue, 01 Feb 2022 08:57:09 -0800 (PST) X-Gm-Message-State: AOAM533iVY2hf5QIDr1J5sqAn0Bsr0mep43cXSAnxX/G++ecLDyhfAjM 80OPT6PmG4+Ug38vvjsbULE8zo9L7zQwatRtv7E= X-Google-Smtp-Source: ABdhPJzSOc/p8QXf9iKp1vUsdoVW+iQ3g90oSnxrxewXvJLfcvnWFU/tfIpZGZmwfdJrgV19CvcE8dIfQ3kFp2F7e1U= X-Received: by 2002:a05:6000:1c9:: with SMTP id t9mr22608756wrx.550.1643734628354; Tue, 01 Feb 2022 08:57:08 -0800 (PST) MIME-Version: 1.0 References: <20220128154103.20752-1-Pierre.Gondois@arm.com> <20220128154103.20752-6-Pierre.Gondois@arm.com> <52503b9c-c820-15b3-65c5-f2cc110f5027@quicinc.com> <60a12a54-1860-f7b8-f4f1-144cf19f347f@arm.com> In-Reply-To: From: "Ard Biesheuvel" Date: Tue, 1 Feb 2022 17:56:56 +0100 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [edk2-devel] [PATCH v3 5/8] ArmVirtPkg/Kvmtool: Add DSDT ACPI table To: Pierre Gondois Cc: Sami Mujawar , Rebecca Cran , "devel@edk2.groups.io" , Ard Biesheuvel , nd Content-Type: text/plain; charset="UTF-8" On Tue, 1 Feb 2022 at 17:56, Ard Biesheuvel wrote: > > On Tue, 1 Feb 2022 at 17:55, Pierre Gondois wrote: > > > > Hi Rebecca, > > > > On 1/31/22 4:21 PM, Sami Mujawar wrote: > > > Hi Rebecca, > > > > > > Thanks for catching this. > > > > > > I think we also need to adda check in Acpiview to report this issue. However, > > > that would be another patch series. > > > > > > Regards, > > > > > > Sami Mujawar > > > > > > *From: *Rebecca Cran > > > *Date: *Monday, 31 January 2022 at 15:17 > > > *To: *devel@edk2.groups.io , Pierre Gondois > > > > > > *Cc: *Ard Biesheuvel , Sami Mujawar > > > > > > *Subject: *Re: [edk2-devel] [PATCH v3 5/8] ArmVirtPkg/Kvmtool: Add DSDT ACPI table > > > > > > On 1/28/22 08:41, PierreGondois wrote: > > > > > >> + Differentiated System Description Table Fields (DSDT) > > >> + > > >> + Copyright (c) 2021 - 2022, ARM Ltd. All rights reserved.
> > >> + SPDX-License-Identifier: BSD-2-Clause-Patent > > >> + > > >> +**/ > > >> + > > >> +DefinitionBlock ("DsdtTable.aml", "DSDT", 1, "ARMLTD", "ARM-KVMT", 1) { > > > > > > The Revision field should probably be 2, not 1. ACPI 6.4 says: > > > > > > > > > 2. This field also sets the global integer width for the AML > > > interpreter. Values less than two will cause the inter- > > > preter to use 32-bit integers and math. Values of two > > > and greater will cause the interpreter to use full 64-bit > > > integers and math. > > > > > > -- > > > Rebecca Cran > > > > > > > Yes indeed. I forgot to add it in the v4... I will send a v5. > > Please give me a minute before sending another version. > > I am still seeing No ACPI PMU IRQ for CPU26 errors and I am trying to figure out why.