[PATCH] weston-launch: Protect KDGKBMODE K_OFF ioctl by KERNEL_VERSION check

Daniel Stone daniel at fooishbar.org
Wed Nov 16 16:48:29 UTC 2016


Hi Krzysztof,

On 27 September 2016 at 22:44, Dima Ryazanov <dima at gmail.com> wrote:
> The kernel version used to build Weston isn't necessarily the same as the
> version that will be used to run it. Weston should already work fine on
> older versions: the second ioctl will return an error - but it's ok as long
> as the first one succeeds.
>
> Also, a compile-time check would prevent Weston built on an old kernel from
> taking advantage of new features when running on a new kernel.

I agree with Dima here. I think it would be better to use the approach
we use elsewhere, which is an #ifdef/#define/#endif. This lets us
build on older kernels, without needing #ifdefs in the main codepath.
As Dima says, the ioctl will just fail anyway.

Cheers,
Daniel


More information about the wayland-devel mailing list