From owner-freebsd-net@FreeBSD.ORG  Wed Nov  7 23:46:32 2012
Return-Path: <owner-freebsd-net@FreeBSD.ORG>
Delivered-To: freebsd-net@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52])
 by hub.freebsd.org (Postfix) with ESMTP id 97D4F37F;
 Wed,  7 Nov 2012 23:46:32 +0000 (UTC)
 (envelope-from adrian.chadd@gmail.com)
Received: from mail-pa0-f54.google.com (mail-pa0-f54.google.com
 [209.85.220.54])
 by mx1.freebsd.org (Postfix) with ESMTP id 5A4F18FC08;
 Wed,  7 Nov 2012 23:46:32 +0000 (UTC)
Received: by mail-pa0-f54.google.com with SMTP id bi1so1642235pad.13
 for <multiple recipients>; Wed, 07 Nov 2012 15:46:32 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113;
 h=mime-version:sender:in-reply-to:references:date
 :x-google-sender-auth:message-id:subject:from:to:cc:content-type;
 bh=1dOelbSSJQcmAOG7UkuvSb1cLBMTZyKpbOJ4iWxkMlI=;
 b=ja50FWPvKjt98p3THTqjRR3X6+4CV9DaqAIe1akX3P6wstOIb1j/HO7mpeRFTA51YX
 NdwNGrmmhBBOWyfXlqwTht9oVwnn/jgeLezzzSzplwCc/DsKXcsCQGnzUmj/UqtX6tGf
 tWycjPoQkK18mR3MZb8AsZAWeTtmVpOagC8p5yNRShjJMi9ysKwTFygffGUVGESAkfIW
 yB3EG65eWJjFqIkFZ4rP9R01vwN76Ol2uGT5aq0p3Edg/YmwI9/u7r5A5SQR7E/7Pgr0
 2+MvOCy+c+goalaJlUu9pScYFxob6NeaZ1iv8dBQL79V4Dk8ntTJUlUh9FiDyGoaPvMx
 qarg==
MIME-Version: 1.0
Received: by 10.68.137.41 with SMTP id qf9mr18231413pbb.103.1352331992011;
 Wed, 07 Nov 2012 15:46:32 -0800 (PST)
Sender: adrian.chadd@gmail.com
Received: by 10.68.124.130 with HTTP; Wed, 7 Nov 2012 15:46:31 -0800 (PST)
In-Reply-To: <509AEDAC.10002@FreeBSD.org>
References: <509AEDAC.10002@FreeBSD.org>
Date: Wed, 7 Nov 2012 15:46:31 -0800
X-Google-Sender-Auth: jNar6e2WSwl_0DvDNZnoj8EG93c
Message-ID: <CAJ-Vmo=tw1peswao9hwk=1fdmvr6SFDfE5T0cmw0b1w_YWRJtQ@mail.gmail.com>
Subject: Re: [patch] reducing arp locking
From: Adrian Chadd <adrian@freebsd.org>
To: "Alexander V. Chernikov" <melifaro@freebsd.org>
Content-Type: text/plain; charset=ISO-8859-1
Cc: freebsd-net@freebsd.org, freebsd-hackers@freebsd.org
X-BeenThere: freebsd-net@freebsd.org
X-Mailman-Version: 2.1.14
Precedence: list
List-Id: Networking and TCP/IP with FreeBSD <freebsd-net.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/options/freebsd-net>,
 <mailto:freebsd-net-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-net>
List-Post: <mailto:freebsd-net@freebsd.org>
List-Help: <mailto:freebsd-net-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-net>,
 <mailto:freebsd-net-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Wed, 07 Nov 2012 23:46:32 -0000

On 7 November 2012 15:24, Alexander V. Chernikov <melifaro@freebsd.org> wrote:
> Hello list!
>
> Currently we need to acquire 2 read locks to perform simple 6-byte copying
> from arp record to packet ethernet header.
>
> It seems that acquiring lle lock for fast path (main traffic flow) is not
> necessary even with current code.
>
> My tests shows ~10% improvement with this patch applied.
>
> If nobody objects I plan to commit this change at the end of next week.

That's a great catch! How'd you discover it?



Adrian