From: Jani Nikula <jani.nikula@linux.intel.com>
To: Lucas De Marchi <lucas.demarchi@intel.com>,
intel-gfx@lists.freedesktop.org
Cc: dri-devel@lists.freedesktop.org,
Andi Shyti <andi.shyti@intel.com>,
Chris Wilson <chris@chris-wilson.co.uk>,
Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Subject: Re: [Intel-gfx] [PATCH 1/4] drm/i915: rename debugfs_gt files
Date: Tue, 14 Sep 2021 12:16:13 +0300 [thread overview]
Message-ID: <87o88v34bm.fsf@intel.com> (raw)
In-Reply-To: <20210909004941.379035-1-lucas.demarchi@intel.com>
On Wed, 08 Sep 2021, Lucas De Marchi <lucas.demarchi@intel.com> wrote:
> We shouldn't be using debugfs_ namespace for this functionality. Rename
> debugfs_gt.[ch] to intel_gt_debugfs.[ch] and then make functions,
> defines and structs follow suit.
>
> While at it and since we are renaming the header, sort the includes
> alphabetically.
I didn't do a detailed review, maybe someone should, but superficially
seems good. On the series,
Acked-by: Jani Nikula <jani.nikula@intel.com>
>
> Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
> ---
> drivers/gpu/drm/i915/Makefile | 2 +-
> drivers/gpu/drm/i915/gt/debugfs_engines.c | 6 +++---
> drivers/gpu/drm/i915/gt/debugfs_gt_pm.c | 14 +++++++-------
> drivers/gpu/drm/i915/gt/intel_gt.c | 6 +++---
> .../gt/{debugfs_gt.c => intel_gt_debugfs.c} | 8 ++++----
> .../gt/{debugfs_gt.h => intel_gt_debugfs.h} | 14 +++++++-------
> drivers/gpu/drm/i915/gt/intel_sseu_debugfs.c | 10 +++++-----
> drivers/gpu/drm/i915/gt/uc/intel_guc_debugfs.c | 18 +++++++++---------
> .../gpu/drm/i915/gt/uc/intel_guc_log_debugfs.c | 8 ++++----
> drivers/gpu/drm/i915/gt/uc/intel_huc_debugfs.c | 6 +++---
> drivers/gpu/drm/i915/gt/uc/intel_uc_debugfs.c | 6 +++---
> 11 files changed, 49 insertions(+), 49 deletions(-)
> rename drivers/gpu/drm/i915/gt/{debugfs_gt.c => intel_gt_debugfs.c} (87%)
> rename drivers/gpu/drm/i915/gt/{debugfs_gt.h => intel_gt_debugfs.h} (71%)
>
> diff --git a/drivers/gpu/drm/i915/Makefile b/drivers/gpu/drm/i915/Makefile
> index c36c8a4f0716..3e171f0b5f6a 100644
> --- a/drivers/gpu/drm/i915/Makefile
> +++ b/drivers/gpu/drm/i915/Makefile
> @@ -80,7 +80,6 @@ i915-$(CONFIG_PERF_EVENTS) += i915_pmu.o
> # "Graphics Technology" (aka we talk to the gpu)
> gt-y += \
> gt/debugfs_engines.o \
> - gt/debugfs_gt.o \
> gt/debugfs_gt_pm.o \
> gt/gen2_engine_cs.o \
> gt/gen6_engine_cs.o \
> @@ -101,6 +100,7 @@ gt-y += \
> gt/intel_gt.o \
> gt/intel_gt_buffer_pool.o \
> gt/intel_gt_clock_utils.o \
> + gt/intel_gt_debugfs.o \
> gt/intel_gt_irq.o \
> gt/intel_gt_pm.o \
> gt/intel_gt_pm_irq.o \
> diff --git a/drivers/gpu/drm/i915/gt/debugfs_engines.c b/drivers/gpu/drm/i915/gt/debugfs_engines.c
> index 5e3725e62241..2980dac5b171 100644
> --- a/drivers/gpu/drm/i915/gt/debugfs_engines.c
> +++ b/drivers/gpu/drm/i915/gt/debugfs_engines.c
> @@ -7,9 +7,9 @@
> #include <drm/drm_print.h>
>
> #include "debugfs_engines.h"
> -#include "debugfs_gt.h"
> #include "i915_drv.h" /* for_each_engine! */
> #include "intel_engine.h"
> +#include "intel_gt_debugfs.h"
>
> static int engines_show(struct seq_file *m, void *data)
> {
> @@ -24,11 +24,11 @@ static int engines_show(struct seq_file *m, void *data)
>
> return 0;
> }
> -DEFINE_GT_DEBUGFS_ATTRIBUTE(engines);
> +DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(engines);
>
> void debugfs_engines_register(struct intel_gt *gt, struct dentry *root)
> {
> - static const struct debugfs_gt_file files[] = {
> + static const struct intel_gt_debugfs_file files[] = {
> { "engines", &engines_fops },
> };
>
> diff --git a/drivers/gpu/drm/i915/gt/debugfs_gt_pm.c b/drivers/gpu/drm/i915/gt/debugfs_gt_pm.c
> index f6733f279890..9222cf68c56c 100644
> --- a/drivers/gpu/drm/i915/gt/debugfs_gt_pm.c
> +++ b/drivers/gpu/drm/i915/gt/debugfs_gt_pm.c
> @@ -6,11 +6,11 @@
>
> #include <linux/seq_file.h>
>
> -#include "debugfs_gt.h"
> #include "debugfs_gt_pm.h"
> #include "i915_drv.h"
> #include "intel_gt.h"
> #include "intel_gt_clock_utils.h"
> +#include "intel_gt_debugfs.h"
> #include "intel_gt_pm.h"
> #include "intel_llc.h"
> #include "intel_rc6.h"
> @@ -36,7 +36,7 @@ static int fw_domains_show(struct seq_file *m, void *data)
>
> return 0;
> }
> -DEFINE_GT_DEBUGFS_ATTRIBUTE(fw_domains);
> +DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(fw_domains);
>
> static void print_rc6_res(struct seq_file *m,
> const char *title,
> @@ -238,7 +238,7 @@ static int drpc_show(struct seq_file *m, void *unused)
>
> return err;
> }
> -DEFINE_GT_DEBUGFS_ATTRIBUTE(drpc);
> +DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(drpc);
>
> static int frequency_show(struct seq_file *m, void *unused)
> {
> @@ -480,7 +480,7 @@ static int frequency_show(struct seq_file *m, void *unused)
>
> return 0;
> }
> -DEFINE_GT_DEBUGFS_ATTRIBUTE(frequency);
> +DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(frequency);
>
> static int llc_show(struct seq_file *m, void *data)
> {
> @@ -533,7 +533,7 @@ static bool llc_eval(void *data)
> return HAS_LLC(gt->i915);
> }
>
> -DEFINE_GT_DEBUGFS_ATTRIBUTE(llc);
> +DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(llc);
>
> static const char *rps_power_to_str(unsigned int power)
> {
> @@ -612,11 +612,11 @@ static bool rps_eval(void *data)
> return HAS_RPS(gt->i915);
> }
>
> -DEFINE_GT_DEBUGFS_ATTRIBUTE(rps_boost);
> +DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(rps_boost);
>
> void debugfs_gt_pm_register(struct intel_gt *gt, struct dentry *root)
> {
> - static const struct debugfs_gt_file files[] = {
> + static const struct intel_gt_debugfs_file files[] = {
> { "drpc", &drpc_fops, NULL },
> { "frequency", &frequency_fops, NULL },
> { "forcewake", &fw_domains_fops, NULL },
> diff --git a/drivers/gpu/drm/i915/gt/intel_gt.c b/drivers/gpu/drm/i915/gt/intel_gt.c
> index 2aeaae036a6f..9dda17553e12 100644
> --- a/drivers/gpu/drm/i915/gt/intel_gt.c
> +++ b/drivers/gpu/drm/i915/gt/intel_gt.c
> @@ -3,7 +3,7 @@
> * Copyright © 2019 Intel Corporation
> */
>
> -#include "debugfs_gt.h"
> +#include "intel_gt_debugfs.h"
>
> #include "gem/i915_gem_lmem.h"
> #include "i915_drv.h"
> @@ -15,11 +15,11 @@
> #include "intel_gt_requests.h"
> #include "intel_migrate.h"
> #include "intel_mocs.h"
> +#include "intel_pm.h"
> #include "intel_rc6.h"
> #include "intel_renderstate.h"
> #include "intel_rps.h"
> #include "intel_uncore.h"
> -#include "intel_pm.h"
> #include "shmem_utils.h"
>
> void intel_gt_init_early(struct intel_gt *gt, struct drm_i915_private *i915)
> @@ -434,7 +434,7 @@ void intel_gt_driver_register(struct intel_gt *gt)
> {
> intel_rps_driver_register(>->rps);
>
> - debugfs_gt_register(gt);
> + intel_gt_register_debugfs(gt);
> }
>
> static int intel_gt_init_scratch(struct intel_gt *gt, unsigned int size)
> diff --git a/drivers/gpu/drm/i915/gt/debugfs_gt.c b/drivers/gpu/drm/i915/gt/intel_gt_debugfs.c
> similarity index 87%
> rename from drivers/gpu/drm/i915/gt/debugfs_gt.c
> rename to drivers/gpu/drm/i915/gt/intel_gt_debugfs.c
> index 591eb60785db..a27ba11605d8 100644
> --- a/drivers/gpu/drm/i915/gt/debugfs_gt.c
> +++ b/drivers/gpu/drm/i915/gt/intel_gt_debugfs.c
> @@ -6,13 +6,13 @@
> #include <linux/debugfs.h>
>
> #include "debugfs_engines.h"
> -#include "debugfs_gt.h"
> #include "debugfs_gt_pm.h"
> +#include "i915_drv.h"
> +#include "intel_gt_debugfs.h"
> #include "intel_sseu_debugfs.h"
> #include "uc/intel_uc_debugfs.h"
> -#include "i915_drv.h"
>
> -void debugfs_gt_register(struct intel_gt *gt)
> +void intel_gt_register_debugfs(struct intel_gt *gt)
> {
> struct dentry *root;
>
> @@ -31,7 +31,7 @@ void debugfs_gt_register(struct intel_gt *gt)
> }
>
> void intel_gt_debugfs_register_files(struct dentry *root,
> - const struct debugfs_gt_file *files,
> + const struct intel_gt_debugfs_file *files,
> unsigned long count, void *data)
> {
> while (count--) {
> diff --git a/drivers/gpu/drm/i915/gt/debugfs_gt.h b/drivers/gpu/drm/i915/gt/intel_gt_debugfs.h
> similarity index 71%
> rename from drivers/gpu/drm/i915/gt/debugfs_gt.h
> rename to drivers/gpu/drm/i915/gt/intel_gt_debugfs.h
> index f77540f727e9..cab0f2cb5009 100644
> --- a/drivers/gpu/drm/i915/gt/debugfs_gt.h
> +++ b/drivers/gpu/drm/i915/gt/intel_gt_debugfs.h
> @@ -3,14 +3,14 @@
> * Copyright © 2019 Intel Corporation
> */
>
> -#ifndef DEBUGFS_GT_H
> -#define DEBUGFS_GT_H
> +#ifndef INTEL_GT_DEBUGFS_H
> +#define INTEL_GT_DEBUGFS_H
>
> #include <linux/file.h>
>
> struct intel_gt;
>
> -#define DEFINE_GT_DEBUGFS_ATTRIBUTE(__name) \
> +#define DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(__name) \
> static int __name ## _open(struct inode *inode, struct file *file) \
> { \
> return single_open(file, __name ## _show, inode->i_private); \
> @@ -23,16 +23,16 @@ static const struct file_operations __name ## _fops = { \
> .release = single_release, \
> }
>
> -void debugfs_gt_register(struct intel_gt *gt);
> +void intel_gt_register_debugfs(struct intel_gt *gt);
>
> -struct debugfs_gt_file {
> +struct intel_gt_debugfs_file {
> const char *name;
> const struct file_operations *fops;
> bool (*eval)(void *data);
> };
>
> void intel_gt_debugfs_register_files(struct dentry *root,
> - const struct debugfs_gt_file *files,
> + const struct intel_gt_debugfs_file *files,
> unsigned long count, void *data);
>
> -#endif /* DEBUGFS_GT_H */
> +#endif /* INTEL_GT_DEBUGFS_H */
> diff --git a/drivers/gpu/drm/i915/gt/intel_sseu_debugfs.c b/drivers/gpu/drm/i915/gt/intel_sseu_debugfs.c
> index 1ba8b7da9d37..8bb3a91dad82 100644
> --- a/drivers/gpu/drm/i915/gt/intel_sseu_debugfs.c
> +++ b/drivers/gpu/drm/i915/gt/intel_sseu_debugfs.c
> @@ -4,9 +4,9 @@
> * Copyright © 2020 Intel Corporation
> */
>
> -#include "debugfs_gt.h"
> -#include "intel_sseu_debugfs.h"
> #include "i915_drv.h"
> +#include "intel_gt_debugfs.h"
> +#include "intel_sseu_debugfs.h"
>
> static void sseu_copy_subslices(const struct sseu_dev_info *sseu,
> int slice, u8 *to_mask)
> @@ -282,7 +282,7 @@ static int sseu_status_show(struct seq_file *m, void *unused)
>
> return intel_sseu_status(m, gt);
> }
> -DEFINE_GT_DEBUGFS_ATTRIBUTE(sseu_status);
> +DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(sseu_status);
>
> static int rcs_topology_show(struct seq_file *m, void *unused)
> {
> @@ -293,11 +293,11 @@ static int rcs_topology_show(struct seq_file *m, void *unused)
>
> return 0;
> }
> -DEFINE_GT_DEBUGFS_ATTRIBUTE(rcs_topology);
> +DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(rcs_topology);
>
> void intel_sseu_debugfs_register(struct intel_gt *gt, struct dentry *root)
> {
> - static const struct debugfs_gt_file files[] = {
> + static const struct intel_gt_debugfs_file files[] = {
> { "sseu_status", &sseu_status_fops, NULL },
> { "rcs_topology", &rcs_topology_fops, NULL },
> };
> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_debugfs.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_debugfs.c
> index 887c8c8f35db..25f09a420561 100644
> --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_debugfs.c
> +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_debugfs.c
> @@ -5,14 +5,14 @@
>
> #include <drm/drm_print.h>
>
> -#include "gt/debugfs_gt.h"
> +#include "gt/intel_gt_debugfs.h"
> +#include "gt/uc/intel_guc_ads.h"
> +#include "gt/uc/intel_guc_ct.h"
> +#include "gt/uc/intel_guc_slpc.h"
> +#include "gt/uc/intel_guc_submission.h"
> #include "intel_guc.h"
> #include "intel_guc_debugfs.h"
> #include "intel_guc_log_debugfs.h"
> -#include "gt/uc/intel_guc_ct.h"
> -#include "gt/uc/intel_guc_ads.h"
> -#include "gt/uc/intel_guc_submission.h"
> -#include "gt/uc/intel_guc_slpc.h"
>
> static int guc_info_show(struct seq_file *m, void *data)
> {
> @@ -35,7 +35,7 @@ static int guc_info_show(struct seq_file *m, void *data)
>
> return 0;
> }
> -DEFINE_GT_DEBUGFS_ATTRIBUTE(guc_info);
> +DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(guc_info);
>
> static int guc_registered_contexts_show(struct seq_file *m, void *data)
> {
> @@ -49,7 +49,7 @@ static int guc_registered_contexts_show(struct seq_file *m, void *data)
>
> return 0;
> }
> -DEFINE_GT_DEBUGFS_ATTRIBUTE(guc_registered_contexts);
> +DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(guc_registered_contexts);
>
> static int guc_slpc_info_show(struct seq_file *m, void *unused)
> {
> @@ -62,7 +62,7 @@ static int guc_slpc_info_show(struct seq_file *m, void *unused)
>
> return intel_guc_slpc_print_info(slpc, &p);
> }
> -DEFINE_GT_DEBUGFS_ATTRIBUTE(guc_slpc_info);
> +DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(guc_slpc_info);
>
> static bool intel_eval_slpc_support(void *data)
> {
> @@ -73,7 +73,7 @@ static bool intel_eval_slpc_support(void *data)
>
> void intel_guc_debugfs_register(struct intel_guc *guc, struct dentry *root)
> {
> - static const struct debugfs_gt_file files[] = {
> + static const struct intel_gt_debugfs_file files[] = {
> { "guc_info", &guc_info_fops, NULL },
> { "guc_registered_contexts", &guc_registered_contexts_fops, NULL },
> { "guc_slpc_info", &guc_slpc_info_fops, &intel_eval_slpc_support},
> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_log_debugfs.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_log_debugfs.c
> index 64e0b86bf258..46026c2c1722 100644
> --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_log_debugfs.c
> +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_log_debugfs.c
> @@ -6,7 +6,7 @@
> #include <linux/fs.h>
> #include <drm/drm_print.h>
>
> -#include "gt/debugfs_gt.h"
> +#include "gt/intel_gt_debugfs.h"
> #include "intel_guc.h"
> #include "intel_guc_log.h"
> #include "intel_guc_log_debugfs.h"
> @@ -17,7 +17,7 @@ static int guc_log_dump_show(struct seq_file *m, void *data)
>
> return intel_guc_log_dump(m->private, &p, false);
> }
> -DEFINE_GT_DEBUGFS_ATTRIBUTE(guc_log_dump);
> +DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(guc_log_dump);
>
> static int guc_load_err_log_dump_show(struct seq_file *m, void *data)
> {
> @@ -25,7 +25,7 @@ static int guc_load_err_log_dump_show(struct seq_file *m, void *data)
>
> return intel_guc_log_dump(m->private, &p, true);
> }
> -DEFINE_GT_DEBUGFS_ATTRIBUTE(guc_load_err_log_dump);
> +DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(guc_load_err_log_dump);
>
> static int guc_log_level_get(void *data, u64 *val)
> {
> @@ -109,7 +109,7 @@ static const struct file_operations guc_log_relay_fops = {
> void intel_guc_log_debugfs_register(struct intel_guc_log *log,
> struct dentry *root)
> {
> - static const struct debugfs_gt_file files[] = {
> + static const struct intel_gt_debugfs_file files[] = {
> { "guc_log_dump", &guc_log_dump_fops, NULL },
> { "guc_load_err_log_dump", &guc_load_err_log_dump_fops, NULL },
> { "guc_log_level", &guc_log_level_fops, NULL },
> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_huc_debugfs.c b/drivers/gpu/drm/i915/gt/uc/intel_huc_debugfs.c
> index 5733c15fd123..15998963b863 100644
> --- a/drivers/gpu/drm/i915/gt/uc/intel_huc_debugfs.c
> +++ b/drivers/gpu/drm/i915/gt/uc/intel_huc_debugfs.c
> @@ -5,7 +5,7 @@
>
> #include <drm/drm_print.h>
>
> -#include "gt/debugfs_gt.h"
> +#include "gt/intel_gt_debugfs.h"
> #include "intel_huc.h"
> #include "intel_huc_debugfs.h"
>
> @@ -21,11 +21,11 @@ static int huc_info_show(struct seq_file *m, void *data)
>
> return 0;
> }
> -DEFINE_GT_DEBUGFS_ATTRIBUTE(huc_info);
> +DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(huc_info);
>
> void intel_huc_debugfs_register(struct intel_huc *huc, struct dentry *root)
> {
> - static const struct debugfs_gt_file files[] = {
> + static const struct intel_gt_debugfs_file files[] = {
> { "huc_info", &huc_info_fops, NULL },
> };
>
> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_uc_debugfs.c b/drivers/gpu/drm/i915/gt/uc/intel_uc_debugfs.c
> index 089d98662f46..c2f7924295e7 100644
> --- a/drivers/gpu/drm/i915/gt/uc/intel_uc_debugfs.c
> +++ b/drivers/gpu/drm/i915/gt/uc/intel_uc_debugfs.c
> @@ -6,7 +6,7 @@
> #include <linux/debugfs.h>
> #include <drm/drm_print.h>
>
> -#include "gt/debugfs_gt.h"
> +#include "gt/intel_gt_debugfs.h"
> #include "intel_guc_debugfs.h"
> #include "intel_huc_debugfs.h"
> #include "intel_uc.h"
> @@ -32,11 +32,11 @@ static int uc_usage_show(struct seq_file *m, void *data)
>
> return 0;
> }
> -DEFINE_GT_DEBUGFS_ATTRIBUTE(uc_usage);
> +DEFINE_INTEL_GT_DEBUGFS_ATTRIBUTE(uc_usage);
>
> void intel_uc_debugfs_register(struct intel_uc *uc, struct dentry *gt_root)
> {
> - static const struct debugfs_gt_file files[] = {
> + static const struct intel_gt_debugfs_file files[] = {
> { "usage", &uc_usage_fops, NULL },
> };
> struct dentry *root;
--
Jani Nikula, Intel Open Source Graphics Center
next prev parent reply other threads:[~2021-09-14 9:16 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-09 0:49 [Intel-gfx] [PATCH 1/4] drm/i915: rename debugfs_gt files Lucas De Marchi
2021-09-09 0:49 ` [Intel-gfx] [PATCH 2/4] drm/i915: rename debugfs_engines files Lucas De Marchi
2021-09-09 0:49 ` [Intel-gfx] [PATCH 3/4] drm/i915: rename debugfs_gt_pm files Lucas De Marchi
2021-09-10 17:52 ` Lucas De Marchi
2021-09-10 21:14 ` Yokoyama, Caz
2021-09-10 21:52 ` Lucas De Marchi
2021-09-10 23:53 ` Yokoyama, Caz
2021-09-15 0:24 ` Matt Roper
2021-09-09 0:49 ` [Intel-gfx] [PATCH 4/4] drm/i915: deduplicate frequency dump on debugfs Lucas De Marchi
2021-09-09 1:06 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/4] drm/i915: rename debugfs_gt files Patchwork
2021-09-09 1:36 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2021-09-09 2:50 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
2021-09-09 6:44 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/4] drm/i915: rename debugfs_gt files (rev2) Patchwork
2021-09-09 7:13 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2021-09-09 8:51 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
2021-09-14 9:16 ` Jani Nikula [this message]
2021-09-14 14:46 ` [Intel-gfx] [PATCH 1/4] drm/i915: rename debugfs_gt files Lucas De Marchi
2021-09-14 15:02 ` Jani Nikula
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87o88v34bm.fsf@intel.com \
--to=jani.nikula@linux.intel.com \
--cc=andi.shyti@intel.com \
--cc=chris@chris-wilson.co.uk \
--cc=dri-devel@lists.freedesktop.org \
--cc=intel-gfx@lists.freedesktop.org \
--cc=joonas.lahtinen@linux.intel.com \
--cc=lucas.demarchi@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox