Re: [Patch 0/2] powerpc: avoid userspace poking to legacy ioports

On Wednesday 13 February 2008, Benjamin Herrenschmidt wrote:

On Wed, 2008-02-13 at 18:35 +0100, Christian Krafft wrote:

quoted

sensors_detect crashes kernel on PowerPC, as it pokes directly to memory.
This patch adds a check_legacy_ioports to read_port and write_port.
It will now return ENXIO, instead of oopsing.

Signed-off-by: Christian Krafft <redacted>

The problem is that this prevents using /proc/ioports to access PCI
IO space, which might be useful.

I hate that sensors_detect.. or for that matter any other userland code
that pokes random ports like that. It should die.

What kind of Oops do you get? Is it because the ioport area is not
ioremapped at all or do you get a machine check? If there is no
mapping, we could possibly change inb and outb do deal with that.

	Arnd <><