Re: [PATCH v2 09/10] Btrfs: add tracepoint for em's EEXIST case

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jan 05, 2018 at 12:51:16PM -0700, Liu Bo wrote:
> This is adding a tracepoint 'btrfs_handle_em_exist' to help debug the
> subtle bugs around merge_extent_mapping.
> 
> Signed-off-by: Liu Bo <bo.li.liu@xxxxxxxxxx>

Nikolay has some concernas about adding the tracepoint, so I'll leave
this patch out of the series for now as we should decide how to proceed.

Thacepoints are considered an ABI by some and not ABI by others. I think
it's a good addition to the debugging aids that also may turn out to be
useful for evaluating performance later.

At minimum we could add some prefix/suffix to the debugging tracepoint
name, so we can let developers add what they need right away.
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Filesystem Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux