linux-kernel - Re: [PATCH v3 1/1] v4l2: use __u32 rather than enums in ioctl() structs

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <201205110916.11308.arnd@arndb.de>
Date:	Fri, 11 May 2012 09:16:10 +0000
From:	Arnd Bergmann <arnd@...db.de>
To:	Sakari Ailus <sakari.ailus@....fi>
Cc:	linux-kernel@...r.kernel.org, linux-media@...r.kernel.org,
	mchehab@...hat.com, remi@...lab.net, hverkuil@...all.nl,
	laurent.pinchart@...asonboard.com
Subject: Re: [PATCH v3 1/1] v4l2: use __u32 rather than enums in ioctl() structs

On Thursday 10 May 2012, Sakari Ailus wrote:
> The issue boils down to whether enums are fundamentally different from __u32
> or not, and can the former be substituted by the latter. During the
> discussion it was concluded that the __u32 has the same size as enums on all
> archs Linux is supported: it has not been shown that replacing those enums
> in IOCTL arguments would break neither source or binary compatibility. If no
> such reason is found, just replacing the enums with __u32s is the way to go.

Well, ARM Android was building stuff with short enums for a while, but it
seems that was corrected now, and using __u32 would in this case only
help maintain compatibility when mixing android kernels with regular
user space or vice versa.

	Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/