On 27 July 2011 18:08, Akinobu Mita akinobu.mita@gmail.com wrote:
2011/7/27 Per Forlin per.forlin@linaro.org:
There is an issue with patch v4 now when fault_attr is per-host. Without your patch the entry is still created at the root but there are many instances of fault-attr. I think it's better to wait for your patch to make it into the mmc-next tree before submitting my patch. I will prepare a patch v5 that depends on your upcoming changes in fault-inject with a note that states the dependency.
Or you can prepare a patch for -mm and ask Andrew to add it to the -mm tree.
Thanks for the tip,
Would you mind adding "patch 1/3" (export_symbol_gpl) to your patch-set since it depends on the new function names in your patch? If not, I can resend the patch on top of your changes to match the new function names if you prefer to have this patch separate.
I recommend it to be a part of your patchset. The new function name (fault_create_debugfs_attr) is not likely to change for a time. You can add Acked-by: Akinobu Mita akinobu.mita@gmail.com
Beside fault_create_debugfs_attr() the other function is should_fail. I presume this name will be changed too, and start with the fault_?
Regards, Per