[dm-devel] Patchwork and patch flow

Ryo Tsuruta ryov at valinux.co.jp
Thu Feb 5 11:45:08 UTC 2009


Hi Alasdair,

> I'd also like to see more people visibly reviewing and testing patches.
> Please respond in the mailing list threads with comments and Reviewed-by: or
> Tested-by: lines as appropriate.  If you've only reviewed part of the patch
> or only tested a subset of the cases you think need testing, explain in the 
> thread but don't add those lines.  Only offer a Reviewed-by line if you have
> performed a complete line-by-line review of the patch and patch header and
> you support its inclusion in the tree.

To add a Tested-by tag, how much test do we actually need?
We've already done several tests of dm-ioband:
  Proportional I/O bandwidth controlling
  Loading/unloading driver and changing settings while under I/O load
  Bounds checking of setting parameters
  Benchmarks
Do you think these are enough for adding the tag?
It's very difficult for me to determine whether it's qualified or not.
It would be helpful if you could give me some advice.

Thanks,
Ryo Tsuruta




More information about the dm-devel mailing list