From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by gabe.freedesktop.org (Postfix) with ESMTPS id 19E9B6E35F for ; Wed, 22 Apr 2020 08:34:50 +0000 (UTC) From: Jani Nikula In-Reply-To: <20200422082818.GD9497@platvala-desk.ger.corp.intel.com> References: <20200421161725.12249-1-juhapekka.heikkila@gmail.com> <20200421161725.12249-3-juhapekka.heikkila@gmail.com> <20200422082818.GD9497@platvala-desk.ger.corp.intel.com> Date: Wed, 22 Apr 2020 11:34:44 +0300 Message-ID: <87a733dil7.fsf@intel.com> MIME-Version: 1.0 Subject: Re: [igt-dev] [PATCH i-g-t 2/8] lib/params: start renaming functions igt_params_* List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" To: Petri Latvala , Juha-Pekka Heikkila Cc: igt-dev@lists.freedesktop.org List-ID: On Wed, 22 Apr 2020, Petri Latvala wrote: > On Tue, Apr 21, 2020 at 07:17:19PM +0300, Juha-Pekka Heikkila wrote: >> Signed-off-by: Jani Nikula >> Signed-off-by: Juha-Pekka Heikkila >> --- >> lib/drmtest.c | 2 +- >> lib/i915/gem_submission.c | 2 +- >> lib/igt_aux.c | 3 +-- >> lib/igt_gt.c | 2 +- >> lib/igt_params.c | 12 +++++------- >> lib/igt_params.h | 5 ++--- >> tests/i915/gem_ctx_exec.c | 2 +- >> tests/i915/gem_ctx_persistence.c | 9 ++++----- >> tests/i915/gem_mmap_gtt.c | 2 +- >> tests/i915/gem_reset_stats.c | 6 ++---- >> tests/i915/sysfs_heartbeat_interval.c | 3 ++- >> tests/i915/sysfs_preempt_timeout.c | 3 ++- >> tests/i915/sysfs_timeslice_duration.c | 3 ++- >> 13 files changed, 25 insertions(+), 29 deletions(-) >> >> diff --git a/lib/drmtest.c b/lib/drmtest.c >> index 1fc39925..17067843 100644 >> --- a/lib/drmtest.c >> +++ b/lib/drmtest.c >> @@ -393,7 +393,7 @@ static void __cancel_work_at_exit(int fd) >> { >> igt_terminate_spins(); /* for older kernels */ >> >> - igt_sysfs_set_parameter(fd, "reset", "%x", -1u /* any method */); >> + igt_params_set(fd, "reset", "%x", -1u /* any method */); > > > Hmm. Now that we're renaming anyway, we have a good chance to consider > if the name is good. > > igt_module_params_set()? No, the point of this series is to make the > params not module-specific. igt_device_params_set()? > > I don't have a strong opinion on this and I can live with > igt_params_set() as well. Just calling out for opinions! Arek? > > As for the patch, whatever the name, straight up renames all around. > Reviewed-by: Petri Latvala The way I envisioned it is that it would look at the debugfs for device params, and if they don't exist, it should fall back to the module params for older kernels. Hence just igt_params_set to avoid being specific about device vs. module. Also, shorter. BR, Jani. -- Jani Nikula, Intel Open Source Graphics Center _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev