Re: [PATCH] SMM BIOS on Dell i8100

[Marcel J.E. Mol]

Maybe this is better:

diff -u i8kutils-1.2.orig/i8kbuttons.c i8kutils-1.2/i8kbuttons.c
--- i8kutils-1.2.orig/i8kbuttons.c      Tue Nov  6 20:07:27 2001
+++ i8kutils-1.2/i8kbuttons.c   Tue Nov  6 20:11:19 2001
@@ -53,15 +53,17 @@

     DPRINTF("exec_cmd: %s\n", cmd);

     if ((rc=fork()) < 0) {
        perror("fork failed");
        return;
     }
 
     if (rc == 0) {
        execl("/bin/sh", "sh", "-c", cmd, NULL);
        exit(0);
     }
+    else
+       wait(&rc);
 }
 
It get rid of the zombies and allows only one setmixer/auimix command to be
active at a time...

On Wed, Nov 07, 2001 at 10:44:05AM +0100, Stephane Jourdois wrote:
> On Tue, Nov 06, 2001 at 05:45:13PM +0100, Massimo Dal Zotto wrote:
> > I have released version 1.2 of the driver. It contains Stephane's patches
> > for the I8100, a new i8kmon and some documentation. You can download from:
> >     http://www.debian.org/~dz/i8k/
> 
> Hello,
> 
> Here is my patch to i8kutils-1.2 :
> 
> ======
> diff -u i8kutils-1.2.orig/i8kbuttons.c i8kutils-1.2/i8kbuttons.c
> --- i8kutils-1.2.orig/i8kbuttons.c      Tue Nov  6 20:07:27 2001
> +++ i8kutils-1.2/i8kbuttons.c   Tue Nov  6 20:11:19 2001
> @@ -53,15 +53,7 @@
> 
>      DPRINTF("exec_cmd: %s\n", cmd);
> 
> -    if ((rc=fork()) < 0) {
> -       perror("fork failed");
> -       return;
> -    }
> -
> -    if (rc == 0) {
> -       execl("/bin/sh", "sh", "-c", cmd, NULL);
> -       exit(0);
> -    }
> +    system(cmd);
>  }
>  
>  static int
> ======
> 
> Without that, I get as much zombies processes as I have pressed the
> volume buttons :-) I know system() is not great, but as security is not
> a problem here...
> I use debian/sid, and aumix as the mixer.
> 
> 
> Now a fundamental question :
> Does the load of the i8k module inhibits the fans start ? I can see my
> processor temp increasing (I saw 80°C ...) without the fans start. Then
> I started i8kmon to avoid an explosion. If the modules inhibits material
> protections, then if that can be modified, it would be great ; if not,
> i8kmon needs to get included in the kernel as a daemon. The i8kmon
> should be a funny tool, not a system critical tool.
> 
> Also, I couldn't understand why sometimes the left fan is printed on red
> color in i8kmon...
> 
> 
> Thank you,
> 
> 	Stephane
> 
> 
> -- 
>  ///  Stephane Jourdois        	/"\  ASCII RIBBON CAMPAIGN \\\
> (((    Ingénieur développement 	\ /    AGAINST HTML MAIL    )))
>  \\\   6, av. de la Belle Image	 X                         ///
>   \\\  94440 Marolles en Brie  	/ \    [phone]    ///

-- 
     ======--------         Marcel J.E. Mol                MESA Consulting B.V.
    =======---------        ph. [phone]          P.O. Box 112
    =======---------        [e-mail]                 2630 AC  Nootdorp
__==== www.mesa.nl ---____U_n_i_x______I_n_t_e_r_n_e_t____ The Netherlands ____
 They couldn't think of a number,           Linux user 1148  --  counter.li.org
    so they gave me a name!  -- Rupert Hine  --  www.ruperthine.com