drm/msm: Convert shrinker msgs to tracepoints
This reduces the spam in dmesg when we start hitting the shrinker, and replaces it with something we can put on a timeline while profiling or debugging system issues. Signed-off-by: Rob Clark <robdclark@chromium.org> Reviewed-by: Jordan Crouse <jcrouse@codeaurora.org> Reviewed-by: Kristian H. Kristensen <hoegsberg@google.com>
This commit is contained in:
parent
74c0a69cc5
commit
fdf38426cd
@ -6,6 +6,7 @@
|
|||||||
|
|
||||||
#include "msm_drv.h"
|
#include "msm_drv.h"
|
||||||
#include "msm_gem.h"
|
#include "msm_gem.h"
|
||||||
|
#include "msm_gpu_trace.h"
|
||||||
|
|
||||||
static bool msm_gem_shrinker_lock(struct drm_device *dev, bool *unlock)
|
static bool msm_gem_shrinker_lock(struct drm_device *dev, bool *unlock)
|
||||||
{
|
{
|
||||||
@ -87,7 +88,7 @@ msm_gem_shrinker_scan(struct shrinker *shrinker, struct shrink_control *sc)
|
|||||||
mutex_unlock(&dev->struct_mutex);
|
mutex_unlock(&dev->struct_mutex);
|
||||||
|
|
||||||
if (freed > 0)
|
if (freed > 0)
|
||||||
pr_info_ratelimited("Purging %lu bytes\n", freed << PAGE_SHIFT);
|
trace_msm_gem_purge(freed << PAGE_SHIFT);
|
||||||
|
|
||||||
return freed;
|
return freed;
|
||||||
}
|
}
|
||||||
@ -123,7 +124,7 @@ msm_gem_shrinker_vmap(struct notifier_block *nb, unsigned long event, void *ptr)
|
|||||||
*(unsigned long *)ptr += unmapped;
|
*(unsigned long *)ptr += unmapped;
|
||||||
|
|
||||||
if (unmapped > 0)
|
if (unmapped > 0)
|
||||||
pr_info_ratelimited("Purging %u vmaps\n", unmapped);
|
trace_msm_gem_purge_vmaps(unmapped);
|
||||||
|
|
||||||
return NOTIFY_DONE;
|
return NOTIFY_DONE;
|
||||||
}
|
}
|
||||||
|
@ -114,6 +114,32 @@ TRACE_EVENT(msm_gmu_freq_change,
|
|||||||
TP_printk("freq=%u, perf_index=%u", __entry->freq, __entry->perf_index)
|
TP_printk("freq=%u, perf_index=%u", __entry->freq, __entry->perf_index)
|
||||||
);
|
);
|
||||||
|
|
||||||
|
|
||||||
|
TRACE_EVENT(msm_gem_purge,
|
||||||
|
TP_PROTO(u32 bytes),
|
||||||
|
TP_ARGS(bytes),
|
||||||
|
TP_STRUCT__entry(
|
||||||
|
__field(u32, bytes)
|
||||||
|
),
|
||||||
|
TP_fast_assign(
|
||||||
|
__entry->bytes = bytes;
|
||||||
|
),
|
||||||
|
TP_printk("Purging %u bytes", __entry->bytes)
|
||||||
|
);
|
||||||
|
|
||||||
|
|
||||||
|
TRACE_EVENT(msm_gem_purge_vmaps,
|
||||||
|
TP_PROTO(u32 unmapped),
|
||||||
|
TP_ARGS(unmapped),
|
||||||
|
TP_STRUCT__entry(
|
||||||
|
__field(u32, unmapped)
|
||||||
|
),
|
||||||
|
TP_fast_assign(
|
||||||
|
__entry->unmapped = unmapped;
|
||||||
|
),
|
||||||
|
TP_printk("Purging %u vmaps", __entry->unmapped)
|
||||||
|
);
|
||||||
|
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
#undef TRACE_INCLUDE_PATH
|
#undef TRACE_INCLUDE_PATH
|
||||||
|
Loading…
Reference in New Issue
Block a user