Re: BUG: using smp_processor_id() in preemptible [00000000] code:icedove-bin/5449

From: Piotr Hosowicz
Date: Tue May 25 2010 - 06:05:52 EST


On 25.05.2010 12:00, Peter Zijlstra wrote:
On Tue, 2010-05-25 at 11:57 +0200, Piotr Hosowicz wrote:
On 25.05.2010 11:43, Ingo Molnar wrote:

This doesnt fix the whole issue. cpu_clock() is local, while the measurements
done in the blk code are global ...

While the warning is fixed this way, the far more serious issue is still
there: time can go backwards if two points of time measurement are on
different CPUs and can mess up the statistics with negative values, etc...

How serious is this? Can it damage my data? I ask because the machine is
my private computer, not any test machine.

I'm not sure, since I didn't really look what they use the timestamps
for, but a guess would say your data is safe, it might schedule the io
funny, but it should not compromise integrity. At best its used purely
for statistics and not even behaviour is affected.

Thanks.

Cheers,

Piotr Hosowicz

--
"Wiedza to władza. Ale niewiedza, niestety, nie oznacza jeszcze
braku władzy." Niels Bohr
NP: Black Stone Cherry - Reverend Wrinkle
NB: 2.6.34-20100524-1752
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/