From owner-svn-src-head@FreeBSD.ORG Thu Jan 22 05:21:06 2015 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 0E18D70D for ; Thu, 22 Jan 2015 05:21:06 +0000 (UTC) Received: from mail-oi0-f52.google.com (mail-oi0-f52.google.com [209.85.218.52]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id CEAF78F2 for ; Thu, 22 Jan 2015 05:21:05 +0000 (UTC) Received: by mail-oi0-f52.google.com with SMTP id h136so19342792oig.11 for ; Wed, 21 Jan 2015 21:21:04 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:sender:content-type:mime-version:subject:from :in-reply-to:date:cc:content-transfer-encoding:message-id:references :to; bh=1omhlkBVCJNTsDzPh+6hXMmwcOYHnQpTXCs8tTJKoEQ=; b=hm7mPKB2Gta7j+hKC55Z1Nd8eBT9iFQPAb9ixsOWYwlRQ8WJhEMBJMK7f3UBHZZ60L SuSsAsGW553xH43LJMLoBsOecLmJ+6ZrfiIt989rieREmZ/xuOFaElbkXaVfW2MYWxcq iHJ/EcYuUEG3cqsuPx6qfYDmAsf1os+pvvdxewBtqcZpJWo5pEw7vSek3WncKNBgGC3w T9JQurGnAEq7Kd9/eYM7WyXZ4ngKTATUpkwW7PSopxfPDj6lxXTZEbekFahDfBkvG1c6 ofpCasTVDe64xb4ccAFnDwA5bN6JRpNUCR0kQuCfUPDSLYqd4fcXxjG2oasIPzWJky8O wbuQ== X-Gm-Message-State: ALoCoQkzF4StFlItGF8ejlbh9iwI3Z0C58wSOpCPkMSnRMxcN0k2sGd6k2FFOP7k/3exHPDZbBKZ X-Received: by 10.182.148.98 with SMTP id tr2mr27417291obb.28.1421904064608; Wed, 21 Jan 2015 21:21:04 -0800 (PST) Received: from [192.168.43.169] ([172.56.8.11]) by mx.google.com with ESMTPSA id r9sm9021703obi.29.2015.01.21.21.21.03 (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Wed, 21 Jan 2015 21:21:04 -0800 (PST) Sender: Warner Losh Content-Type: text/plain; charset=utf-8 Mime-Version: 1.0 (Mac OS X Mail 8.1 \(1993\)) Subject: Re: svn commit: r277213 - in head: share/man/man9 sys/kern sys/ofed/include/linux sys/sys From: Warner Losh In-Reply-To: <54BE0528.2050503@selasky.org> Date: Wed, 21 Jan 2015 22:21:02 -0700 Content-Transfer-Encoding: quoted-printable Message-Id: <19408675-1E2B-4F38-8C9A-66975B6C01DF@bsdimp.com> References: <201501151532.t0FFWV2Y037455@svn.freebsd.org> <54BDD9E1.6090505@selasky.org> <54BDE58F.3070909@selasky.org> <54BE0528.2050503@selasky.org> To: Hans Petter Selasky X-Mailer: Apple Mail (2.1993) Cc: "svn-src-head@freebsd.org" , Adrian Chadd , "src-committers@freebsd.org" , "svn-src-all@freebsd.org" X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 22 Jan 2015 05:21:06 -0000 > On Jan 20, 2015, at 12:35 AM, Hans Petter Selasky = wrote: >=20 > On 01/20/15 06:22, Adrian Chadd wrote: >> Sweet, thanks. I'l test it, but anything that changes the locking to >> TCP is going to need a more thorough review. The "there be dragons" >> disclaimer is appropriate.:) >=20 > No changes in locking - simply some minor code reordering. This isn=E2=80=99t entirely true. You changed the INFO_WLOCK protocol, = and also drop the WLOCK to acquire the INFO_WLOCK in places, and it = isn=E2=80=99t clear to me at all why this is safe to do. Please document = the analysis you did to show that was safe. Warner