From owner-freebsd-bugs@FreeBSD.ORG  Tue Feb  5 12:10:01 2013
Return-Path: <owner-freebsd-bugs@FreeBSD.ORG>
Delivered-To: freebsd-bugs@smarthost.ysv.freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org
 [IPv6:2001:1900:2254:206a::19:1])
 by hub.freebsd.org (Postfix) with ESMTP id 3B45A532
 for <freebsd-bugs@smarthost.ysv.freebsd.org>;
 Tue,  5 Feb 2013 12:10:01 +0000 (UTC)
 (envelope-from gnats@FreeBSD.org)
Received: from freefall.freebsd.org (freefall.freebsd.org
 [IPv6:2001:1900:2254:206c::16:87])
 by mx1.freebsd.org (Postfix) with ESMTP id 2DBC12C5
 for <freebsd-bugs@smarthost.ysv.freebsd.org>;
 Tue,  5 Feb 2013 12:10:01 +0000 (UTC)
Received: from freefall.freebsd.org (localhost [127.0.0.1])
 by freefall.freebsd.org (8.14.6/8.14.6) with ESMTP id r15CA1M5031845
 for <freebsd-bugs@freefall.freebsd.org>; Tue, 5 Feb 2013 12:10:01 GMT
 (envelope-from gnats@freefall.freebsd.org)
Received: (from gnats@localhost)
 by freefall.freebsd.org (8.14.6/8.14.6/Submit) id r15CA1Z6031841;
 Tue, 5 Feb 2013 12:10:01 GMT (envelope-from gnats)
Date: Tue, 5 Feb 2013 12:10:01 GMT
Message-Id: <201302051210.r15CA1Z6031841@freefall.freebsd.org>
To: freebsd-bugs@FreeBSD.org
Cc: 
From: Gleb Smirnoff <glebius@FreeBSD.org>
Subject: Re: kern/175759: Correct data types for fields of struct qm_trace{}
 from <sys/queue.h>
X-BeenThere: freebsd-bugs@freebsd.org
X-Mailman-Version: 2.1.14
Precedence: list
Reply-To: Gleb Smirnoff <glebius@FreeBSD.org>
List-Id: Bug reports <freebsd-bugs.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/options/freebsd-bugs>,
 <mailto:freebsd-bugs-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-bugs>
List-Post: <mailto:freebsd-bugs@freebsd.org>
List-Help: <mailto:freebsd-bugs-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-bugs>,
 <mailto:freebsd-bugs-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Tue, 05 Feb 2013 12:10:01 -0000

The following reply was made to PR kern/175759; it has been noted by GNATS.

From: Gleb Smirnoff <glebius@FreeBSD.org>
To: Andrey Simonenko <simon@comsys.ntu-kpi.kiev.ua>
Cc: Bruce Evans <brde@optusnet.com.au>, FreeBSD-gnats-submit@freebsd.org
Subject: Re: kern/175759: Correct data types for fields of struct qm_trace{}
 from <sys/queue.h>
Date: Tue, 5 Feb 2013 16:02:18 +0400

 On Tue, Feb 05, 2013 at 01:35:50PM +0200, Andrey Simonenko wrote:
 A> > Unsigned long is unnecessarily large.  It wastes space on 64-bit
 A> > arches.  The change doesn't change the wastage, because space was
 A> > already wasted on 64-bit arches by mispacking the struct (with
 A> > unnamed padding after the ints).  It changes the API unnecessarily
 A> > by changing signed variables to unsigned.  Sign variables are
 A> > easier to use, and changing to unsigned ones risks sign extension
 A> > bugs.
 A> 
 A> I did not change order of fields to not change API, that's why
 A> bigger data type is used without changing size of that structure
 A> due to padding (at least for current sizes of int and long).
 
 We don't claim to be ABI stable for binaries that contain additional
 debugging information. They don't go into releases, and they alredy
 are incompatible with non-debugging binaries.
 
 So this isn't a problem.
 
 
 -- 
 Totus tuus, Glebius.