LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* (no subject)
@ 2008-02-17 21:07 dpreed
  0 siblings, 0 replies; only message in thread
From: dpreed @ 2008-02-17 21:07 UTC (permalink / raw)


>From David P. Reed <dpreed@reed.com> Sat Feb 16 15:05:17 2008
Message-Id: <20080216200517.113862251@reed.com>>
User-Agent: quilt/0.46-1
Date: Sat, 16 Feb 2008 15:05:17 -0500
From: David P. Reed <dpreed@reed.com>
To: Thomas Gleixner <tglx@linutronix.de>,
 Ingo Molnar <mingo@redhat.com>,
 "H. Peter Anvin" <hpa@zytor.com>
Cc: linux-kernel@vger.kernel.org
Subject: [PATCH 0/3] x86: cleanup primary motherboard chip port access delays

cleanup motherboard chip io port delays.  inb_p and outb_p have traditionally
used a write to port 80 (a non-existent port) as a delay.  Though there is an
argument that that is a good delay for devices on the ISA or PCI expansion buses
it is not a good mechanism for devices in the processor chipset or on the
"motherboard".  The write to port 80 at best causes an abort on the ISA or LPC
bus, and on some machines (like many of the HP laptops manufactured by Quanta)
actually writes data to real i/o devices.  For example, the ENE Embedded
Controller chip family defaults to provide a register at port 80 that 
can be written, and which can cause an interrupt in the Embedded Controller.

This has been shown to cause hangs on some machines, especially in accessing
the CMOS RTC during bootup.

This patch series addresses three of the places where these are used in common
kernel code - in particular the three uses that affect the HP laptops mentioned
above, modifying the delays to match the worst known delay issues for the
specific chips.

The patch set is complementary to the iodelay= kernel parameter added in 2.6.25,
since it means fewer users will need to add that parameter to run linux "out of the box" without hanging.

-- 

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2008-02-17 21:18 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-02-17 21:07 dpreed

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).