From owner-cvs-all Sun Nov 14 5:15:24 1999 Delivered-To: cvs-all@freebsd.org Received: from post.mail.nl.demon.net (post-10.mail.nl.demon.net [194.159.73.20]) by hub.freebsd.org (Postfix) with ESMTP id 6973D14D17; Sun, 14 Nov 1999 05:15:17 -0800 (PST) (envelope-from marcel@scc.nl) Received: from [212.238.132.94] (helo=scones.sup.scc.nl) by post.mail.nl.demon.net with esmtp (Exim 2.02 #1) id 11mzUi-00048p-00; Sun, 14 Nov 1999 13:15:00 +0000 Received: from scc.nl (scones.sup.scc.nl [192.168.2.4]) by scones.sup.scc.nl (8.9.3/8.9.3) with ESMTP id OAA78593; Sun, 14 Nov 1999 14:14:53 +0100 (CET) (envelope-from marcel@scc.nl) Message-ID: <382EB5CD.823839D6@scc.nl> Date: Sun, 14 Nov 1999 14:14:53 +0100 From: Marcel Moolenaar Organization: SCC vof X-Mailer: Mozilla 4.7 [en] (X11; I; Linux 2.2.5 i386) X-Accept-Language: en MIME-Version: 1.0 To: Bruce Evans Cc: Martin Cracauer , cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/i386/include signal.h References: Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk Bruce Evans wrote: > > On Sun, 14 Nov 1999, Marcel Moolenaar wrote: > > > Martin Cracauer wrote: > > > > > Reserve space for FPU state in struct sigcontext. Fix some style bugs > > > > and comments while there. > > > > > > I still don't like this approach of unstructred allocated space, an > > > array of unnamed bytes. > > > > It's what it says it is: it's a space reservation. If someone actually > > implements saving the FPU state, then he/she is free (IMO) to change the > > reservation into whatever he/she things he/she needs or thinks is > > meaningful. > > Martin implemented saving it before you complicated things by changing > the signal handling :-). You can't expect me to know what patchsets everybody has laying around on their machines :-) > We seem to have missed allocating some necessary space. glibc-2.1.1 has: > > int sc_fpkind; /* FP_NO, FP_387, etc. */ > int sc_fpused; /* If zero, ignore rest of float state. */ > > int at least some declarations of struct sigcontext. I consider that very implementation specific and therefore can't say that it's necessary in any implementation we come up with. In any case, I don't think the kind of FP is context sensitive and needs to be stored in the context itself. Also, there are plenty of bits (and bit combinations) in the total space that can encode whether the context data has any meaning or not and to what degree. If we need to save the context of the GPL math emulator, we still have exactly 1 int32_t left for our own purposes (eg flags :-). -- Marcel Moolenaar mailto:marcel@scc.nl SCC Internetworking & Databases http://www.scc.nl/ The FreeBSD project mailto:marcel@FreeBSD.org To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message