linux-kernel - RE: [HP ProLiant WatchDog driver] hpwdt HP WatchDog Patch

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: <66ED1F0546CFFE4BABB226CD6633D4A6015F7723@G3W0635.americas.hpqcorp.net>
Date:	Mon, 15 Oct 2007 20:24:48 -0000
From:	"Mingarelli, Thomas" <Thomas.Mingarelli@...com>
To:	<linux-kernel@...r.kernel.org>
Subject: RE: [HP ProLiant WatchDog driver] hpwdt HP WatchDog Patch

This could be done but then it adds complexity to the patch (as now two
different parts of the kernel need to be modified) and requires
regression testing for multiple platforms and OEM's.

This patch is isolated similar to the other HW specific HW Watchdog
Timers and yet provides a common Linux user interface that is applicable
across all OEM platforms that choose to implement a HW watchdog timer
device driver.

We chose to keep the changes isolated at this time to gain quick
acceptance upstream. 

-----Original Message-----
From: Arjan van de Ven [mailto:arjan@...radead.org] 
Sent: Monday, October 15, 2007 1:16 PM
To: Mingarelli, Thomas
Cc: linux-kernel@...r.kernel.org; Mingarelli, Thomas
Subject: Re: [HP ProLiant WatchDog driver] hpwdt HP WatchDog Patch

On Mon, 15 Oct 2007 14:05:50 -0400 (EDT) thomas.mingarelli@...com wrote:

> Hp is providing a Hardware WatchDog Timer driver that will only work 
> with the specific HW Timer located in the HP ProLiant iLO 2 ASIC. The 
> iLO 2 HW Timer will generate a Non-maskable Interrupt (NMI) 9 seconds 
> before physically resetting the server, by removing power, so that the

> event can be logged to the HP Integrated Management Log (IML), a 
> Non-Volatile Random Access Memory (NVRAM). The logging of the event is

> performed using the HP ProLiant ROM via an Industry Standard access 
> known as a BIOS Service Directory Entry.

Hi,

Your patch looks quite clean in general; however it does make me wonder
if it should either leverage or expand the arch/x86/pci/pcbios.c
infrastructure for doing BIOS32 calls and share that.... it's kinda
unpleasant as a general thought to have drivers poke this deep into
various guts of the system/bios.... esp if the common code has to do
something very similar already.

Greetings,
   Arjan van de Ven
-
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/