On 08/02, Zach Levis wrote: > > Obligatory first-patchset whitespace commit Looks good, perhaps you send it as 1/3 or even separately, so you won't need to redo it if you need to update 1-2. > @@ -63,11 +63,11 @@ static int load_em86(struct linux_binprm *bprm) > */ > remove_arg_zero(bprm); > retval = copy_strings_kernel(1, &bprm->filename, bprm); > - if (retval < 0) return retval; > + if (retval < 0) return retval; And perhaps you can also fix the coding style when your patch touches such a code. Oleg. -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html