From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.0 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id C7FA6C43381 for ; Thu, 21 Mar 2019 13:41:37 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id A03E82075E for ; Thu, 21 Mar 2019 13:41:37 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728703AbfCUNlg (ORCPT ); Thu, 21 Mar 2019 09:41:36 -0400 Received: from mx2.suse.de ([195.135.220.15]:36726 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1728652AbfCUNl1 (ORCPT ); Thu, 21 Mar 2019 09:41:27 -0400 X-Virus-Scanned: by amavisd-new at test-mx.suse.de Received: from relay2.suse.de (unknown [195.135.220.254]) by mx1.suse.de (Postfix) with ESMTP id E363FAF8C; Thu, 21 Mar 2019 13:41:24 +0000 (UTC) Received: by unicorn.suse.cz (Postfix, from userid 1000) id 956D0E00BF; Thu, 21 Mar 2019 14:41:24 +0100 (CET) Message-Id: In-Reply-To: References: From: Michal Kubecek Subject: [PATCH net-next v4 22/22] ethtool: set message level with SET_SETTINGS request To: David Miller , netdev@vger.kernel.org Cc: Jakub Kicinski , Jiri Pirko , Andrew Lunn , Florian Fainelli , John Linville , linux-kernel@vger.kernel.org Date: Thu, 21 Mar 2019 14:41:24 +0100 (CET) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Allow setting device message level using ETHA_SETTINGS_DEBUG nested attribute. Unlike in ioctl interface "message level" is called "message mask" (as it is in fact used as a bit mask) and put inside a nested attribute to allow future extensions. Signed-off-by: Michal Kubecek --- Documentation/networking/ethtool-netlink.txt | 8 +++- net/ethtool/settings.c | 45 +++++++++++++++++++- 2 files changed, 50 insertions(+), 3 deletions(-) diff --git a/Documentation/networking/ethtool-netlink.txt b/Documentation/networking/ethtool-netlink.txt index 603acfbefe29..20e43d42afdd 100644 --- a/Documentation/networking/ethtool-netlink.txt +++ b/Documentation/networking/ethtool-netlink.txt @@ -345,6 +345,8 @@ to be passed with SET_SETTINGS request: ETHA_SETTINGS_WOL (nested) wake on LAN settings ETHA_WOL_MODES (bitfield32) wake on LAN modes ETHA_WOL_SOPASS (binary) SecureOn(tm) password + ETHA_SETTINGS_DEBUG (nested) debugging + ETHA_DEBUG_MSG_MASK (bitfield32) message mask ETHA_LINKMODES_OURS bit set allows setting advertised link modes. If autonegotiation is on (either set now or kept from before), advertised modes @@ -360,7 +362,9 @@ selector are set to 0 or 1 according to value. To allow the semantics of the ioctl interface where the whole bitmap is set rather than only modified, selectors may have also bits not supported by device set and an error is only issued if any of them is also set in the value (i.e. if userspace tries to -enable mode not supported by device). +enable mode not supported by device). ETHA_SETTINGS_MSGLEVEL bitfield also +allows bits not recognized by kernel in selector as long as the request does +not attempt to enable them. Request translation @@ -379,7 +383,7 @@ ETHTOOL_GREGS n/a ETHTOOL_GWOL ETHNL_CMD_GET_SETTINGS ETHTOOL_SWOL ETHNL_CMD_SET_SETTINGS ETHTOOL_GMSGLVL ETHNL_CMD_GET_SETTINGS -ETHTOOL_SMSGLVL n/a +ETHTOOL_SMSGLVL ETHNL_CMD_SET_SETTINGS ETHTOOL_NWAY_RST n/a ETHTOOL_GLINK ETHNL_CMD_GET_SETTINGS ETHTOOL_GEEPROM n/a diff --git a/net/ethtool/settings.c b/net/ethtool/settings.c index 53409dd8af34..bc7ca03aa205 100644 --- a/net/ethtool/settings.c +++ b/net/ethtool/settings.c @@ -538,6 +538,12 @@ static const struct nla_policy set_wol_policy[ETHA_LINKINFO_MAX + 1] = { .len = SOPASS_MAX }, }; +static const struct nla_policy set_debug_policy[ETHA_DEBUG_MAX + 1] = { + [ETHA_DEBUG_UNSPEC] = { .type = NLA_REJECT }, + [ETHA_DEBUG_MSG_MASK] = { .type = NLA_BITFIELD32, + .validation_data = &all_bits }, +}; + static const struct nla_policy set_settings_policy[ETHA_SETTINGS_MAX + 1] = { [ETHA_SETTINGS_UNSPEC] = { .type = NLA_REJECT }, [ETHA_SETTINGS_DEV] = { .type = NLA_NESTED }, @@ -547,7 +553,7 @@ static const struct nla_policy set_settings_policy[ETHA_SETTINGS_MAX + 1] = { [ETHA_SETTINGS_LINK_MODES] = { .type = NLA_NESTED }, [ETHA_SETTINGS_LINK_STATE] = { .type = NLA_REJECT }, [ETHA_SETTINGS_WOL] = { .type = NLA_NESTED }, - [ETHA_SETTINGS_DEBUG] = { .type = NLA_REJECT }, + [ETHA_SETTINGS_DEBUG] = { .type = NLA_NESTED }, }; static int ethnl_set_link_ksettings(struct genl_info *info, @@ -732,6 +738,36 @@ static int update_wol(struct genl_info *info, struct nlattr *nest, return ret; } +static int update_debug(struct genl_info *info, struct nlattr *nest, + struct net_device *dev) +{ + struct nlattr *tb[ETHA_DEBUG_MAX + 1]; + u32 msglevel; + int ret; + + if (!nest) + return 0; + ret = nla_parse_nested_strict(tb, ETHA_DEBUG_MAX, nest, + set_debug_policy, info->extack); + if (ret < 0) + return ret; + + if (!dev->ethtool_ops->get_msglevel || + !dev->ethtool_ops->set_msglevel) { + ETHNL_SET_ERRMSG(info, + "device does not provide msglvl access"); + return -EOPNOTSUPP; + } + ret = 0; + msglevel = dev->ethtool_ops->get_msglevel(dev); + if (ethnl_update_bitfield32(&msglevel, tb[ETHA_DEBUG_MSG_MASK])) { + dev->ethtool_ops->set_msglevel(dev, msglevel); + ret = 1; + } + + return ret; +} + int ethnl_set_settings(struct sk_buff *skb, struct genl_info *info) { struct nlattr *tb[ETHA_SETTINGS_MAX + 1]; @@ -766,6 +802,13 @@ int ethnl_set_settings(struct sk_buff *skb, struct genl_info *info) if (ret) req_mask |= ETH_SETTINGS_IM_WOL; } + if (tb[ETHA_SETTINGS_DEBUG]) { + ret = update_debug(info, tb[ETHA_SETTINGS_DEBUG], dev); + if (ret < 0) + goto out_ops; + if (ret) + req_mask |= ETH_SETTINGS_IM_DEBUG; + } ret = 0; out_ops: -- 2.21.0