On 11-07-25, 14:41, Takashi Iwai wrote:
On Fri, 11 Jul 2025 13:56:47 +0200,
Would it not be slightly simpler to just update all the in kernel bits to use 64-bit and then only convert to 32-bit for the existing 32-bit IOCTLs? Why do we need 32-bit callbacks into the drivers for example?
Right, it's a usual pattern to have only the 64bit ops in the kernel driver side while providing the 32bit stuff converted in the core layer. Having two different ops are rather confusing and superfluous after conversions.
If there are tons of users for this API, it'd be needed to convert gradually, and eventually drop the 32bit ops at the end. But in this case, there doesn't seem so many relevant drivers, hence the conversion can be done in a shot as done in your patch 4.
I agree we should do that. Kernel can be 64bit only while we keep maintaining the 32bit ioctls, cant drop that one