On Fri, Mar 14, 2014 at 09:41:13AM -0500, Jeff McGee wrote: > On Fri, Mar 14, 2014 at 10:27:48AM +0100, Daniel Vetter wrote: > > Big core platforms need some seriuos omph to break a sweat. > > > > This fixes min-max-config-loaded here on my ivb. > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=75146 > > Cc: Jeff McGee <jeff.mcgee@xxxxxxxxx> > > Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxx> > > --- > > tests/pm_rps.c | 13 +++++++++++++ > > 1 file changed, 13 insertions(+) > > > > diff --git a/tests/pm_rps.c b/tests/pm_rps.c > > index fc6bac647f4a..b5dd494443ff 100644 > > --- a/tests/pm_rps.c > > +++ b/tests/pm_rps.c > > @@ -156,6 +156,7 @@ static struct load_helper { > > enum load load; > > bool exit; > > struct igt_helper_process igt_proc; > > + drm_intel_bo *src, *dst; > > } lh; > > > > static void load_helper_signal_handler(int sig) > > @@ -195,6 +196,7 @@ static void emit_store_dword_imm(uint32_t val) > > } > > > > #define LOAD_HELPER_PAUSE_USEC 500 > > +#define LOAD_HELPER_BO_SIZE (16*1024*1024) > > static void load_helper_set_load(enum load load) > > { > > assert(lh.igt_proc.running); > > @@ -226,6 +228,10 @@ static void load_helper_run(enum load load) > > signal(SIGUSR2, load_helper_signal_handler); > > > > while (!lh.exit) { > > + if (lh.load == HIGH) > > + intel_copy_bo(lh.batch, lh.dst, lh.dst, > > + LOAD_HELPER_BO_SIZE); > > + > Did you mean to use lh.src here? > > emit_store_dword_imm(val); > > intel_batchbuffer_flush_on_ring(lh.batch, 0); > > val++; > > @@ -270,6 +276,13 @@ static void load_helper_init(void) > > lh.target_buffer = drm_intel_bo_alloc(lh.bufmgr, "target bo", > > 4096, 4096); > > igt_assert(lh.target_buffer); > > + > > + lh.dst = drm_intel_bo_alloc(lh.bufmgr, "dst bo", > > + LOAD_HELPER_BO_SIZE, 4096); > > + igt_assert(lh.dst); > > + lh.src = drm_intel_bo_alloc(lh.bufmgr, "src bo", > > + LOAD_HELPER_BO_SIZE, 4096); > > + igt_assert(lh.src); > > } > > > > static void load_helper_deinit(void) > Should we free these in load_helper_deinit? I've thrown a fixup patch on top with these two things fixed. Thanks a lot for your feedback. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
