From nobody Tue May 7 21:14:15 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=fail; 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; dmarc=fail(p=none dis=none) header.from=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1593355145; cv=none; d=zohomail.com; s=zohoarc; b=YzIz/4t9BchQ4L0J5u5sbKn5MBfZXCGFIf2lMeJ4disB9CQsO6gymdw1zfczQeBc+z0ebxlAvVeOQRGhzP4XxuPz5iKzC8UpLeVQW7v9ausBhQhrgcRy3xhtUI3NIZweDGUtm6c92i2j6tPz0UNYSU4ZXrMWE9LR4l1q0RAait4= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1593355145; 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=ZjKuW1mQwpJiy6HB5bGM9HstZflvVeFdcxiGRekmHQY=; b=gqVpGpW4PpYa5VrI1b6O/Js1/eqdaxcWazjzAT+ucVSJqjK+jHF1UT5DVu2e8JSTM8FQLqmA3sEvSHoUdsJQTBjSRi/f4JEPx0HUZg6fAu49e/m/VTiHsP5uNZ5B5J1AnHegFmBetjZiO2oJifoZSpFfM2QQZvJO2q7PJzuNfRI= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=fail; 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; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1593355145541257.0406912249837; Sun, 28 Jun 2020 07:39:05 -0700 (PDT) Received: from localhost ([::1]:37456 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jpYSW-0002Ef-6O for importer@patchew.org; Sun, 28 Jun 2020 10:39:04 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:39384) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1jpYIy-0005fe-Lb; Sun, 28 Jun 2020 10:29:12 -0400 Received: from mail-ed1-x542.google.com ([2a00:1450:4864:20::542]:42612) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1jpYIx-0005Tz-3c; Sun, 28 Jun 2020 10:29:12 -0400 Received: by mail-ed1-x542.google.com with SMTP id z17so10711236edr.9; Sun, 28 Jun 2020 07:29:10 -0700 (PDT) Received: from Tank.localdomain (82-73-161-131.cable.dynamic.v4.ziggo.nl. [82.73.161.131]) by smtp.gmail.com with ESMTPSA id kt4sm13025837ejb.48.2020.06.28.07.29.08 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 28 Jun 2020 07:29:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=ZjKuW1mQwpJiy6HB5bGM9HstZflvVeFdcxiGRekmHQY=; b=ngVPIuStvwTnANyO6vMcKp3J1eVRVlMBoTIe0qiDzL73gKAm01b+PwM2ZkEFb7mhi9 6kr0ZTNFubjwqelMfBMC+hynS13TZ+D0vlT3hHRf8uEAExC3VAnluZxFPvaBkpqnJmRQ wAxnggkmoYxvug/A8Q+OWdtNHQWATy6vhotvGN/hBEPtbLIe1i6N2VnOEpO4/gOrMd2G vYA+nwCzEGbc9HtrWp+Hj5VZy3IRI/mz7m/Yu8mFgNHPMuZs1FXOkYyiHdWtiVW/nXj8 4lHmem3Ur7EWTts/aNpT3F7gyGLYnxMsbQF/n0lEF5CojQCAmspWLQtXJx7lCfMhZzsd qMJg== 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=ZjKuW1mQwpJiy6HB5bGM9HstZflvVeFdcxiGRekmHQY=; b=rk0Fq2IhWA5fn64bvSxtlBOFM/8lgT/bZ7oUg5CRW7Kr1g/2nvXYBkvh/ILPukZ/wM SazFptgouGIyqfjGcx5gIkZ+8siRsZNii6Am+/fvkqMnuUmb0Hkp8or8PDD5Jq9HewsM HI1yEGgNn1n6R5OpCOPMXuC2Ggjf5etpa9bzGttKzWOcogI1YRvWAcC9KtV5XG5sZQ78 /RM6uQqZSJT2smfHpqs3fgyTa0pqRrCquuPjOvSmgBsXk2snN4TiMF8pu0Bi+3/HhCgc 97SHKJKO0x45qDHP/pzNgR0Xv9cxhYcBu0Y1g7xh/HupbAGlXPrjLmlKZP49JaiKiMWH 1XzA== X-Gm-Message-State: AOAM533Adecu9ME/d3lMDIvM6MT2iwjVhsZAlrcSBF6MgLpKhdBQGMym XdotjYPyjBcHX2vH+7jhhFA= X-Google-Smtp-Source: ABdhPJyfc9EvczjpeF+nbOV792W88dwi85PMpl8LlyBitnu4QvgE4tt+Qv8c0Hfw/YNgnkXb5SpW4g== X-Received: by 2002:a50:cf05:: with SMTP id c5mr13511180edk.232.1593354549304; Sun, 28 Jun 2020 07:29:09 -0700 (PDT) From: erik-smit To: erik.lucas.smit@gmail.com Subject: [PATCH v2] ftgmac100: fix dblac write test Date: Sun, 28 Jun 2020 16:26:59 +0200 Message-Id: <20200628142658.29264-1-erik.lucas.smit@gmail.com> X-Mailer: git-send-email 2.25.1 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=2a00:1450:4864:20::542; envelope-from=erik.lucas.smit@gmail.com; helo=mail-ed1-x542.google.com X-detected-operating-system: by eggs.gnu.org: No matching host in p0f cache. That's all we know. X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, 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: Peter Maydell , Andrew Jeffery , Jason Wang , "open list:All patches CC here" , "open list:ASPEED BMCs" , =?UTF-8?q?C=C3=A9dric=20Le=20Goater?= , Joel Stanley Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) Content-Type: text/plain; charset="utf-8" The test of the write of the dblac register was testing the old value instead of the new value. This would accept the write of an invalid value but subsequently refuse any following valid writes. Signed-off-by: erik-smit Reviewed-by: Andrew Jeffery Reviewed-by: C=C3=A9dric Le Goater --- Changes since v1: Changed %ld to HWADDR_PRIx to fix building on mingw hw/net/ftgmac100.c | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/hw/net/ftgmac100.c b/hw/net/ftgmac100.c index 043ba61b86..b69e3dd14e 100644 --- a/hw/net/ftgmac100.c +++ b/hw/net/ftgmac100.c @@ -810,16 +810,18 @@ static void ftgmac100_write(void *opaque, hwaddr addr, s->phydata =3D value & 0xffff; break; case FTGMAC100_DBLAC: /* DMA Burst Length and Arbitration Control */ - if (FTGMAC100_DBLAC_TXDES_SIZE(s->dblac) < sizeof(FTGMAC100Desc)) { + if (FTGMAC100_DBLAC_TXDES_SIZE(value) < sizeof(FTGMAC100Desc)) { qemu_log_mask(LOG_GUEST_ERROR, - "%s: transmit descriptor too small : %d bytes\n", - __func__, FTGMAC100_DBLAC_TXDES_SIZE(s->dblac)); + "%s: transmit descriptor too small: %" HWADDR_PR= Ix + " bytes\n", __func__, + FTGMAC100_DBLAC_TXDES_SIZE(value)); break; } - if (FTGMAC100_DBLAC_RXDES_SIZE(s->dblac) < sizeof(FTGMAC100Desc)) { + if (FTGMAC100_DBLAC_RXDES_SIZE(value) < sizeof(FTGMAC100Desc)) { qemu_log_mask(LOG_GUEST_ERROR, - "%s: receive descriptor too small : %d bytes\n", - __func__, FTGMAC100_DBLAC_RXDES_SIZE(s->dblac)); + "%s: receive descriptor too small : %" HWADDR_PR= Ix + " bytes\n", __func__, + FTGMAC100_DBLAC_RXDES_SIZE(value)); break; } s->dblac =3D value; --=20 2.25.1