>> jumping to label error from a failure in p9_client_fcreate (e.g. EPERM >> or something perfectly valid) will, with your patch, call clunk with fid >> == NULL and thus generate a stack trace, while it is a perfectly normal >> code path that should only return to userspace with EPERM. > > Actually just seen that this precise example is fixed along with more > similar code paths in subsequents (!) patchs of the set.
Do you refer to my update suggestions with a subject like "One function call less in v9fs_…" here? > It could actually be interesting to see if we could get all such > paths "fixed". Would you like to see any more specific source code clean-up? Which kind of fine-tuning have you got in mind? Regards, Markus -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
