server seed: remove acpid?

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

server seed: remove acpid?

Andreas Hasenack-5
Hi,

we are reviewing the server seeds
(https://code.launchpad.net/~ubuntu-core-dev/ubuntu-seeds/+git/ubuntu/+ref/disco)
and the acpid package came up. It's currently in the server seed,
meaning it's installed by default.

I did some tests in a VM, and a bare metal machine, running cosmic,
and the power and suspend buttons, both from an usb keyboard and the
box itself, worked just fine without acpid running.
In fact, it's not even running on a default install unless the admin
either enables it explicitly, or installs a package that populates
/etc/acpi/events, or connects to the unix socket in /run/acpid.socket.

Does anybody know what we would miss by not having it installed by default?

--
ubuntu-devel mailing list
[hidden email]
Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel
Reply | Threaded
Open this post in threaded view
|

Re: server seed: remove acpid?

Robert Collins


On Thu., 1 Nov. 2018, 08:55 Andreas Hasenack <[hidden email] wrote:
Hi,

we are reviewing the server seeds
(https://code.launchpad.net/~ubuntu-core-dev/ubuntu-seeds/+git/ubuntu/+ref/disco)
and the acpid package came up. It's currently in the server seed,
meaning it's installed by default.

I did some tests in a VM, and a bare metal machine, running cosmic,
and the power and suspend buttons, both from an usb keyboard and the
box itself, worked just fine without acpid running.
In fact, it's not even running on a default install unless the admin
either enables it explicitly, or installs a package that populates
/etc/acpi/events, or connects to the unix socket in /run/acpid.socket.

That's good I think. Systemd has built in ACPI event support, better not be processing sleep twice.

As far as I know removing it should be fine.

Rob

--
ubuntu-devel mailing list
[hidden email]
Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel
Reply | Threaded
Open this post in threaded view
|

Re: server seed: remove acpid?

Steve Langasek-6
In reply to this post by Andreas Hasenack-5
On Wed, Oct 31, 2018 at 04:54:54PM -0300, Andreas Hasenack wrote:
> Hi,

> we are reviewing the server seeds
> (https://code.launchpad.net/~ubuntu-core-dev/ubuntu-seeds/+git/ubuntu/+ref/disco)
> and the acpid package came up. It's currently in the server seed,
> meaning it's installed by default.

> I did some tests in a VM, and a bare metal machine, running cosmic,
> and the power and suspend buttons, both from an usb keyboard and the
> box itself, worked just fine without acpid running.
> In fact, it's not even running on a default install unless the admin
> either enables it explicitly, or installs a package that populates
> /etc/acpi/events, or connects to the unix socket in /run/acpid.socket.

> Does anybody know what we would miss by not having it installed by default?

The above is what I expected to see, which is why I proposed it for dropping
from the seed.  I expect systemd to handle the core acpi events (shutdown,
suspend) directly, with no need for acpid; so acpid is only needed if you
have other custom ACPI event handlers.  If you have custom ACPI event
handlers on your server, you can install the acpid package as needed; but it
is no longer part of our platform and should not be installed by default.

(It is separately still being included by default in the desktop seed
because it is a dependency of acpi-support, which ships a handful of event
handlers that had never been ported elsewhere; it is worth revisiting
whether these are now handled elsewhere or if they should be dropped as
obsolete, but in any case that should not stop us from dropping acpid from
the server seed.)

--
Steve Langasek                   Give me a lever long enough and a Free OS
Debian Developer                   to set it on, and I can move the world.
Ubuntu Developer                                   https://www.debian.org/
[hidden email]                                     [hidden email]

--
ubuntu-devel mailing list
[hidden email]
Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel

signature.asc (849 bytes) Download Attachment