Re: [PATCH] SMM BIOS on Dell i8100

Marcel J.E. Mol (marcel@mesa.nl)
Wed, 7 Nov 2001 11:01:41 +0100


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 / \ +33 6 8643 3085 ///

-- 
     ======--------         Marcel J.E. Mol                MESA Consulting B.V.
    =======---------        ph. +31-(0)6-54724868          P.O. Box 112
    =======---------        marcel@mesa.nl                 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
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/