From owner-svn-src-head@freebsd.org Mon Nov 16 17:57:44 2015 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 4E51CA30A1D; Mon, 16 Nov 2015 17:57:44 +0000 (UTC) (envelope-from jhb@freebsd.org) Received: from bigwig.baldwin.cx (bigwig.baldwin.cx [IPv6:2001:470:1f11:75::1]) (using TLSv1 with cipher DHE-RSA-CAMELLIA256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 1CD851281; Mon, 16 Nov 2015 17:57:44 +0000 (UTC) (envelope-from jhb@freebsd.org) Received: from ralph.baldwin.cx (c-73-231-226-104.hsd1.ca.comcast.net [73.231.226.104]) by bigwig.baldwin.cx (Postfix) with ESMTPSA id 09DE6B9A8; Mon, 16 Nov 2015 12:57:43 -0500 (EST) From: John Baldwin To: Randall Stewart Cc: Bryan Drewery , "Alexander V. Chernikov" , "src-committers@freebsd.org" , "svn-src-all@freebsd.org" , "svn-src-head@freebsd.org" , Adrian Chadd , imp@freebsd.org Subject: Re: svn commit: r290664 - in head: share/man/man9 sys/kern sys/sys Date: Fri, 13 Nov 2015 15:47:12 -0800 Message-ID: <2705265.iiJmELE41x@ralph.baldwin.cx> User-Agent: KMail/4.14.3 (FreeBSD/10.2-STABLE; KDE/4.14.3; amd64; ; ) In-Reply-To: References: <201511101449.tAAEnXIi065747@repo.freebsd.org> <56465991.5090906@FreeBSD.org> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.2.7 (bigwig.baldwin.cx); Mon, 16 Nov 2015 12:57:43 -0500 (EST) X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.20 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: Mon, 16 Nov 2015 17:57:44 -0000 On Friday, November 13, 2015 04:47:53 PM Randall Stewart wrote: >=20 > On Nov 13, 2015, at 4:43 PM, Bryan Drewery wro= te: >=20 > > On 11/13/2015 1:24 PM, Randall Stewart wrote: > >> Looking at the patch, we need a define of your > >>=20 > >> _callout_stop_safe > >>=20 > >> and we need to switch > >>=20 > >> callout_stop()=E2=80=99s define to use the new _callout_stop_safe(= ) > >=20 > > For both cases, there would be no reason to have new code call my > > wrapper. The defines in my patch are fine for new code. The "new" > > _callout_stop_safe function is just for existing modules. > >=20 > > I think the historical consensus is to not commit my patch though a= s > > people should recompile their modules when the kernel is updated, b= efore > > rebooting into the new kernel even. >=20 > I did (as Han=E2=80=99s suggested) bump the FreeBSD version number.. >=20 > So maybe thats good enough. I think this will be problematic to MFC. It would be both an API and ABI change for existing modules, and callout(9) is a fairly common API that you would expect modules to use (not some internal thing that few if any modules use). The ABI stub would need to return the old values (so map -1 to 0). Changing the API in this way though means that code can silently be recompiled and break (there's no compile error, etc. The API only changes in that the semantics of the functions change). --=20 John Baldwin