From owner-freebsd-arch@FreeBSD.ORG  Thu Feb 26 14:23:09 2015
Return-Path: <owner-freebsd-arch@FreeBSD.ORG>
Delivered-To: freebsd-arch@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 E644740E
 for <freebsd-arch@freebsd.org>; Thu, 26 Feb 2015 14:23:09 +0000 (UTC)
Received: from mailout.easymail.ca (mailout.easymail.ca [64.68.201.169])
 (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))
 (Client did not present a certificate)
 by mx1.freebsd.org (Postfix) with ESMTPS id A1075CDC
 for <freebsd-arch@freebsd.org>; Thu, 26 Feb 2015 14:23:09 +0000 (UTC)
Received: from localhost (localhost [127.0.0.1])
 by mailout.easymail.ca (Postfix) with ESMTP id DC5DDE722
 for <freebsd-arch@freebsd.org>; Thu, 26 Feb 2015 09:23:07 -0500 (EST)
X-Virus-Scanned: Debian amavisd-new at mailout.easymail.ca
X-Spam-Flag: NO
X-Spam-Score: -3.856
X-Spam-Level: 
X-Spam-Status: No, score=-3.856 required=5 tests=[ALL_TRUSTED=-1.8,
 AWL=-0.149, BAYES_00=-2.599, DNS_FROM_AHBL_RHSBL=0.692]
Received: from mailout.easymail.ca ([127.0.0.1])
 by localhost (easymail-mailout.easydns.vpn [127.0.0.1]) (amavisd-new,
 port 10024)
 with ESMTP id NDv7SsThGuYg for <freebsd-arch@freebsd.org>;
 Thu, 26 Feb 2015 09:23:07 -0500 (EST)
Received: from bsddt1241.lv01.astrodoggroup.com (unknown [40.141.24.126])
 (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits))
 (No client certificate requested)
 by mailout.easymail.ca (Postfix) with ESMTPSA id 8B1C1E71E
 for <freebsd-arch@freebsd.org>; Thu, 26 Feb 2015 09:23:07 -0500 (EST)
Message-ID: <54EF2C54.7030207@astrodoggroup.com>
Date: Thu, 26 Feb 2015 06:23:16 -0800
From: Harrison Grundy <harrison.grundy@astrodoggroup.com>
User-Agent: Mozilla/5.0 (X11; FreeBSD amd64;
 rv:31.0) Gecko/20100101 Thunderbird/31.3.0
MIME-Version: 1.0
To: freebsd-arch@freebsd.org
Subject: Minor ULE changes and optimizations
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8bit
X-BeenThere: freebsd-arch@freebsd.org
X-Mailman-Version: 2.1.18-1
Precedence: list
List-Id: Discussion related to FreeBSD architecture <freebsd-arch.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/options/freebsd-arch>,
 <mailto:freebsd-arch-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-arch/>
List-Post: <mailto:freebsd-arch@freebsd.org>
List-Help: <mailto:freebsd-arch-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-arch>,
 <mailto:freebsd-arch-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 26 Feb 2015 14:23:10 -0000

In addition to the random() replacement patch, I have two other minor
ULE patches:

https://reviews.freebsd.org/D1970
This fixes the comment in sched_balance_pair to reflect what ULE is
actually doing, and moves the load comparison outside of tdq_lock to
avoid taking the lock unless there is actually a load imbalance
between the two tdqs.

https://reviews.freebsd.org/D1969
This allows a non-migratable thread to pin itself to a CPU if it is
already running on that CPU.

I've been running these patches for the past week or so without issue.
Any additional testing or comments would be greatly appreciated.

--- Harrison