From owner-svn-src-projects@FreeBSD.ORG Fri Jan 23 19:43:59 2015 Return-Path: Delivered-To: svn-src-projects@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id C03D0666; Fri, 23 Jan 2015 19:43:59 +0000 (UTC) Received: from svn.freebsd.org (svn.freebsd.org [IPv6:2001:1900:2254:2068::e6a:0]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id AB3C48A1; Fri, 23 Jan 2015 19:43:59 +0000 (UTC) Received: from svn.freebsd.org ([127.0.1.70]) by svn.freebsd.org (8.14.9/8.14.9) with ESMTP id t0NJhxbC060591; Fri, 23 Jan 2015 19:43:59 GMT (envelope-from glebius@FreeBSD.org) Received: (from glebius@localhost) by svn.freebsd.org (8.14.9/8.14.9/Submit) id t0NJhwZl060585; Fri, 23 Jan 2015 19:43:58 GMT (envelope-from glebius@FreeBSD.org) Message-Id: <201501231943.t0NJhwZl060585@svn.freebsd.org> X-Authentication-Warning: svn.freebsd.org: glebius set sender to glebius@FreeBSD.org using -f From: Gleb Smirnoff Date: Fri, 23 Jan 2015 19:43:58 +0000 (UTC) To: src-committers@freebsd.org, svn-src-projects@freebsd.org Subject: svn commit: r277605 - in projects/ifnet/sys: dev/msk dev/virtio/network net netgraph X-SVN-Group: projects MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-src-projects@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: "SVN commit messages for the src " projects" tree" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 23 Jan 2015 19:43:59 -0000 Author: glebius Date: Fri Jan 23 19:43:57 2015 New Revision: 277605 URL: https://svnweb.freebsd.org/changeset/base/277605 Log: While not too late, change order of arguments to if_drvioctl(), making it more "natural: - ifp is the first argument - ioctl cmd and argument go together Modified: projects/ifnet/sys/dev/msk/if_msk.c projects/ifnet/sys/dev/virtio/network/if_vtnet.c projects/ifnet/sys/net/if.c projects/ifnet/sys/net/if.h projects/ifnet/sys/netgraph/ng_iface.c Modified: projects/ifnet/sys/dev/msk/if_msk.c ============================================================================== --- projects/ifnet/sys/dev/msk/if_msk.c Fri Jan 23 19:33:46 2015 (r277604) +++ projects/ifnet/sys/dev/msk/if_msk.c Fri Jan 23 19:43:57 2015 (r277605) @@ -1097,10 +1097,10 @@ msk_ioctl(if_t ifp, u_long command, void struct ifreq tmp; MSK_IF_UNLOCK(sc_if); - if_drvioctl(SIOCGIFCAP, ifp, &tmp, td); + if_drvioctl(ifp, SIOCGIFCAP, &tmp, td); tmp.ifr_reqcap = tmp.ifr_curcap & ~(MSK_CSUM_FEATURES | CSUM_TSO); - if_drvioctl(SIOCSIFCAP, ifp, &tmp, td); + if_drvioctl(ifp, SIOCSIFCAP, &tmp, td); MSK_IF_LOCK(sc_if); } } Modified: projects/ifnet/sys/dev/virtio/network/if_vtnet.c ============================================================================== --- projects/ifnet/sys/dev/virtio/network/if_vtnet.c Fri Jan 23 19:33:46 2015 (r277604) +++ projects/ifnet/sys/dev/virtio/network/if_vtnet.c Fri Jan 23 19:43:57 2015 (r277605) @@ -3071,13 +3071,13 @@ vtnet_attach_disable_promisc(struct vtne VTNET_CORE_LOCK(sc); if ((sc->vtnet_flags & VTNET_FLAG_CTRL_RX) == 0) { - (void )if_drvioctl(SIOCGIFFLAGS, ifp, &ifr, curthread); + (void )if_drvioctl(ifp, SIOCGIFFLAGS, &ifr, curthread); ifr.ifr_flags |= IFF_PROMISC; - (void )if_drvioctl(SIOCSIFFLAGS, ifp, &ifr, curthread); + (void )if_drvioctl(ifp, SIOCSIFFLAGS, &ifr, curthread); } else if (vtnet_set_promisc(sc, 0) != 0) { - (void )if_drvioctl(SIOCGIFFLAGS, ifp, &ifr, curthread); + (void )if_drvioctl(ifp, SIOCGIFFLAGS, &ifr, curthread); ifr.ifr_flags |= IFF_PROMISC; - (void )if_drvioctl(SIOCSIFFLAGS, ifp, &ifr, curthread); + (void )if_drvioctl(ifp, SIOCSIFFLAGS, &ifr, curthread); device_printf(sc->vtnet_dev, "cannot disable default promiscuous mode\n"); } Modified: projects/ifnet/sys/net/if.c ============================================================================== --- projects/ifnet/sys/net/if.c Fri Jan 23 19:33:46 2015 (r277604) +++ projects/ifnet/sys/net/if.c Fri Jan 23 19:43:57 2015 (r277605) @@ -2283,7 +2283,7 @@ ifunit(const char *name) * Hardware specific interface ioctls. */ int -if_drvioctl(u_long cmd, struct ifnet *ifp, void *data, struct thread *td) +if_drvioctl(struct ifnet *ifp, u_long cmd, void *data, struct thread *td) { struct ifreq *ifr; size_t namelen, onamelen; @@ -2793,7 +2793,7 @@ ifioctl(struct socket *so, u_long cmd, c return (ENXIO); } - error = if_drvioctl(cmd, ifp, data, td); + error = if_drvioctl(ifp, cmd, data, td); if (error != ENOIOCTL) { if_rele(ifp); CURVNET_RESTORE(); Modified: projects/ifnet/sys/net/if.h ============================================================================== --- projects/ifnet/sys/net/if.h Fri Jan 23 19:33:46 2015 (r277604) +++ projects/ifnet/sys/net/if.h Fri Jan 23 19:43:57 2015 (r277605) @@ -716,7 +716,7 @@ void if_setbaudrate(if_t, uint64_t); void if_link_state_change(if_t, int); void * if_getsoftc(if_t, ift_feature); int if_printf(if_t, const char *, ...) __printflike(2, 3); -int if_drvioctl(u_long, struct ifnet *, void *, struct thread *); +int if_drvioctl(if_t, u_long, void *, struct thread *); uint64_t if_get_counter_default(if_t, ift_counter); /* Modified: projects/ifnet/sys/netgraph/ng_iface.c ============================================================================== --- projects/ifnet/sys/netgraph/ng_iface.c Fri Jan 23 19:33:46 2015 (r277604) +++ projects/ifnet/sys/netgraph/ng_iface.c Fri Jan 23 19:43:57 2015 (r277605) @@ -569,7 +569,7 @@ ng_iface_rcvmsg(node_p node, item_p item error = ENOMEM; break; } - if_drvioctl(SIOCGIFINDEX, ifp, &ifr, curthread); + if_drvioctl(ifp, SIOCGIFINDEX, &ifr, curthread); *((uint32_t *)resp->data) = ifr.ifr_index; break; }