From owner-cvs-src@FreeBSD.ORG Sun Aug 19 00:52:03 2007 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 5C16816A419; Sun, 19 Aug 2007 00:52:03 +0000 (UTC) (envelope-from deischen@freebsd.org) Received: from mail.netplex.net (mail.netplex.net [204.213.176.10]) by mx1.freebsd.org (Postfix) with ESMTP id 159BD13C469; Sun, 19 Aug 2007 00:52:02 +0000 (UTC) (envelope-from deischen@freebsd.org) Received: from sea.ntplx.net (sea.ntplx.net [204.213.176.11]) by mail.netplex.net (8.14.1/8.14.1/NETPLEX) with ESMTP id l7J0pvaB010804; Sat, 18 Aug 2007 20:51:57 -0400 (EDT) X-Virus-Scanned: by AMaViS and Clam AntiVirus (mail.netplex.net) X-Greylist: Message whitelisted by DRAC access database, not delayed by milter-greylist-3.0 (mail.netplex.net [204.213.176.10]); Sat, 18 Aug 2007 20:51:58 -0400 (EDT) Date: Sat, 18 Aug 2007 20:51:57 -0400 (EDT) From: Daniel Eischen X-X-Sender: eischen@sea.ntplx.net To: Robert Watson In-Reply-To: <20070818204223.D1234@fledge.watson.org> Message-ID: References: <200708160526.l7G5Qg0b008022@repoman.freebsd.org> <46C4FD02.3090708@freebsd.org> <200708182118.37998.tijl@ulyssis.org> <20070818204223.D1234@fledge.watson.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed Cc: wine-freebsd@hub.org, src-committers@freebsd.org, cvs-src@freebsd.org, cvs-all@freebsd.org, David Xu , Tijl Coosemans , Xin LI Subject: Re: cvs commit: src/sys/kern kern_thr.c syscalls.master src/sys/sys thr.h X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: Daniel Eischen List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 19 Aug 2007 00:52:03 -0000 On Sat, 18 Aug 2007, Robert Watson wrote: > > On Sat, 18 Aug 2007, Tijl Coosemans wrote: > >> Perhaps it should be stressed in documentation that this syscall is only >> meant for Wine and to be removed at some point when better solutions become >> available. I want to test this sigqueue solution some more, because the >> indirection in signal delivery causes trouble when a thread signals itself. >> I guess this would have to be special-cased somehow then. > > As a general rule, removing system calls is discouraged as it breaks backward > compatibility allowing newer kernels to run older applications, so I think we > should avoid adding a system call with the intent to remove it in mind at the > time it's added :-). While thr_kill2() is imperfect from several > perspectives, it is certainly a minimalist and logical construction that can > be easily understood, and that's pretty valuable. You can say that about a lot of APIs, but that doesn't mean we should add them. This is a bit of a hack to satisfy one application (Wine) when there are other (perhaps harder) ways to skin the cat. We certainly don't want anything else using this API, so I'd advocate removing it. -- DE