<feed xmlns='http://www.w3.org/2005/Atom'>
<title>linux/drivers/ata/ata_piix.c, branch v2.6.28</title>
<subtitle>Linux Kernel (branches are rebased on master from time to time)</subtitle>
<id>https://sre.ring0.de/linux/atom?h=v2.6.28</id>
<link rel='self' href='https://sre.ring0.de/linux/atom?h=v2.6.28'/>
<link rel='alternate' type='text/html' href='https://sre.ring0.de/linux/'/>
<updated>2008-12-09T05:43:54Z</updated>
<entry>
<title>ATA: piix, fix pointer deref on suspend</title>
<updated>2008-12-09T05:43:54Z</updated>
<author>
<name>Jiri Slaby</name>
<email>jirislaby@gmail.com</email>
</author>
<published>2008-12-03T08:13:46Z</published>
<link rel='alternate' type='text/html' href='https://sre.ring0.de/linux/commit/?id=9ac6212660fa3d8d0d21e782a22b956a6bc78a43'/>
<id>urn:sha1:9ac6212660fa3d8d0d21e782a22b956a6bc78a43</id>
<content type='text'>
Hi,

I've found this issue in the mmotm 2008-12-02-17-08.

--

Commit
ata_piix: add borked Tecra M4 to broken suspend list
introduced DMI variables checking, but they can be null, so that
we possibly dereference null.

Check if they are null and avoid checks in that case.

Solves:
BUG: unable to handle kernel NULL pointer dereference at 0000000000000000
IP: [&lt;ffffffff8043da97&gt;] piix_pci_device_suspend+0x117/0x230

Signed-off-by: Jiri Slaby &lt;jirislaby@gmail.com&gt;
Cc: Jeff Garzik &lt;jgarzik@redhat.com&gt;
Cc: Alexandru Romanescu &lt;a_romanescu@yahoo.co.uk&gt;
Cc: Tejun Heo &lt;tj@kernel.org&gt;
Signed-off-by: Jeff Garzik &lt;jgarzik@redhat.com&gt;
</content>
</entry>
<entry>
<title>ata_piix: add borked Tecra M4 to broken suspend list</title>
<updated>2008-12-01T18:48:45Z</updated>
<author>
<name>Tejun Heo</name>
<email>tj@kernel.org</email>
</author>
<published>2008-11-29T13:37:21Z</published>
<link rel='alternate' type='text/html' href='https://sre.ring0.de/linux/commit/?id=1eedb4a90c958d8d59e0e4f19c297b445df21cf9'/>
<id>urn:sha1:1eedb4a90c958d8d59e0e4f19c297b445df21cf9</id>
<content type='text'>
Tecra M4 sometimes forget what it is and reports bogus data via DMI
which makes the machine evade broken suspend matching and thus fail
suspend/resume.  This patch updates piix_broken_suspend() such that it
can match such case.  As the borked DMI data is a bit generic,
matching many entries to make the match more specific is necessary.
As the usual DMI matching is limited to four entries, this patch uses
hard coded manual matching.

This is reported by Alexandru Romanescu.

Signed-off-by: Tejun Heo &lt;tj@kernel.org&gt;
Cc: Alexandru Romanescu &lt;a_romanescu@yahoo.co.uk&gt;
Signed-off-by: Jeff Garzik &lt;jgarzik@redhat.com&gt;
</content>
</entry>
<entry>
<title>ATA: remove excess kernel-doc notation</title>
<updated>2008-10-31T05:42:49Z</updated>
<author>
<name>Randy Dunlap</name>
<email>randy.dunlap@oracle.com</email>
</author>
<published>2008-10-30T05:35:08Z</published>
<link rel='alternate' type='text/html' href='https://sre.ring0.de/linux/commit/?id=5b97fbd093ac2e0da0c7eec894fee065a04af55d'/>
<id>urn:sha1:5b97fbd093ac2e0da0c7eec894fee065a04af55d</id>
<content type='text'>
Remove excess kernel-doc function parameter notation from drivers/ata/:

Warning(drivers/ata/libata-core.c:1622): Excess function parameter or struct member 'fn' description in 'ata_pio_queue_task'
Warning(drivers/ata/libata-core.c:4655): Excess function parameter or struct member 'err_mask' description in 'ata_qc_complete'
Warning(drivers/ata/ata_piix.c:751): Excess function parameter or struct member 'udma' description in 'do_pata_set_dmamode'

Signed-off-by: Randy Dunlap &lt;randy.dunlap@oracle.com&gt;
Signed-off-by: Jeff Garzik &lt;jgarzik@redhat.com&gt;
</content>
</entry>
<entry>
<title>ata: Switch all my stuff to a common address</title>
<updated>2008-10-28T03:54:50Z</updated>
<author>
<name>Alan Cox</name>
<email>alan@lxorguk.ukuu.org.uk</email>
</author>
<published>2008-10-27T15:09:10Z</published>
<link rel='alternate' type='text/html' href='https://sre.ring0.de/linux/commit/?id=ab77163008c596aad9624ceab190d840c0143fa8'/>
<id>urn:sha1:ab77163008c596aad9624ceab190d840c0143fa8</id>
<content type='text'>
Signed-off-by: Jeff Garzik &lt;jgarzik@redhat.com&gt;
</content>
</entry>
<entry>
<title>ata_piix: IDE Mode SATA patch for Intel Ibex Peak DeviceIDs</title>
<updated>2008-09-29T04:29:12Z</updated>
<author>
<name>Seth Heasley</name>
<email>seth.heasley@intel.com</email>
</author>
<published>2008-08-27T23:40:06Z</published>
<link rel='alternate' type='text/html' href='https://sre.ring0.de/linux/commit/?id=0395e61babd59c749fb5efe112affbfaa7d50eb7'/>
<id>urn:sha1:0395e61babd59c749fb5efe112affbfaa7d50eb7</id>
<content type='text'>
This patch updates the Intel Ibex Peak (PCH) IDE mode SATA Controller DeviceIDs.

Signed-off-by: Seth Heasley &lt;seth.heasley@intel.com&gt;
Signed-off-by: Jeff Garzik &lt;jgarzik@redhat.com&gt;
</content>
</entry>
<entry>
<title>ata_piix: add Hercules EC-900 mini-notebook to ich_laptop short cable list</title>
<updated>2008-09-29T04:28:40Z</updated>
<author>
<name>Herton Ronaldo Krzesinski</name>
<email>herton@mandriva.com.br</email>
</author>
<published>2008-09-17T17:29:05Z</published>
<link rel='alternate' type='text/html' href='https://sre.ring0.de/linux/commit/?id=d09addf65cb5b3b19a536aa3329efeedbc6bb56c'/>
<id>urn:sha1:d09addf65cb5b3b19a536aa3329efeedbc6bb56c</id>
<content type='text'>
Signed-off-by: Herton Ronaldo Krzesinski &lt;herton@mandriva.com.br&gt;
Signed-off-by: Jeff Garzik &lt;jgarzik@redhat.com&gt;
</content>
</entry>
<entry>
<title>ata_piix: drop merged SCR access and use slave_link instead</title>
<updated>2008-09-29T04:25:32Z</updated>
<author>
<name>Tejun Heo</name>
<email>tj@kernel.org</email>
</author>
<published>2008-07-31T08:02:44Z</published>
<link rel='alternate' type='text/html' href='https://sre.ring0.de/linux/commit/?id=be77e43abb433c2d6f2fc69352289e34dcbf040a'/>
<id>urn:sha1:be77e43abb433c2d6f2fc69352289e34dcbf040a</id>
<content type='text'>
Now that libata has slave_link, there's no need to keep ugly merged
SCR access.  Drop it and use slave_link instead.  This results in
simpler code and much better separate link handling for master and
slave.

Signed-off-by: Tejun Heo &lt;tj@kernel.org&gt;
Signed-off-by: Jeff Garzik &lt;jgarzik@redhat.com&gt;
</content>
</entry>
<entry>
<title>libata: make SCR access ops per-link</title>
<updated>2008-09-29T04:22:28Z</updated>
<author>
<name>Tejun Heo</name>
<email>tj@kernel.org</email>
</author>
<published>2008-07-31T08:02:40Z</published>
<link rel='alternate' type='text/html' href='https://sre.ring0.de/linux/commit/?id=82ef04fb4c82542b3eda81cca461f0594ce9cd0b'/>
<id>urn:sha1:82ef04fb4c82542b3eda81cca461f0594ce9cd0b</id>
<content type='text'>
Logically, SCR access ops should take @link; however, there was no
compelling reason to convert all SCR access ops when adding @link
abstraction as there's one-to-one mapping between a port and a non-PMP
link.  However, that assumption won't hold anymore with the scheduled
addition of slave link.

Make SCR access ops per-link.

Signed-off-by: Tejun Heo &lt;tj@kernel.org&gt;
Signed-off-by: Jeff Garzik &lt;jgarzik@redhat.com&gt;
</content>
</entry>
<entry>
<title>ata: duplicate variable sparse warning</title>
<updated>2008-09-13T20:43:18Z</updated>
<author>
<name>Stephen Hemminger</name>
<email>shemminger@vyatta.com</email>
</author>
<published>2008-09-08T16:31:39Z</published>
<link rel='alternate' type='text/html' href='https://sre.ring0.de/linux/commit/?id=da3ceb2288d0b50373b69d57a81c34fdd7cd11aa'/>
<id>urn:sha1:da3ceb2288d0b50373b69d57a81c34fdd7cd11aa</id>
<content type='text'>
drivers/ata/ata_piix.c:1502:7: warning: symbol 'rc' shadows an earlier one

Signed-off-by: Stephen Hemminger &lt;shemminger@vyatta.com&gt;
Signed-off-by: Jeff Garzik &lt;jgarzik@redhat.com&gt;
</content>
</entry>
<entry>
<title>ata_piix: IDE Mode SATA patch for Intel Ibex Peak DeviceIDs</title>
<updated>2008-08-22T06:08:04Z</updated>
<author>
<name>Seth Heasley</name>
<email>seth.heasley@intel.com</email>
</author>
<published>2008-08-12T00:03:18Z</published>
<link rel='alternate' type='text/html' href='https://sre.ring0.de/linux/commit/?id=c6c6a1afefe51d488c050464f261d4711bea9a37'/>
<id>urn:sha1:c6c6a1afefe51d488c050464f261d4711bea9a37</id>
<content type='text'>
This patch adds the Intel Ibex Peak (PCH) IDE mode SATA Controller DeviceIDs.

Signed-off-by: Seth Heasley &lt;seth.heasley@intel.com&gt;
Signed-off-by: Jeff Garzik &lt;jgarzik@redhat.com&gt;
</content>
</entry>
</feed>
