From nobody Sat Apr 20 00:31:43 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org ARC-Seal: i=1; a=rsa-sha256; t=1560356814; cv=none; d=zoho.com; s=zohoarc; b=Dr3YCveXzyrP4QF4oH6+I97mz70/gJaX4om0lJxwlCR4lCT6xSb6dOlYPMzcE9uTMGtcxwX50M992f/HwSDbXn2mRybmvtfX79oJSmmeFeqOMiTNps9+gCzrf7RkKFR9+OjAbXmuxni1s+tBLaAULVR+J9reWt80+8JpanqUkmc= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1560356814; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Sender:Subject:To:ARC-Authentication-Results; bh=E7/5QnD32p04Dk7MAPgD/U/Bc/rRYgxb0SyZrtLnuF8=; b=AGAS7ADFV8iIpPUNq/2ZT8N32PgeYyOqEulv/3DOlJ1VhZQyPxaT7dyeSygvg1nK1CoYQQ7AKQN0f65Ksp0idYvr3Azs90WzirW/IyD0eHH241LgnnqMd1zKHn6AOn4XjxA/ckPdEgnggfb7qkEGDivq0EQWk9p8PV5o0Nsb+B4= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1560356814969508.71767139808594; Wed, 12 Jun 2019 09:26:54 -0700 (PDT) Received: from localhost ([::1]:33678 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hb65J-0007O6-UC for importer@patchew.org; Wed, 12 Jun 2019 12:26:49 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:51351) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hb62m-00060b-4R for qemu-devel@nongnu.org; Wed, 12 Jun 2019 12:24:13 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hb62l-0007ZQ-6p for qemu-devel@nongnu.org; Wed, 12 Jun 2019 12:24:12 -0400 Received: from 9.mo173.mail-out.ovh.net ([46.105.72.44]:46209) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1hb62l-0007Wu-00 for qemu-devel@nongnu.org; Wed, 12 Jun 2019 12:24:11 -0400 Received: from player691.ha.ovh.net (unknown [10.108.42.142]) by mo173.mail-out.ovh.net (Postfix) with ESMTP id 17E1710D825 for ; Wed, 12 Jun 2019 18:24:07 +0200 (CEST) Received: from kaod.org (lfbn-1-10649-41.w90-89.abo.wanadoo.fr [90.89.235.41]) (Authenticated sender: clg@kaod.org) by player691.ha.ovh.net (Postfix) with ESMTPSA id 044B36DB934D; Wed, 12 Jun 2019 16:24:02 +0000 (UTC) From: =?UTF-8?q?C=C3=A9dric=20Le=20Goater?= To: David Gibson Date: Wed, 12 Jun 2019 18:23:57 +0200 Message-Id: <20190612162357.29566-1-clg@kaod.org> X-Mailer: git-send-email 2.21.0 MIME-Version: 1.0 X-Ovh-Tracer-Id: 2821223695458733030 X-VR-SPAMSTATE: OK X-VR-SPAMSCORE: -100 X-VR-SPAMCAUSE: gggruggvucftvghtrhhoucdtuddrgeduuddrudehjedguddthecutefuodetggdotefrodftvfcurfhrohhfihhlvgemucfqggfjpdevjffgvefmvefgnecuuegrihhlohhuthemucehtddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmd Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 46.105.72.44 Subject: [Qemu-devel] [PATCH] ppc/pnv: fix StoreEOI activation X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: qemu-ppc@nongnu.org, qemu-devel@nongnu.org, =?UTF-8?q?C=C3=A9dric=20Le=20Goater?= Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" The firmware (skiboot) of the PowerNV machines can configure the XIVE interrupt controller to activate StoreEOI on the ESB pages of the interrupts. This feature lets software do an EOI with a store instead of a load. It is not activated today on P9 for rare race condition issues but it should be on future processors. Nevertheless, QEMU has a model for StoreEOI which can be used today by experimental firmwares. But, the use of object_property_set_int() in the PnvXive model is incorrect and crashes QEMU. Replace it with a direct access to the ESB flags of the XiveSource object modeling the internal sources of the interrupt controller. Signed-off-by: C=C3=A9dric Le Goater --- hw/intc/pnv_xive.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/hw/intc/pnv_xive.c b/hw/intc/pnv_xive.c index bb0877cbdf3b..12c91adcd27e 100644 --- a/hw/intc/pnv_xive.c +++ b/hw/intc/pnv_xive.c @@ -780,8 +780,7 @@ static void pnv_xive_ic_reg_write(void *opaque, hwaddr = offset, * support recently though) */ if (val & (VC_SBC_CONF_CPLX_CIST | VC_SBC_CONF_CIST_BOTH)) { - object_property_set_int(OBJECT(&xive->ipi_source), - XIVE_SRC_STORE_EOI, "flags", &error_fa= tal); + xive->ipi_source.esb_flags |=3D XIVE_SRC_STORE_EOI; } break; =20 --=20 2.21.0