On 2/6/13 8:27 AM, Gene Czarcinski wrote: > On 02/05/2013 07:49 PM, Zach Brown wrote: >> Hi gang, >> >> Eric and I went through the warnings that a Coverity scan raised >> against the reasonably recent btrfs-progs that's in Fedora. We >> tried to tackle the lowest hanging fruit: these are the most >> obvious and least risky fixes. >> > If you ran your tests against the btrfs-progs in Fedora 18 then your > tests may not have found problems fixed by the "valgrind" patch on > F18. This patch is not applied to the current David Sterba's > integration branches. The original one was done against F18 with the valgrind patch in place, yes. So it may have missed several things. I can easily do the same analysis against any codebase if/when it's appropriate. > If you have the time, it might be useful (my opinion) to run your > tests against Sterba's integration-02130201 branch ... some of the > problems may be fixed and other not but you also may find some > additional problems. IMO, this branch (or something similar) will be > the basis for a future v0.20 btrfs-progs ... at least that is what I > am using/testing with. There are over 80 commits in this branch over > what is the baseis for the package in Fedora 18. I agree that we need to re-run against all those patches, but I think I will wait until they make it all the way upstream. Until it gets to the point where development and patch integration happens upstream (or at least in an orderly, predictable manner), and we have timely releases of userspace for distro consumption, it's going to be a little hit and miss with tools like this, since there are so many different codebases out there right now, with distros all maintaining their own large patchsets. There are certainly more static analysis issues to fix, but Zach and I thought we'd just see if this group of patches managed to make it upstream before we went a lot further with it. Thanks, -Eric > Gene -- 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
