Re: [PATCH] md5: accepts a new --threads option (original) (raw)

[Top][All Lists]

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

From: Jim Meyering
Subject: Re: [PATCH] md5: accepts a new --threads option
Date: Wed, 21 Oct 2009 15:52:17 +0200

Pádraig Brady wrote: ...

> Odd... that doesn't fail on any of the systems where I tried it: > rawhide, fedora 11, debian unstable.

Yep I noticed it triggered on my linux-2.6.22 glibc-2.6-1 box but not on my F11 laptop. I've setup the test in the attached to output more than 16KiB per process which triggers on F11 at least. ... +(mkdir tmp && cd tmp && seq 500 | xargs touch) + +# This will output at least 16KiB per process +# which is enough to trigger on Fedora 11 at least +(find tmp tmp -type f | xargs -n500 -P2 md5sum) | +sed -n '/[0-9a-f]{32} /!p' | +grep . > /dev/null && fail=1

Thanks. That's better. However, it too sometimes fails to fail: [here, the grep matched nothing 2 of 200 times]

$ for i in $(seq 200); do (find tmp tmp -type f | xargs -n500 -P2 md5sum)
| sed -n '/[0-9a-f]{32} /!p'|grep -q . && t=x || t=.; printf $t; done xxxxxxxxxxxxxxxxxxxxxxxx.xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx.xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx

Adding one more "tmp" made the above spot a failure for 1500 iterations in a row.