Commit ac2300d4 authored by Damien Lespiau's avatar Damien Lespiau Committed by Daniel Vetter

drm/i915: Sample the frame counter instead of a timestamp for CRCs

Signed-off-by: default avatarDamien Lespiau <damien.lespiau@intel.com>
Signed-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
parent b2c88f5b
...@@ -1748,14 +1748,14 @@ static int i915_pipe_crc(struct seq_file *m, void *data) ...@@ -1748,14 +1748,14 @@ static int i915_pipe_crc(struct seq_file *m, void *data)
return 0; return 0;
} }
seq_puts(m, " timestamp CRC1 CRC2 CRC3 CRC4 CRC5\n"); seq_puts(m, " frame CRC1 CRC2 CRC3 CRC4 CRC5\n");
head = atomic_read(&pipe_crc->head); head = atomic_read(&pipe_crc->head);
tail = atomic_read(&pipe_crc->tail); tail = atomic_read(&pipe_crc->tail);
while (CIRC_CNT(head, tail, INTEL_PIPE_CRC_ENTRIES_NR) >= 1) { while (CIRC_CNT(head, tail, INTEL_PIPE_CRC_ENTRIES_NR) >= 1) {
struct intel_pipe_crc_entry *entry = &pipe_crc->entries[tail]; struct intel_pipe_crc_entry *entry = &pipe_crc->entries[tail];
seq_printf(m, "%12u %8x %8x %8x %8x %8x\n", entry->timestamp, seq_printf(m, "%8u %8x %8x %8x %8x %8x\n", entry->frame,
entry->crc[0], entry->crc[1], entry->crc[2], entry->crc[0], entry->crc[1], entry->crc[2],
entry->crc[3], entry->crc[4]); entry->crc[3], entry->crc[4]);
......
...@@ -1226,7 +1226,7 @@ enum intel_pipe_crc_source { ...@@ -1226,7 +1226,7 @@ enum intel_pipe_crc_source {
}; };
struct intel_pipe_crc_entry { struct intel_pipe_crc_entry {
uint32_t timestamp; uint32_t frame;
uint32_t crc[5]; uint32_t crc[5];
}; };
......
...@@ -1195,8 +1195,7 @@ static void ivb_pipe_crc_update(struct drm_device *dev, enum pipe pipe) ...@@ -1195,8 +1195,7 @@ static void ivb_pipe_crc_update(struct drm_device *dev, enum pipe pipe)
struct drm_i915_private *dev_priv = dev->dev_private; struct drm_i915_private *dev_priv = dev->dev_private;
struct intel_pipe_crc *pipe_crc = &dev_priv->pipe_crc[pipe]; struct intel_pipe_crc *pipe_crc = &dev_priv->pipe_crc[pipe];
struct intel_pipe_crc_entry *entry; struct intel_pipe_crc_entry *entry;
ktime_t now; int head, tail;
int ts, head, tail;
head = atomic_read(&pipe_crc->head); head = atomic_read(&pipe_crc->head);
tail = atomic_read(&pipe_crc->tail); tail = atomic_read(&pipe_crc->tail);
...@@ -1208,10 +1207,7 @@ static void ivb_pipe_crc_update(struct drm_device *dev, enum pipe pipe) ...@@ -1208,10 +1207,7 @@ static void ivb_pipe_crc_update(struct drm_device *dev, enum pipe pipe)
entry = &pipe_crc->entries[head]; entry = &pipe_crc->entries[head];
now = ktime_get(); entry->frame = I915_READ(PIPEFRAME(pipe));
ts = ktime_to_us(now);
entry->timestamp = ts;
entry->crc[0] = I915_READ(PIPE_CRC_RES_1_IVB(pipe)); entry->crc[0] = I915_READ(PIPE_CRC_RES_1_IVB(pipe));
entry->crc[1] = I915_READ(PIPE_CRC_RES_2_IVB(pipe)); entry->crc[1] = I915_READ(PIPE_CRC_RES_2_IVB(pipe));
entry->crc[2] = I915_READ(PIPE_CRC_RES_3_IVB(pipe)); entry->crc[2] = I915_READ(PIPE_CRC_RES_3_IVB(pipe));
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment