From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rui Santos Subject: Re: [WATCHDOG] iTCO_wdt.c - ICH9 reboot issue - testing wanted Date: Mon, 15 Jun 2009 20:58:37 +0100 Message-ID: <4A36A7ED.2050702@grupopie.com> References: <20090430094918.GC4141@infomag.iguana.be> <4A2967F6.2000303@grupopie.com> <20090610081845.GH16090@infomag.iguana.be> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20090610081845.GH16090-flHiHfN8CTwhDM6iD19NGrNAH6kLmebB@public.gmane.org> Sender: kernel-testers-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-ID: Content-Type: text/plain; charset="us-ascii" To: Wim Van Sebroeck Cc: Stephen Clark , Denys Fedoryschenko , Johannes Dewender , "Rafael J. Wysocki" , Frans Pop , Rutger Nijlunsing , Kernel Testers List , Andriy Gapon , Andrew Morton , LKML Wim Van Sebroeck wrote: > Hi Rui, > Hi Wim, > >> With your patch, the Intel DG35EC board will not allow my distribution >> reboot or halt the machine. In order to circumvent that problem, I've >> made a few addition to your previous patch witch allows the restore of >> the changed Bit 0 to it's previous value if the module is unloaded. >> My only doubt is if it should be done every time the gbl_smi_en is zero, >> or in conjunction with nowayout when the value also equals zero. This >> patch has what I described and a commented gbl_smi_en only. >> > > Forget the previous patch. I don't like having this hack in the main iTCO_wdt code. > So I added it to the iTCO_vendor_support code with the necessary warnings. > Can you test this (please note that the iTCO_vendor_support module needs to be > loaded with the vendorsupport=911 module parameter). > It's working like a charm. Also, the Bit 0 is changed to its original value, so all reboots work. I've tested on 2.6.30 and 2.6.27 ( with a few mods... ). > Thanks in advance, > Thank you, > Wim. > Rui