RE: [PATCH] dmtimer posting

Woodruff, Richard
2008-03-21T00:04:34Z

Hi

> Attached is a rev-4 with the cpu_relax() added to while loops as
> suggested by Ladislav, and CTRL register defines moved to the right
> location.

I confirm this version of the patch works on 3430 with latest kernel.

I don't have by break out board for trace to get time stamps right now.
Next week I can get that back or I can pass a kernel off to someone else
like I was doing before. Maybe Kevin can also make a test run on it.

If you just look the instructions trace from ETB alone, the load
function shows up here but not in the older version. Reason for that is
the long stalls are not accounted for just the number of loops, and in
non-posted mode there is nothing to loop on, just wait. When you hook
up ETM, it will correlate with an external time base then you get the
actual time, and its here where you see it stalled out for a long time.

Seems like its good to commit. Maybe someone could make sure a 1710
works.

Regards,
Richard W.