From nobody Fri Mar 3 16:02:24 2023 X-Original-To: dev-commits-src-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4PSt5r4cpgz3w4Dk; Fri, 3 Mar 2023 16:02:24 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4PSt5r4C1tz3NYx; Fri, 3 Mar 2023 16:02:24 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1677859344; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=pVkMGgYz9Ge/toov4E6ibmcVPAdLO9WIdSU2MFVsR8A=; b=vuAqH+xM9mBNp89fJFNvrogJCWT1jIunLqie2UZr9PyZ2mKKxaBUgEMJSyexWrGI3by5Fg pVtIrcqVM0xwieNVRKcGiJ+CBnGAI3lOPxf1DIsAkx57Q5iA0DQj5jtywKHGMZI8gfdSP0 sOlK1Zf8pBy7fHM57EAzhJxaiINHlT4IIdL9M5MqNL4wU6PA3wLZ9vImlZ8jLTOzES+Q2y aMsNrJ9TJ+bTd68W9fXi2wmoAzOqIO6lEkDWYxFmYEmW2ireBzEKLw+Qe/Eb2+nglHsfbZ 32E8/S5H5bGyJ/v/2STIJn0dK40UTerHO/Nq8+2BUctasOCCq8VM2hSuTtXLzQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1677859344; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=pVkMGgYz9Ge/toov4E6ibmcVPAdLO9WIdSU2MFVsR8A=; b=RsYcdtwNZI0gXbfEbH2gzfSHfHAsU1B2TT6JVKb2gpIpTtrvwLi9OjoSTitVDXJX5Ln74m KTrRbxvkHi4Eoo7Hz9+d3byQ5VgzYj+CTkj+WvZ6MQHZYt+nyASSP7tg9lZp2Zd1iY9xCM L1vCaLR343OhfUVNMR8QcLR4q9FIAtSSFl0vfH1MwFNqJf5m7voGAgUUeOgHvfqOcppOLU 3X8+R6b5eePl9jbosWo4fjrO9OudeOxwAOIo80HblxlW/RribhPXw6IDJEMBZ4Yr9MvfzX 9W5nEmnu1ynmjzRQY78d4Jsr9ftljI7eG9Ax0Gl64HyCRqzakDADvd7yrxgdzg== ARC-Authentication-Results: i=1; mx1.freebsd.org; none ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1677859344; a=rsa-sha256; cv=none; b=cFh9ys91QuPvNnitygU+slC74stHngRyPzTJpWUcGTJNUrFS0Xw2/3oohLPrjiNAHYyUkz QkKikqp0GKtIgzcieE3aro/TvnVsIuV75MM8KFLINbTlk9kSgN6NZXC53/seK+1VAu4SjV 0dB2UQvnBPEeuTsfXpa9l74LVi+eoQds+w8oOsMNJ9uGdPUm05aN8Y+j9F7da0v0sQG9By vk9CH4wqNyuEToRsG5S4aFWhvvKlmp/f5j9RdtQZVkPoOePLvwx7IpOtCtASDzBD9PMHcQ HMdYamYGC9lcOBki0TGHXNIJNCzr208svIcQUuILtqXPZCuXed9/juWIEoeW3Q== Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 4PSt5r3HDwz14Ck; Fri, 3 Mar 2023 16:02:24 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org ([127.0.1.44]) by gitrepo.freebsd.org (8.16.1/8.16.1) with ESMTP id 323G2Ofh083946; Fri, 3 Mar 2023 16:02:24 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.16.1/8.16.1/Submit) id 323G2Oqj083945; Fri, 3 Mar 2023 16:02:24 GMT (envelope-from git) Date: Fri, 3 Mar 2023 16:02:24 GMT Message-Id: <202303031602.323G2Oqj083945@gitrepo.freebsd.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org From: "Alexander V. Chernikov" Subject: git: ce9f95bd835e - main - netlink: fix vlan interface creation List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-all@freebsd.org X-BeenThere: dev-commits-src-all@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: melifaro X-Git-Repository: src X-Git-Refname: refs/heads/main X-Git-Reftype: branch X-Git-Commit: ce9f95bd835ec7d3ac6def5a910282509b1413f0 Auto-Submitted: auto-generated X-ThisMailContainsUnwantedMimeParts: N The branch main has been updated by melifaro: URL: https://cgit.FreeBSD.org/src/commit/?id=ce9f95bd835ec7d3ac6def5a910282509b1413f0 commit ce9f95bd835ec7d3ac6def5a910282509b1413f0 Author: Alexander V. Chernikov AuthorDate: 2023-03-03 16:01:34 +0000 Commit: Alexander V. Chernikov CommitDate: 2023-03-03 16:02:05 +0000 netlink: fix vlan interface creation MFC after: 2 weeks --- sys/netlink/route/iface_drivers.c | 34 +++++++++++++++++++--------------- tests/sys/netlink/test_rtnl_iface.py | 1 - 2 files changed, 19 insertions(+), 16 deletions(-) diff --git a/sys/netlink/route/iface_drivers.c b/sys/netlink/route/iface_drivers.c index 7cfaa5a6614b..be28a0f3b676 100644 --- a/sys/netlink/route/iface_drivers.c +++ b/sys/netlink/route/iface_drivers.c @@ -144,20 +144,14 @@ store_cookie(struct nl_pstate *npt, struct ifnet *ifp) nlmsg_report_cookie(npt, nla_cookie); } -/* - * Generic creation interface handler. - * Responsible for creating interfaces w/o parameters and setting - * misc attributes such as state, mtu or description. - */ static int -create_generic(struct nl_parsed_link *lattrs, const struct nlattr_bmask *bm, - struct nlpcb *nlp, struct nl_pstate *npt) +create_generic_ifd(struct nl_parsed_link *lattrs, const struct nlattr_bmask *bm, + struct ifc_data *ifd, struct nlpcb *nlp, struct nl_pstate *npt) { int error = 0; - struct ifc_data ifd = {}; struct ifnet *ifp = NULL; - error = ifc_create_ifp(lattrs->ifla_ifname, &ifd, &ifp); + error = ifc_create_ifp(lattrs->ifla_ifname, ifd, &ifp); NLP_LOG(LOG_DEBUG2, nlp, "clone for %s returned %d", lattrs->ifla_ifname, error); @@ -177,6 +171,19 @@ create_generic(struct nl_parsed_link *lattrs, const struct nlattr_bmask *bm, return (error); } +/* + * Generic creation interface handler. + * Responsible for creating interfaces w/o parameters and setting + * misc attributes such as state, mtu or description. + */ +static int +create_generic(struct nl_parsed_link *lattrs, const struct nlattr_bmask *bm, + struct nlpcb *nlp, struct nl_pstate *npt) +{ + struct ifc_data ifd = {}; + + return (create_generic_ifd(lattrs, bm, &ifd, nlp, npt)); +} struct nl_cloner generic_cloner = { .name = "_default_", @@ -258,17 +265,14 @@ create_vlan(struct nl_parsed_link *lattrs, const struct nlattr_bmask *bm, return (ENOENT); } - /* Waiting till if_clone changes lands */ -/* struct vlanreq params = { .vlr_tag = attrs.vlan_id, .vlr_proto = attrs.vlan_proto, }; -*/ - int ifname_len = strlen(lattrs->ifla_ifname) + 1; - error = if_clone_create(lattrs->ifla_ifname, ifname_len, (char *)NULL); + strlcpy(params.vlr_parent, if_name(ifp), sizeof(params.vlr_parent)); + struct ifc_data ifd = { .flags = IFC_F_SYSSPACE, .params = ¶ms }; - NLP_LOG(LOG_DEBUG2, nlp, "clone for %s returned %d", lattrs->ifla_ifname, error); + error = create_generic_ifd(lattrs, bm, &ifd, nlp, npt); if_rele(ifp); return (error); diff --git a/tests/sys/netlink/test_rtnl_iface.py b/tests/sys/netlink/test_rtnl_iface.py index 35015ea2ec6e..da3db93b05c6 100644 --- a/tests/sys/netlink/test_rtnl_iface.py +++ b/tests/sys/netlink/test_rtnl_iface.py @@ -313,7 +313,6 @@ class TestRtNlIface(NetlinkTestTemplate, SingleVnetTestTemplate): # * {{nla_len=8, nla_type=IFLA_INFO_KIND}, "vlan"...}, # * {{nla_len=12, nla_type=IFLA_INFO_DATA}, "\x06\x00\x01\x00\x16\x00\x00\x00"} # */ - @pytest.mark.skip(reason="vlan support needs more work") @pytest.mark.require_user("root") def test_create_vlan_plain(self): """Creates 802.1Q VLAN interface in vlanXX and ifX fashion"""