<div dir="ltr"><div>Hi All,</div><div><br></div><div>I found two doubtful points in thin-provisioning-tools,</div><div>which might need your help to provide some opinions:</div><div><br></div><div>1. thin_check doesn't open metadata and data space maps.</div><div>   It uses the core space_map to open the mapping_tree,</div><div>   instead of the data space map. Is that legal?</div><div>   (Although block_time_ref_counter was never used during tree traversal)</div><div><br></div><div>2. Class mapping_tree_detail::ll_damage_visitor should not be used to traverse</div><div>   a single_mapping_tree. This class was designed for two-level data mapping tree,</div><div>   or the top-level data mapping tree (i.e., mapping_tree and dev_tree types).</div><div>   It generates missing_devices while visiting a damage with zero-length-path,</div><div>   which is not semantically suitable for single_mapping_tree.</div><div><br></div><div><br></div><div><span style="font-size:14px">Thanks,</span><br></div><div>Ming-Hung Tsai</div></div>