From nobody Mon Feb 9 23:31:26 2026 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; spf=pass (zohomail.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=1591855181; cv=none; d=zohomail.com; s=zohoarc; b=Hg3JeeFz7uR4e+IgXhg0gubmtT324s8VyoP5KVATKUuFOjyzGmWxntAyzzPiexbIJhN8fMb8+YXDy1rVbqAp3DlBDX8Z3WVARM+hdHT4Iut8XobeGBwoEt2u/17rbLnMA6XdG6sVnsocYPQdrRK33RA/5IIQmN74T6KDBuqca6U= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1591855181; h=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; bh=hi7asyT08MJpRVZtlrtD3h0DgWR9WoNxciZGNbdqAo0=; b=C0+ZgcNBuXTX0o/c6GkS+T0HOD4xb5b0/v0dW+2vOYwPqGYK0QgF3eYHbAKZyeTuaVTWdF5N4PI0afWf//PzNVnWnuSQ04i6nW39yeiMj2d2OAbNo2bOQqJuSwc/zdL7Yoxb/BTyd3Rvc4RY8NDasX/NrniArQPo7+EtsWZIyoI= ARC-Authentication-Results: i=1; mx.zohomail.com; spf=pass (zohomail.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 1591855180822377.9036394105244; Wed, 10 Jun 2020 22:59:40 -0700 (PDT) Received: from localhost ([::1]:57468 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jjGFX-00046p-GQ for importer@patchew.org; Thu, 11 Jun 2020 01:59:39 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:53684) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1jjGEJ-0002oN-NE for qemu-devel@nongnu.org; Thu, 11 Jun 2020 01:58:23 -0400 Received: from mail-wm1-f65.google.com ([209.85.128.65]:38536) by eggs.gnu.org with esmtps (TLS1.3:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1jjGEH-0002YB-6b for qemu-devel@nongnu.org; Thu, 11 Jun 2020 01:58:23 -0400 Received: by mail-wm1-f65.google.com with SMTP id f185so3806478wmf.3 for ; Wed, 10 Jun 2020 22:58:14 -0700 (PDT) Received: from thl530.multi.box (pd9e83fc6.dip0.t-ipconnect.de. [217.232.63.198]) by smtp.gmail.com with ESMTPSA id 88sm3363831wre.45.2020.06.10.22.58.11 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 10 Jun 2020 22:58:12 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=hi7asyT08MJpRVZtlrtD3h0DgWR9WoNxciZGNbdqAo0=; b=FH+CRdFzW7WyOjGvzhMR7zeMnzfSjGJFAlr4NxOB8sUu2G83fRZxBXe/lIm47g8JSs afrqyPZk4ZltH139KeCLTxOSvovUkhkGE+hMk4LA6j5HQFK2Qd/cOoupBtNcJ49FDdOG mylJsJU/WlmFXUs3zugdoekfKBRE7i96FaQ8/x5pPYQOzobe5/Fkbyt1Rfqs9iyDVKAz q55IIsQ0+j9UpvzPpMPpAAWYu4YD3SfMETczqz0y/cWdCWjgHMjOiwEXo/v8V5NgTiU5 MF3T+eCaVgmFu1ywQtCHQ/nKWVS+FJqA2apOrNsLMiktbmytOSxFhLxfuPjseIgX+TBt kKqw== X-Gm-Message-State: AOAM532a7rU205RTJUYjBd81xBqlk07h2DMgHSKgneazQkwxgsK4nqUl r0xVgFxN9HySVMPM/BMkT9qL9zK2 X-Google-Smtp-Source: ABdhPJxMSJgCgXVzhx/mz8OcGxNaQwCT85U7DOK7EcmKuQXnH6nbtUjpxi8d19JqPHkFjMI1eaR4zw== X-Received: by 2002:a1c:de82:: with SMTP id v124mr6496058wmg.89.1591855092887; Wed, 10 Jun 2020 22:58:12 -0700 (PDT) From: Thomas Huth To: qemu-devel@nongnu.org Subject: [PATCH] hw/m68k/mcf5206: Replace remaining hw_error()s by qemu_log_mask() Date: Thu, 11 Jun 2020 07:58:07 +0200 Message-Id: <20200611055807.15921-1-huth@tuxfamily.org> X-Mailer: git-send-email 2.26.2 MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.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; Received-SPF: pass client-ip=209.85.128.65; envelope-from=th.huth@gmail.com; helo=mail-wm1-f65.google.com X-detected-operating-system: by eggs.gnu.org: First seen = 2020/06/11 01:58:13 X-ACL-Warn: Detected OS = Linux 2.2.x-3.x [generic] [fuzzy] X-Spam_score_int: -8 X-Spam_score: -0.9 X-Spam_bar: / X-Spam_report: (-0.9 / 5.0 requ) BAYES_00=-1.9, FREEMAIL_FORGED_FROMDOMAIN=1, FREEMAIL_FROM=0.001, HEADER_FROM_DIFFERENT_DOMAINS=1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-1, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=_AUTOLEARN X-Spam_action: no action 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: =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , laurent@vivier.eu Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" hw_error() dumps the CPU state and exits QEMU. This is ok during initial code development (to see where the guest code is currently executing), but it is certainly not the desired behavior that we want to present to normal users, and it can also cause trouble when e.g. fuzzing devices. Thus let's replace these hw_error()s by qemu_log_mask()s instead. Signed-off-by: Thomas Huth Reviewed-by: Laurent Vivier Reviewed-by: Philippe Mathieu-Daud=C3=A9 --- hw/m68k/mcf5206.c | 39 ++++++++++++++++++++++++++++----------- 1 file changed, 28 insertions(+), 11 deletions(-) diff --git a/hw/m68k/mcf5206.c b/hw/m68k/mcf5206.c index a2fef04f8e..94a37a1a46 100644 --- a/hw/m68k/mcf5206.c +++ b/hw/m68k/mcf5206.c @@ -10,7 +10,6 @@ #include "qemu/error-report.h" #include "qemu/log.h" #include "cpu.h" -#include "hw/hw.h" #include "hw/irq.h" #include "hw/m68k/mcf.h" #include "qemu/timer.h" @@ -69,10 +68,16 @@ static void m5206_timer_recalibrate(m5206_timer_state *= s) if (mode =3D=3D 2) prescale *=3D 16; =20 - if (mode =3D=3D 3 || mode =3D=3D 0) - hw_error("m5206_timer: mode %d not implemented\n", mode); - if ((s->tmr & TMR_FRR) =3D=3D 0) - hw_error("m5206_timer: free running mode not implemented\n"); + if (mode =3D=3D 3 || mode =3D=3D 0) { + qemu_log_mask(LOG_UNIMP, "m5206_timer: mode %d not implemented\n", + mode); + goto exit; + } + if ((s->tmr & TMR_FRR) =3D=3D 0) { + qemu_log_mask(LOG_UNIMP, + "m5206_timer: free running mode not implemented\n"); + goto exit; + } =20 /* Assume 66MHz system clock. */ ptimer_set_freq(s->timer, 66000000 / prescale); @@ -391,7 +396,9 @@ static uint32_t m5206_mbar_readb(void *opaque, hwaddr o= ffset) m5206_mbar_state *s =3D (m5206_mbar_state *)opaque; offset &=3D 0x3ff; if (offset >=3D 0x200) { - hw_error("Bad MBAR read offset 0x%x", (int)offset); + qemu_log_mask(LOG_GUEST_ERROR, "Bad MBAR read offset 0x%" HWADDR_P= RIX, + offset); + return 0; } if (m5206_mbar_width[offset >> 2] > 1) { uint16_t val; @@ -410,7 +417,9 @@ static uint32_t m5206_mbar_readw(void *opaque, hwaddr o= ffset) int width; offset &=3D 0x3ff; if (offset >=3D 0x200) { - hw_error("Bad MBAR read offset 0x%x", (int)offset); + qemu_log_mask(LOG_GUEST_ERROR, "Bad MBAR read offset 0x%" HWADDR_P= RIX, + offset); + return 0; } width =3D m5206_mbar_width[offset >> 2]; if (width > 2) { @@ -434,7 +443,9 @@ static uint32_t m5206_mbar_readl(void *opaque, hwaddr o= ffset) int width; offset &=3D 0x3ff; if (offset >=3D 0x200) { - hw_error("Bad MBAR read offset 0x%x", (int)offset); + qemu_log_mask(LOG_GUEST_ERROR, "Bad MBAR read offset 0x%" HWADDR_P= RIX, + offset); + return 0; } width =3D m5206_mbar_width[offset >> 2]; if (width < 4) { @@ -458,7 +469,9 @@ static void m5206_mbar_writeb(void *opaque, hwaddr offs= et, int width; offset &=3D 0x3ff; if (offset >=3D 0x200) { - hw_error("Bad MBAR write offset 0x%x", (int)offset); + qemu_log_mask(LOG_GUEST_ERROR, "Bad MBAR write offset 0x%" HWADDR_= PRIX, + offset); + return; } width =3D m5206_mbar_width[offset >> 2]; if (width > 1) { @@ -482,7 +495,9 @@ static void m5206_mbar_writew(void *opaque, hwaddr offs= et, int width; offset &=3D 0x3ff; if (offset >=3D 0x200) { - hw_error("Bad MBAR write offset 0x%x", (int)offset); + qemu_log_mask(LOG_GUEST_ERROR, "Bad MBAR write offset 0x%" HWADDR_= PRIX, + offset); + return; } width =3D m5206_mbar_width[offset >> 2]; if (width > 2) { @@ -510,7 +525,9 @@ static void m5206_mbar_writel(void *opaque, hwaddr offs= et, int width; offset &=3D 0x3ff; if (offset >=3D 0x200) { - hw_error("Bad MBAR write offset 0x%x", (int)offset); + qemu_log_mask(LOG_GUEST_ERROR, "Bad MBAR write offset 0x%" HWADDR_= PRIX, + offset); + return; } width =3D m5206_mbar_width[offset >> 2]; if (width < 4) { --=20 2.26.2