Re: [Intel-gfx] [PATCH i-g-t v2] i915/pm_rps: install SIGTERM handler for load_helper process

2019-11-21 Thread Liu, Chuansheng
> -Original Message- > From: Chris Wilson > Sent: Thursday, November 21, 2019 3:47 PM > To: Liu, Chuansheng ; > igt-...@lists.freedesktop.org > Cc: intel-gfx@lists.freedesktop.org > Subject: RE: [Intel-gfx] [PATCH i-g-t v2] i915/pm_rps: install SIGTERM handler >

Re: [Intel-gfx] [PATCH i-g-t v2] i915/pm_rps: install SIGTERM handler for load_helper process

2019-11-20 Thread Chris Wilson
Quoting Liu, Chuansheng (2019-11-21 01:34:24) > Thanks for reviewing the patch, please see below comments. > > > > So here we install the proper handler for signal SIGTERM in the > > > similar way. Without this patch, the GT may keep busy after > > > running this subtest. Enabling rps should be

Re: [Intel-gfx] [PATCH i-g-t v2] i915/pm_rps: install SIGTERM handler for load_helper process

2019-11-20 Thread Liu, Chuansheng
Thanks for reviewing the patch, please see below comments. > > So here we install the proper handler for signal SIGTERM in the > > similar way. Without this patch, the GT may keep busy after > > running this subtest. Enabling rps should be tracked on the > > other side. > > > > Signed-off-by:

Re: [Intel-gfx] [PATCH i-g-t v2] i915/pm_rps: install SIGTERM handler for load_helper process

2019-11-20 Thread Chris Wilson
Quoting Chuansheng Liu (2019-11-20 06:29:12) > Reference: > https://bugs.freedesktop.org/show_bug.cgi?id=112126 > > The issue we hit is the GPU keeps very high load after running > the subtest min-max-config-loaded. > > Some background of the issue: > Currently the rps is not fully enabled yet

[Intel-gfx] [PATCH i-g-t v2] i915/pm_rps: install SIGTERM handler for load_helper process

2019-11-19 Thread Chuansheng Liu
Reference: https://bugs.freedesktop.org/show_bug.cgi?id=112126 The issue we hit is the GPU keeps very high load after running the subtest min-max-config-loaded. Some background of the issue: Currently the rps is not fully enabled yet on TGL, and running the subtest min-max-config-loaded will hit