From owner-svn-src-stable@FreeBSD.ORG Tue May 15 20:08:49 2012 Return-Path: Delivered-To: svn-src-stable@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 2C462106564A; Tue, 15 May 2012 20:08:49 +0000 (UTC) (envelope-from avg@FreeBSD.org) Received: from citadel.icyb.net.ua (citadel.icyb.net.ua [212.40.38.140]) by mx1.freebsd.org (Postfix) with ESMTP id D2DAA8FC08; Tue, 15 May 2012 20:08:44 +0000 (UTC) Received: from porto.starpoint.kiev.ua (porto-e.starpoint.kiev.ua [212.40.38.100]) by citadel.icyb.net.ua (8.8.8p3/ICyb-2.3exp) with ESMTP id XAA13866; Tue, 15 May 2012 23:08:40 +0300 (EEST) (envelope-from avg@FreeBSD.org) Received: from localhost ([127.0.0.1]) by porto.starpoint.kiev.ua with esmtp (Exim 4.34 (FreeBSD)) id 1SUO2x-000Ki2-UP; Tue, 15 May 2012 23:08:39 +0300 Message-ID: <4FB2B7C5.2080604@FreeBSD.org> Date: Tue, 15 May 2012 23:08:37 +0300 From: Andriy Gapon User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:12.0) Gecko/20120503 Thunderbird/12.0.1 MIME-Version: 1.0 To: Konstantin Belousov References: <201205151721.q4FHLlgq097452@svn.freebsd.org> <20120515182747.GD2358@deviant.kiev.zoral.com.ua> In-Reply-To: <20120515182747.GD2358@deviant.kiev.zoral.com.ua> X-Enigmail-Version: 1.5pre Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: svn-src-stable@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org, svn-src-stable-9@FreeBSD.org Subject: Re: svn commit: r235480 - in stable/9/sys: dev/usb geom geom/mountver i386/conf kern sys X-BeenThere: svn-src-stable@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for all the -stable branches of the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 15 May 2012 20:08:49 -0000 on 15/05/2012 21:27 Konstantin Belousov said the following: > On Tue, May 15, 2012 at 05:21:47PM +0000, Andriy Gapon wrote: >> Author: avg Date: Tue May 15 17:21:46 2012 New Revision: 235480 URL: >> http://svn.freebsd.org/changeset/base/235480 >> >> Log: MFC r230643: stop_scheduler -> td_stopsched Modified: >> stable/9/sys/sys/proc.h >> ============================================================================== >> >> --- stable/9/sys/sys/proc.h Tue May 15 17:11:00 2012 (r235479) >> +++ stable/9/sys/sys/proc.h Tue May 15 17:21:46 2012 (r235480) @@ -235,6 >> +235,7 @@ struct thread { short td_locks; /* (k) Count of non-spin >> locks. */ short td_rw_rlocks; /* (k) Count of rwlock read locks. */ >> short td_lk_slocks; /* (k) Count of lockmgr shared locks. */ + short >> td_stopsched; /* (k) Scheduler stopped. */ struct turnstile *td_blocked; >> /* (t) Lock thread is blocked on. */ const char *td_lockname; /* (t) Name >> of lock blocked on. */ LIST_HEAD(, turnstile) td_contested; /* (q) >> Contested locks. */ >> > Did you verified that there is a gap between td_lk_slocks and td_blocked an > all tier-1 architectures large enough to adopt td_stopsched without > changing the layout of struct thread after the new member ? I haven't verified that. but I presumed that the gap was either 2 bytes on 32-bit architectures or 6 bytes on 64-bit ones. What is the proper and doable way of verifying this? > If not, this breaks KBI. I can just revert the commit to not risk an obscure breakage. What would you suggest? Thank you for spotting this. -- Andriy Gapon