Looks mostly sane (assuming it works on 32-bit userspace on 64-bit kernel now), but: > - context = kmalloc(sizeof(*context), GFP_KERNEL); > + context = kzalloc(sizeof(*context), GFP_KERNEL); Why do you need this? Is this an unrelated change? - R.