The patch itself now looks fine to me.  The log entry is inadequate.
It needs to mention each file and function touched and say what was
done there.  "Update caller." is sufficient text for a function to
which the only change was trivial argument reordering or whatnot.  But
in most of these cases, each caller dropped a stat* call as well and
so it needs to say "Pass it -1 instead of calling stat64." or suchlike.


Thanks,
Roland

Reply via email to