Comment 3 for bug 1347776

Revision history for this message
Manoj Iyer (manjo) wrote :

SRU Request
==========

[Impact]

* User won't be able to initiate a soft shutdown from the chassis manager.

[Test Case]

* To reproduce this bug, initiate a soft shutdown from the chassis manager, for example from ilo you could do
<ilo> set node power off shutdown <node number>

[Test Result]

== BEFORE PATCH ==

$ cat /lib/udev/rules.d/70-power-switch.rules
# This file is part of systemd.
#
# systemd is free software; you can redistribute it and/or modify it
# under the terms of the GNU Lesser General Public License as published by
# the Free Software Foundation; either version 2.1 of the License, or
# (at your option) any later version.

ACTION=="remove", GOTO="power_switch_end"

SUBSYSTEM=="input", KERNEL=="event*", SUBSYSTEMS=="acpi", TAG+="power-switch"
SUBSYSTEM=="input", KERNEL=="event*", KERNELS=="thinkpad_acpi", TAG+="power-switch"

LABEL="power_switch_end"
$

</>hpiLO-> set node power off shutdown c3n2

  c3: #Cartridge 3
    c3n2: #Node 2 Shutting node down gracefully

hpiLO-> show node list

Slot ID Proc Manufacturer Architecture Memory Power Health
---- ----- ---------------------- -------------------- ------ ----- ------
  3 c3n1 **** ARM Architecture 8 GB On OK
  3 c3n2 **** ARM Architecture 8 GB On OK
  3 c3n3 **** ARM Architecture 8 GB On OK
  3 c3n4 **** ARM Architecture 8 GB Off OK

hpiLO->

== AFTER PATCH ==

hpiLO-> set node power off shutdown c3n1

  c3: #Cartridge 3
    c3n1: #Node 1 Shutting node down gracefully

hpiLO-> show node list

Slot ID Proc Manufacturer Architecture Memory Power Health
---- ----- ---------------------- -------------------- ------ ----- ------
  3 c3n1 **** ARM Architecture 8 GB Off OK
  3 c3n2 **** ARM Architecture 8 GB On OK
  3 c3n3 **** ARM Architecture 8 GB On OK
  3 c3n4 **** ARM Architecture 8 GB Off OK

hpiLO->

[Regression Potential]

None.
Note: Regarding the possibility of gpio_key.12 being used by other systems to map to some other trigger, I put in the check that the gpio_key.12 is associated with power control (keys=116). '116' is supposedly linux generic power control in DTS.