On Tue, Feb 18, 2014 at 11:10 AM, Greg KH <gre...@linuxfoundation.org> wrote: > On Mon, Feb 17, 2014 at 01:58:41PM -0800, John Stultz wrote: >> From: Arve Hjønnevåg <a...@android.com> >> >> Add binder_size_t and binder_uintptr_t that is used instead of size_t and >> void __user * in the user-space interface. >> >> Use 64 bit pointers on all systems unless CONFIG_ANDROID_BINDER_IPC_32BIT >> is set (which enables the old protocol on 32 bit systems). >> >> Change BINDER_CURRENT_PROTOCOL_VERSION to 8 if >> CONFIG_ANDROID_BINDER_IPC_32BIT is not set. >> >> Add compat ioctl. >> >> Cc: Colin Cross <ccr...@android.com> >> Cc: Arve Hjønnevåg <a...@android.com> >> Cc: Android Kernel Team <kernel-t...@android.com> >> Signed-off-by: Arve Hjønnevåg <a...@android.com> >> [jstultz: Merged with upstream type changes. Tweaked commit message. >> Various whitespace fixes and longer Kconfig description for checkpatch] >> Signed-off-by: John Stultz <john.stu...@linaro.org> > > Ok, I see that Arve has signed off on this, why the Cc: above to confuse > me? :)
Apologies for the confusion. These are definitely big changes that landed recently. > Anyway, what about the ARM patches that this was based on? And why the > previous patch that breaks things on its own? So, the previous change was making the now-old 32bit binder api consistent on 64bit, and then this change converts binder api to the new 64bit API, which will break existing 32bit binder users. Because of this, they provide a CONFIG_ANDROID_BINDER_IPC_32BIT option which switches back to the old ABI for current userspace users. But yea, I'm fine resubmitting these independently. I'll also work with Serban to see if we can expand the commit message to be more clear. thanks -john -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
