[uml-devel] Re: [patch 2/2] add-SIGPROF-get-set_signals

[Jeff Dike <jdike@addtoit.com>]

On Sun, Sep 12, 2004 at 08:26:29PM +0200, blaisorblade_spam@yahoo.it wrote:
>  {
>  	sigset_t mask;
> +	sigset_t old_mask;
>  	int ret;

Weren't you just complaining about stack consumption?  Do you know how
big a sigset_t is?

	(gdb) p sizeof(sigset_t)
	$1 = 128

I went to a reasonable amount of trouble to get that function to only
use one sigset_t and now you are proposing to add the old one back.  

Fix the patch and I'll put it in.

Also, if you're going to fix something, send a patch which does
nothing but make the fix, and save whatever cleanups you want to make
for a differet patch.

				Jeff


-------------------------------------------------------
This SF.Net email is sponsored by: YOU BE THE JUDGE. Be one of 170
Project Admins to receive an Apple iPod Mini FREE for your judgement on
who ports your project to Linux PPC the best. Sponsored by IBM. 
Deadline: Sept. 13. Go here: http://sf.net/ppc_contest.php
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel