search

dorimanx / exfat-nofuse

Android ARM Linux non-fuse read/write kernel driver for exFat and VFat Android file systems
GNU General Public License v2.0
710 stars 326 forks source link

CONFIG_USER_NS #13

Closed ReubenM closed 11 years ago

ReubenM commented 11 years ago

Setting all licensing issues aside for the moment, this module needs work to be very useful. Most notably it does not support namespace changes in the last few kernel versions.

The following options will cause module build to fail: CONFIG_USER_NS=y CONFIG_UIDGID_CONVERTED=y CONFIG_UIDGID_STRICT_TYPE_CHECKS=y

The zfs module is in the midst of fixing the same deficiencies for anybody looking for reference. https://github.com/zfsonlinux/zfs

rxrz commented 11 years ago

Thank you for the hint. It seems like I have added the uidgid support into the driver successfully, but would appreciate any corrections.

Civil commented 11 years ago

I've tried to patch latest samsung exfat opensource driver (1.2.4), it seems that uidgid support is not properly added here. See https://github.com/Civil/exfat/commit/3a87269a0ec0d887e367fe2c2fb9ce915aca78ab for example of how it can be implemented (there are additional 2 patches to make it compilable on older kernels, but I haven't tested them yet).

Civil commented 11 years ago

I've made a pull request with fixes, plus I've tried to port changes that were introduced in 1.2.4.

dorimanx commented 11 years ago

here some more elegant change :) https://github.com/dorimanx/exfat-nofuse/commit/e7c4be209c9da71d1cc3a34476fa6edb5f156942

dorimanx commented 11 years ago

should work now.