linux-kernel - Re: [PATCH] amd64_edac: fix insmod fail while ecc is enabled

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <368206.16549.qm@web50108.mail.re2.yahoo.com>
Date:	Mon, 27 Jul 2009 11:01:26 -0700 (PDT)
From:	Doug Thompson <norsk5@...oo.com>
To:	dougthompson@...ssion.com, borislav.petkov@....com,
	wan wei <onewayforever@...il.com>
Cc:	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] amd64_edac: fix insmod fail while ecc is enabled

Thanks for this. 

It is the same as one I posted last week or so.

Without it, the module fails to initialize and no polling occurs.

Boris,  

In a prior email you indicated that you refactored the function in question. What that in the series of patches you submitted?  I think that set is too big to get in the RC branch.

We need this in 2.6.31 in order for the module to even work.

Shall we push my simple one, or your complete refactor?

doug t


--- On Mon, 7/27/09, wan wei <onewayforever@...il.com> wrote:

> From: wan wei <onewayforever@...il.com>
> Subject: [PATCH] amd64_edac: fix insmod fail while ecc is enabled
> To: dougthompson@...ssion.com, borislav.petkov@....com
> Cc: linux-kernel@...r.kernel.org
> Date: Monday, July 27, 2009, 10:47 AM
> Signed-off-by: Wan Wei<wanwei@...l.dawning.com.cn>
> 
> diff --git a/drivers/edac/amd64_edac.c
> b/drivers/edac/amd64_edac.c
> index 858fe60..95b1096 100644
> --- a/drivers/edac/amd64_edac.c
> +++ b/drivers/edac/amd64_edac.c
> @@ -2979,6 +2979,7 @@ static int
> amd64_check_ecc_enabled(struct amd64_pvt *pvt)
> 
>                 /*
> CLEAR the override, since BIOS controlled it */
>                
> ecc_enable_override = 0;
> +           
>    ret = 0;
>         }
> 
>         return ret;
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/