From nobody Thu Dec 18 18:02:05 2025 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 37BC318A95A for ; Thu, 29 Aug 2024 20:49:10 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=170.10.133.124 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1724964552; cv=none; b=sovOM5Zrk4dOag8NmiF/ucBGnNB0mCP9sgym/4CQxM1TbR3j4E0U2tjhJi5yoEudIPJ8nu/uAycPBtMoIDg8cpsvMDs4S9l+JT5bC7jFXoBPvNZS651fP+naI2ypxB8yCVe0s481/aHKrEmvMXLgjUfevQCJw/4dv5EWlVF5HDI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1724964552; c=relaxed/simple; bh=6V1zQyb4JJQe1x1GsfPpxi5B8rqhMNM4ly+Ac85hImg=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:To:Cc; b=iz+1gAOx7iTThU4hgQVyiSZWD/ZBTuGKt+DqKPLgxpX2YMfW8QFZ5JGZ5VfsnxPz8JJ0s8FapVpfMJxJd1oL4cUtnef5w3rn8a7G4dB8ovW+oPplPmJpUGtD0hffApNwK+dlax+hM0JAZBIBrKxBxcwOsBJWiakkBQY0Yr4nPTI= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=redhat.com; spf=pass smtp.mailfrom=redhat.com; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b=Fh+A9Fzu; arc=none smtp.client-ip=170.10.133.124 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=redhat.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=redhat.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="Fh+A9Fzu" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1724964550; 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; bh=VFLjaq4t+p4ySJaRjHHMD+djaojUeqjD8JR+7Pp4jkk=; b=Fh+A9FzuG+xXJErcF1Kwho8GLVSEvaOJnITbLH2eOpHJ+ZLMxyryXBCMjb/1SvJEXq4CB9 h0p9glWolRNmOllTKyMAKLNT48xXUF2ozzzCmfQJ39LOTWBKgUcJsef2Doo5lhOzIgnD8r f3A8H+fqpPHLiDBoOUGravVX/DBYUhk= Received: from mail-qt1-f199.google.com (mail-qt1-f199.google.com [209.85.160.199]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-648-qbmSN8G5Pe-r6Sc8HUzSyw-1; Thu, 29 Aug 2024 16:49:08 -0400 X-MC-Unique: qbmSN8G5Pe-r6Sc8HUzSyw-1 Received: by mail-qt1-f199.google.com with SMTP id d75a77b69052e-45689398751so5593331cf.0 for ; Thu, 29 Aug 2024 13:49:08 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1724964548; x=1725569348; h=cc:to:message-id:content-transfer-encoding:mime-version:subject :date:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=VFLjaq4t+p4ySJaRjHHMD+djaojUeqjD8JR+7Pp4jkk=; b=sQypSswhd1Jznl/GDhN32YSY4rmo0MnQVLzzn0EaQMSp56J8muYmmb2E6dCwcn4UAY IRFJstfkCtzcX48wgEae5xqUhzxfo/fsAfyxzk7MT3zUQSaRYTSGhMNAk9hQDY1LyyoI aRgIkJFvaxb3Acka0jlCB+YXr1wgr43ALyhJdIQE+BQcF46QUu+z0uoEjmgHTvloLQat ixs5qstvV8ck9SaJJ37RZmwLQLS37VqmRUVtyuNI7lVJY542LGxJuOp3/nz7DT9zXREF pkWQ0AUsG03Z7T4ujEp+tIgWaECoZMcxJGmjmkxehngdKu7b61t1+ihhX1xosdKeZXpZ R75Q== X-Forwarded-Encrypted: i=1; AJvYcCVHsGPERUGUWVg63ctx7YUPclIjZ3c4rFOSzcW00ngY6mfgHcR3DBqRB664azFW4A6A/KLNcdT4mjOCuis=@vger.kernel.org X-Gm-Message-State: AOJu0YwYMWDgjI8xvrXUd0Ib6xdqdZGnY6wxnDt/hLT6knr5jyKb0N3x EzjfWXmiPoYTvqQcYGaXx2LoLYt9vbyvcoEpqt/Ieq5BXKyivs8EsfcT0tlKNQcsCEuMVjtyFvn Mp1Wdmv7N92gbqiIaLbAxn+v25Rg9AuqgaH76F4FMkuEC/FJIDE0t37SsYJxEuw== X-Received: by 2002:a05:622a:578f:b0:453:5d83:6275 with SMTP id d75a77b69052e-4567f580d8fmr43858581cf.34.1724964547715; Thu, 29 Aug 2024 13:49:07 -0700 (PDT) X-Google-Smtp-Source: AGHT+IGeb+yBjMEz9BPdapOW3IkPpcjpvG1lfzUxlrzusYIy/pZk9xn7Op0zmI0YQmjcAC+y5+roSg== X-Received: by 2002:a05:622a:578f:b0:453:5d83:6275 with SMTP id d75a77b69052e-4567f580d8fmr43858371cf.34.1724964547321; Thu, 29 Aug 2024 13:49:07 -0700 (PDT) Received: from x1gen2nano.local ([2600:1700:1ff0:d0e0::33]) by smtp.gmail.com with ESMTPSA id d75a77b69052e-45682ce3058sm8168951cf.52.2024.08.29.13.49.05 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 29 Aug 2024 13:49:06 -0700 (PDT) From: Andrew Halaney Date: Thu, 29 Aug 2024 15:48:44 -0500 Subject: [PATCH net-next v2] net: stmmac: drop the ethtool begin() callback Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20240829-stmmac-no-ethtool-begin-v2-1-a11b497a7074@redhat.com> X-B4-Tracking: v=1; b=H4sIAKze0GYC/3WNwQqDMBBEf0X23C3JNgj21P8oHmzcmIWalCSII v57g/Tawxwew7zZIXMSznBvdki8SJYYKtClAeuHMDHKWBlIkVGGDOYyz4PFEJGLLzG+8cWTBHQ 31TpyusZAXX8SO1lP8xMCFwy8Fuhr4yWXmLbzctFn/7N3f+2LRo3K2JY6q6029Eg8+qFcbZyhP 47jC+FGFz3JAAAA To: Alexandre Torgue , Jose Abreu , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Maxime Coquelin , Abhishek Chauhan , Sagar Cheluvegowda , Dmitry Dolenko Cc: netdev@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Andrew Halaney X-Mailer: b4 0.14.0 This callback doesn't seem to serve much purpose, and prevents things like: - systemd.link files from disabling autonegotiation - carrier detection in NetworkManager - any ethtool setting prior to userspace bringing the link up. The only fear I can think of is accessing unclocked resources due to pm_runtime, but ethtool ioctls handle that as of commit f32a21376573 ("ethtool: runtime-resume netdev parent before ethtool ioctl o= ps") Reviewed-by: Dmitry Dolenko Tested-by: Dmitry Dolenko Signed-off-by: Andrew Halaney Reviewed-by: Alexander Lobakin --- Changes in v2: - Rebase on next-20240829 - Drop RFC/RFT tags, add Dmitry's Review/Test tag - Link to v1: https://lore.kernel.org/r/20240429-stmmac-no-ethtool-begin-v1= -1-04c629c1c142@redhat.com I'd still like a few more folks to test this to feel more confident that I'm not breaking anyone, but at least I've gotten one on list feedback and one off list that its fine for them. --- drivers/net/ethernet/stmicro/stmmac/stmmac_ethtool.c | 8 -------- 1 file changed, 8 deletions(-) diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_ethtool.c b/drivers= /net/ethernet/stmicro/stmmac/stmmac_ethtool.c index 7008219fd88d..220c582904f4 100644 --- a/drivers/net/ethernet/stmicro/stmmac/stmmac_ethtool.c +++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_ethtool.c @@ -438,13 +438,6 @@ static void stmmac_ethtool_setmsglevel(struct net_devi= ce *dev, u32 level) =20 } =20 -static int stmmac_check_if_running(struct net_device *dev) -{ - if (!netif_running(dev)) - return -EBUSY; - return 0; -} - static int stmmac_ethtool_get_regs_len(struct net_device *dev) { struct stmmac_priv *priv =3D netdev_priv(dev); @@ -1273,7 +1266,6 @@ static int stmmac_set_tunable(struct net_device *dev, static const struct ethtool_ops stmmac_ethtool_ops =3D { .supported_coalesce_params =3D ETHTOOL_COALESCE_USECS | ETHTOOL_COALESCE_MAX_FRAMES, - .begin =3D stmmac_check_if_running, .get_drvinfo =3D stmmac_ethtool_getdrvinfo, .get_msglevel =3D stmmac_ethtool_getmsglevel, .set_msglevel =3D stmmac_ethtool_setmsglevel, --- base-commit: b18bbfc14a38b5234e09c2adcf713e38063a7e6e change-id: 20240424-stmmac-no-ethtool-begin-f306f2f1f2f4 Best regards, --=20 Andrew Halaney