<feed xmlns='http://www.w3.org/2005/Atom'>
<title>kernel/linux.git/drivers/char/ipmi, branch v4.17.1</title>
<subtitle>Linux kernel stable tree (mirror)</subtitle>
<id>https://git.radix-linux.su/kernel/linux.git/atom?h=v4.17.1</id>
<link rel='self' href='https://git.radix-linux.su/kernel/linux.git/atom?h=v4.17.1'/>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/'/>
<updated>2018-03-27T16:04:17+00:00</updated>
<entry>
<title>ipmi/parisc: Add IPMI chassis poweroff for certain HP PA-RISC and IA-64 servers</title>
<updated>2018-03-27T16:04:17+00:00</updated>
<author>
<name>Helge Deller</name>
<email>deller@gmx.de</email>
</author>
<published>2018-03-27T15:58:32+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=c6185e285c5c7cfeab739bae7f206ced695f09c7'/>
<id>urn:sha1:c6185e285c5c7cfeab739bae7f206ced695f09c7</id>
<content type='text'>
This patch allows HP PA-RISC servers like rp3410/rp3440 and the HP C8000
workstation with an IPMI controller that predate IPMI 1.5 to use the standard
poweroff or powercycle commands.

These systems firmware don't set the chassis capability bit in the Get
Device ID, but they do implement the standard poweroff and powercycle
commands.

Signed-off-by: Helge Deller &lt;deller@gmx.de&gt;
Signed-off-by: Corey Minyard &lt;cminyard@mvista.com&gt;
</content>
</entry>
<entry>
<title>ipmi_ssif: Fix kernel panic at msg_done_handler</title>
<updated>2018-03-13T12:12:26+00:00</updated>
<author>
<name>Kamlakant Patel</name>
<email>kamlakant.patel@cavium.com</email>
</author>
<published>2018-03-13T11:02:27+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=f002612b9d86613bc6fde0a444e0095225f6053e'/>
<id>urn:sha1:f002612b9d86613bc6fde0a444e0095225f6053e</id>
<content type='text'>
This happens when BMC doesn't return any data and the code is trying
to print the value of data[2].

Getting following crash:
[  484.728410] Unable to handle kernel NULL pointer dereference at virtual address 00000002
[  484.736496] pgd = ffff0000094a2000
[  484.739885] [00000002] *pgd=00000047fcffe003, *pud=00000047fcffd003, *pmd=0000000000000000
[  484.748158] Internal error: Oops: 96000005 [#1] SMP
[...]
[  485.101451] Call trace:
[...]
[  485.188473] [&lt;ffff000000a46e68&gt;] msg_done_handler+0x668/0x700 [ipmi_ssif]
[  485.195249] [&lt;ffff000000a456b8&gt;] ipmi_ssif_thread+0x110/0x128 [ipmi_ssif]
[  485.202038] [&lt;ffff0000080f1430&gt;] kthread+0x108/0x138
[  485.206994] [&lt;ffff0000080838e0&gt;] ret_from_fork+0x10/0x30
[  485.212294] Code: aa1903e1 aa1803e0 b900227f 95fef6a5 (39400aa3)

Adding a check to validate the data len before printing data[2] to fix this issue.

Signed-off-by: Kamlakant Patel &lt;kamlakant.patel@cavium.com&gt;
Signed-off-by: Corey Minyard &lt;cminyard@mvista.com&gt;
</content>
</entry>
<entry>
<title>ipmi:pci: Blacklist a Realtek "IPMI" device</title>
<updated>2018-03-13T11:58:47+00:00</updated>
<author>
<name>Corey Minyard</name>
<email>cminyard@mvista.com</email>
</author>
<published>2018-02-15T22:58:26+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=bc48fa1b9d3b04106055b27078da824cd209865a'/>
<id>urn:sha1:bc48fa1b9d3b04106055b27078da824cd209865a</id>
<content type='text'>
Realtek has some sort of "Virtual" IPMI device on the PCI bus as a
KCS controller, but whatever it is, it's not one.  Ignore it if seen.

Reported-by: Chris Chiu &lt;chiu@endlessm.com&gt;
Signed-off-by: Corey Minyard &lt;cminyard@mvista.com&gt;
Tested-by: Daniel Drake &lt;drake@endlessm.com&gt;
</content>
</entry>
<entry>
<title>ipmi: Remove ACPI SPMI probing from the system interface driver</title>
<updated>2018-03-12T12:03:06+00:00</updated>
<author>
<name>Corey Minyard</name>
<email>cminyard@mvista.com</email>
</author>
<published>2018-03-08T20:57:53+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=4876234acc5441e325f262eec48672ee7f4a0cc6'/>
<id>urn:sha1:4876234acc5441e325f262eec48672ee7f4a0cc6</id>
<content type='text'>
The IPMI spec states:

  The purpose of the SPMI Table is to provide a mechanism that can
  be used by the OSPM (an ACPI term for “OS Operating System-directed
  configuration and Power Management” essentially meaning an ACPI-aware
  OS or OS loader) very early in the boot process, e.g., before the
  ability to execute ACPI control methods in the OS is available.

When we are probing IPMI in Linux, ACPI control methods are available,
so we shouldn't be probing using SPMI.  It could cause some confusion
during the probing process.

Signed-off-by: Corey Minyard &lt;cminyard@mvista.com&gt;
</content>
</entry>
<entry>
<title>ipmi: Remove ACPI SPMI probing from the SSIF (I2C) driver</title>
<updated>2018-03-12T12:02:47+00:00</updated>
<author>
<name>Corey Minyard</name>
<email>cminyard@mvista.com</email>
</author>
<published>2018-03-08T21:00:28+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=4866b1dce0389013a268f0ab63f7229b30c6e5fe'/>
<id>urn:sha1:4866b1dce0389013a268f0ab63f7229b30c6e5fe</id>
<content type='text'>
The IPMI spec states:

  The purpose of the SPMI Table is to provide a mechanism that can
  be used by the OSPM (an ACPI term for “OS Operating System-directed
  configuration and Power Management” essentially meaning an ACPI-aware
  OS or OS loader) very early in the boot process, e.g., before the
  ability to execute ACPI control methods in the OS is available.

When we are probing IPMI in Linux, ACPI control methods are available,
so we shouldn't be probing using SPMI.  It could cause some confusion
during the probing process.

Signed-off-by: Corey Minyard &lt;cminyard@mvista.com&gt;
Tested-by: Jiandi An &lt;anjiandi@codeaurora.org&gt;
</content>
</entry>
<entry>
<title>ipmi: missing error code in try_smi_init()</title>
<updated>2018-03-07T01:47:52+00:00</updated>
<author>
<name>Dan Carpenter</name>
<email>dan.carpenter@oracle.com</email>
</author>
<published>2018-03-06T09:58:03+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=532ed9264da27eb3062f49e4283f9b393a8e17d0'/>
<id>urn:sha1:532ed9264da27eb3062f49e4283f9b393a8e17d0</id>
<content type='text'>
If platform_device_alloc() then we should return -ENOMEM instead of
returning success.

Signed-off-by: Dan Carpenter &lt;dan.carpenter@oracle.com&gt;
Signed-off-by: Corey Minyard &lt;cminyard@mvista.com&gt;
</content>
</entry>
<entry>
<title>ipmi: use ARRAY_SIZE for poweroff_functions array sizing calculation</title>
<updated>2018-03-07T01:47:51+00:00</updated>
<author>
<name>Colin Ian King</name>
<email>colin.king@canonical.com</email>
</author>
<published>2018-03-02T14:14:51+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=e8824babad2d81fcff43e02f028cc437a3f99ce5'/>
<id>urn:sha1:e8824babad2d81fcff43e02f028cc437a3f99ce5</id>
<content type='text'>
Use the ARRAY_SIZE macro on a array poweroff_functions to determine
size of the array. Improvement suggested by Coccinelle.

Signed-off-by: Colin Ian King &lt;colin.king@canonical.com&gt;
Signed-off-by: Corey Minyard &lt;cminyard@mvista.com&gt;
</content>
</entry>
<entry>
<title>ipmi: Consolidate cleanup code</title>
<updated>2018-03-07T01:47:50+00:00</updated>
<author>
<name>Corey Minyard</name>
<email>cminyard@mvista.com</email>
</author>
<published>2018-02-28T17:40:16+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=71404a2f75c1456d82eda47c3a9699770dc766c9'/>
<id>urn:sha1:71404a2f75c1456d82eda47c3a9699770dc766c9</id>
<content type='text'>
The cleanup code for an init failure and for a device removal were
quite similar, consolidate all that into one function.

Signed-off-by: Corey Minyard &lt;cminyard@mvista.com&gt;
</content>
</entry>
<entry>
<title>ipmi: Remove some unnecessary initializations</title>
<updated>2018-03-07T01:47:49+00:00</updated>
<author>
<name>Corey Minyard</name>
<email>cminyard@mvista.com</email>
</author>
<published>2018-02-28T13:56:47+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=da18e2c6a81376ff37a326d28f16c3dcaa368f90'/>
<id>urn:sha1:da18e2c6a81376ff37a326d28f16c3dcaa368f90</id>
<content type='text'>
The data is allocated with kzalloc, no need to set things to NULL.

Signed-off-by: Corey Minyard &lt;cminyard@mvista.com&gt;
</content>
</entry>
<entry>
<title>ipmi: Fix some error cleanup issues</title>
<updated>2018-03-07T01:47:17+00:00</updated>
<author>
<name>Corey Minyard</name>
<email>cminyard@mvista.com</email>
</author>
<published>2018-02-28T14:09:49+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=cc095f0ac1f7c200e51a5c2a78a43c9f42049dbb'/>
<id>urn:sha1:cc095f0ac1f7c200e51a5c2a78a43c9f42049dbb</id>
<content type='text'>
device_remove_group() was called on any cleanup, even if the
device attrs had not been added yet.  That can occur in certain
error scenarios, so add a flag to know if it has been added.

Also make sure we remove the dev if we added it ourselves.

Signed-off-by: Corey Minyard &lt;cminyard@mvista.com&gt;
Cc: stable@vger.kernel.org # 4.15
Cc: Laura Abbott &lt;labbott@redhat.com&gt;
Tested-by: Bill Perkins &lt;wmp@grnwood.net&gt;
</content>
</entry>
</feed>
