From owner-svn-src-projects@FreeBSD.ORG Wed Mar 25 14:18:57 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 11AF5E3E; Wed, 25 Mar 2015 14:18:57 +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 D7004A9D; Wed, 25 Mar 2015 14:18:56 +0000 (UTC) Received: from svn.freebsd.org ([127.0.1.70]) by svn.freebsd.org (8.14.9/8.14.9) with ESMTP id t2PEIuiv001368; Wed, 25 Mar 2015 14:18:56 GMT (envelope-from glebius@FreeBSD.org) Received: (from glebius@localhost) by svn.freebsd.org (8.14.9/8.14.9/Submit) id t2PEIu3W001367; Wed, 25 Mar 2015 14:18:56 GMT (envelope-from glebius@FreeBSD.org) Message-Id: <201503251418.t2PEIu3W001367@svn.freebsd.org> X-Authentication-Warning: svn.freebsd.org: glebius set sender to glebius@FreeBSD.org using -f From: Gleb Smirnoff Date: Wed, 25 Mar 2015 14:18:56 +0000 (UTC) To: src-committers@freebsd.org, svn-src-projects@freebsd.org Subject: svn commit: r280616 - projects/ifnet/sys/net 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: Wed, 25 Mar 2015 14:18:57 -0000 Author: glebius Date: Wed Mar 25 14:18:55 2015 New Revision: 280616 URL: https://svnweb.freebsd.org/changeset/base/280616 Log: Slightly change macros in ifdriver_bless() that assign ifdrv fields to default or type-specific values. Improve comments. No functional change. Modified: projects/ifnet/sys/net/if.c Modified: projects/ifnet/sys/net/if.c ============================================================================== --- projects/ifnet/sys/net/if.c Wed Mar 25 14:10:20 2015 (r280615) +++ projects/ifnet/sys/net/if.c Wed Mar 25 14:18:55 2015 (r280616) @@ -435,18 +435,23 @@ static void ifdriver_bless(struct ifdriver *ifdrv, struct iftype *ift) { + /* + * If the driver doesn't define certain op, but its type has + * default implementation, then copy it. + */ if (ift != NULL) { -#define COPY(op) if (ifdrv->ifdrv_ops.op == NULL) \ - ifdrv->ifdrv_ops.op = ift->ift_ops.op - COPY(ifop_input); - COPY(ifop_transmit); - COPY(ifop_output); - COPY(ifop_ioctl); - COPY(ifop_get_counter); - COPY(ifop_qflush); - COPY(ifop_resolvemulti); - COPY(ifop_reassign); -#undef COPY +#define COPYOP(op) if (ifdrv->ifdrv_ops.ifop_ ## op == NULL) \ + ifdrv->ifdrv_ops.ifop_ ## op = \ + ift->ift_ops.ifop_ ## op + COPYOP(input); + COPYOP(transmit); + COPYOP(output); + COPYOP(ioctl); + COPYOP(get_counter); + COPYOP(qflush); + COPYOP(resolvemulti); + COPYOP(reassign); +#undef COPYOP #define COPY(f) if (ifdrv->ifdrv_ ## f == 0) \ ifdrv->ifdrv_ ## f = ift->ift_ ## f COPY(hdrlen); @@ -457,7 +462,7 @@ ifdriver_bless(struct ifdriver *ifdrv, s } /* - * If driver has ifdrv_maxqlen defined, then it opts-in + * If the driver has ifdrv_maxqlen defined, then opts-in * for * generic software queue, and thus for default * ifop_qflush. */ @@ -468,8 +473,15 @@ ifdriver_bless(struct ifdriver *ifdrv, s ifdrv->ifdrv_ops.ifop_qflush = if_snd_qflush; } - if (ifdrv->ifdrv_ops.ifop_get_counter == NULL) - ifdrv->ifdrv_ops.ifop_get_counter = if_get_counter_default; + /* + * If neither driver nor its type has a definitation of an op + * that is mandatory, then set it to default implementation. + */ +#define DEFAULTOP(op) if (ifdrv->ifdrv_ops.ifop_ ## op == NULL) \ + ifdrv->ifdrv_ops.ifop_ ## op = \ + if_ ## op ## _default + DEFAULTOP(get_counter); +#undef DEFAULTOP #if defined(INET) || defined(INET6) /* Use defaults for TSO, if nothing is set. */