Commit 2386b492 authored by Chris Wilson's avatar Chris Wilson
Browse files

drm/i915: Prefer '%ps' for printing function symbol names 



%pS includes the offset, which is useful for return addresses but noise
when we are pretty printing a known (and expected) function entry point.

Signed-off-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: default avatarTvrtko ursulin <tvrtko.ursulin@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20200319091943.7815-1-chris@chris-wilson.co.uk
parent 394ad36c

drivers/gpu/drm/i915/i915_sw_fence.c

+1 −1
Original line number Diff line number Diff line
@@ -421,7 +421,7 @@ static void timer_i915_sw_fence_wake(struct timer_list *t) 
	if (!fence)

		return;



	pr_notice("Asynchronous wait on fence %s:%s:%llx timed out (hint:%pS)\n",

	pr_notice("Asynchronous wait on fence %s:%s:%llx timed out (hint:%ps)\n",

		  cb->dma->ops->get_driver_name(cb->dma),

		  cb->dma->ops->get_timeline_name(cb->dma),

		  cb->dma->seqno,

drivers/gpu/drm/i915/selftests/i915_active.c

+1 −1
Original line number Diff line number Diff line
@@ -277,7 +277,7 @@ static struct intel_engine_cs *node_to_barrier(struct active_node *it) 


void i915_active_print(struct i915_active *ref, struct drm_printer *m)

{

	drm_printf(m, "active %pS:%pS\n", ref->active, ref->retire);

	drm_printf(m, "active %ps:%ps\n", ref->active, ref->retire);

	drm_printf(m, "\tcount: %d\n", atomic_read(&ref->count));

	drm_printf(m, "\tpreallocated barriers? %s\n",

		   yesno(!llist_empty(&ref->preallocated_barriers)));

drivers/gpu/drm/i915/selftests/i915_request.c

+1 −1
Original line number Diff line number Diff line
@@ -1233,7 +1233,7 @@ static int live_parallel_engines(void *arg) 
		struct igt_live_test t;

		unsigned int idx;



		snprintf(name, sizeof(name), "%pS", fn);

		snprintf(name, sizeof(name), "%ps", fn);

		err = igt_live_test_begin(&t, i915, __func__, name);

		if (err)

			break;

CRA Git | Maintained and supported by SUSTech CRA and CCSE