> +void klp_unpatch_object(struct klp_object *obj, enum klp_func_type ftype) > { > struct klp_func *func; > + bool patched = false; > > - klp_for_each_func(obj, func) > - if (func->patched) > + klp_for_each_func(obj, func) { > + if (!func->patched) > + continue; > + > + if (ftype == KLP_FUNC_ANY || ftype == func->ftype)
You defined klp_is_func_type() exactly for this purpose. Otherwise, it looks good. There is no functional change. Miroslav