On 12/07/2024 18:39, Chaitanya Kulkarni wrote:
Signed-off-by: John Garry<john.g.garry@xxxxxxxxxx>
This patch looks good to me, however formatting of the patch seems
little odd to me, what I meant is section describing the number of flies
changes and lines per file seems to be missing, e.g. (from different
patch) :-
yeah, I often use -p (no diffstat) for generating single patches. The
kernel documentation for submitting patches does not mandate a diffstat
AFAICS, but maybe it's strongly preferred on the xfs mailing list.
"
---
fs/xfs/scrub/trace.h | 10 ++++------
fs/xfs/xfs_trace.h | 10 ++++------
2 files changed, 8 insertions(+), 12 deletions(-)
"
Reviewed-by: Chaitanya Kulkarni<kch@xxxxxxxxxx>
thanks.
