From owner-freebsd-hackers@FreeBSD.ORG Wed Aug 22 13:26:41 2012 Return-Path: Delivered-To: freebsd-hackers@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 4E95A106564A for ; Wed, 22 Aug 2012 13:26:41 +0000 (UTC) (envelope-from imp@bsdimp.com) Received: from mail-pb0-f54.google.com (mail-pb0-f54.google.com [209.85.160.54]) by mx1.freebsd.org (Postfix) with ESMTP id 182078FC1B for ; Wed, 22 Aug 2012 13:26:40 +0000 (UTC) Received: by pbbrp2 with SMTP id rp2so1525179pbb.13 for ; Wed, 22 Aug 2012 06:26:40 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=sender:subject:mime-version:content-type:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to:x-mailer :x-gm-message-state; bh=cNzWHgU9ZibzOlCTJUGq2Bl4M+lRChyXB/vpy0ofcTg=; b=MQuomDCVJGcK/cIa01cJeLs91taE4wxsrd7GIULu9wFyb6WCh5toGvDbKtq9Vd6MaD +FddBcg5EdFQ1bQwRUY27Tc9S7W8suqe+ORstA5ktrv7G67vnI91+G/AiB4Icx6UNmid C/uDlWC11xgVN4PoApn/6BwS7PI88va8lgvN1M3EJyI65tH9rqgPEElQ9kKN9/Y0TCfO oArE83731vgCJ4WooPwl8Fe+yygd0oCG44b0dp5O3saDDGgIvaH3UUytwXbzVqu8ori+ IC/3yZ2T+s4clrN53hUDlgYVcg7uUplq8vOZK0fHceukWS+DL9vLqxn1/FMMa3+AnED2 V6eA== Received: by 10.66.86.166 with SMTP id q6mr45967443paz.83.1345642000454; Wed, 22 Aug 2012 06:26:40 -0700 (PDT) Received: from [10.0.0.63] (50-78-194-198-static.hfc.comcastbusiness.net. [50.78.194.198]) by mx.google.com with ESMTPS id hx9sm3710761pbc.68.2012.08.22.06.26.36 (version=TLSv1/SSLv3 cipher=OTHER); Wed, 22 Aug 2012 06:26:37 -0700 (PDT) Sender: Warner Losh Mime-Version: 1.0 (Apple Message framework v1084) Content-Type: text/plain; charset=us-ascii From: Warner Losh In-Reply-To: <201208220802.14588.jhb@freebsd.org> Date: Wed, 22 Aug 2012 07:26:34 -0600 Content-Transfer-Encoding: quoted-printable Message-Id: <2DB48390-4FB8-49ED-A176-CF593ED57502@bsdimp.com> References: <50324DB4.6080905@cabletv.dp.ua> <420BA06C-C776-47DB-B3BB-F1414C115F99@bsdimp.com> <201208220802.14588.jhb@freebsd.org> To: John Baldwin X-Mailer: Apple Mail (2.1084) X-Gm-Message-State: ALoCoQnMvKR2EzvDaGunu/4ka6nomGFFvbQVfETeHXPfpGPXa7iFgY7SdMeR0hhQ/0hw5E73QvtH Cc: freebsd-hackers@freebsd.org, Adrian Chadd , freebsd-net@freebsd.org, Mitya , Wojciech Puchar Subject: Re: Replace bcopy() to update ether_addr X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 22 Aug 2012 13:26:41 -0000 On Aug 22, 2012, at 6:02 AM, John Baldwin wrote: > On Tuesday, August 21, 2012 12:34:42 pm Adrian Chadd wrote: >> Hi, >>=20 >> What about just creating an ETHER_ADDR_COPY(dst, src) and putting = that >> in a relevant include file, then hide the ugliness there? >>=20 >> The same benefits will likely appear when copying wifi MAC addresses >> to/from headers. >>=20 >> Thanks, I'm glad someone noticed this. >=20 > I doubt we even _need_ the ugliness. We should just use *dst =3D *src > unless there is a compelling reason not to. Agreed. We should do that, and then check the generated code to be sure = there isn't a compelling reason there :) Warner