From: Jani Nikula <jani.nikula@intel.com>
To: intel-gfx@lists.freedesktop.org
Cc: jani.nikula@intel.com
Subject: [Intel-gfx] [PATCH 00/16] drm/i915: stop modifying "const" device info
Date: Mon, 20 Jun 2022 11:37:39 +0300 [thread overview]
Message-ID: <cover.1655712106.git.jani.nikula@intel.com> (raw)
Move any device info that gets modified runtime into runtime info,
making device info a const pointer. Finally throw mkwrite_device_info()
into the curb.
The data size increases by sizeof(struct intel_runtime_info) for each
struct intel_device_info in i915_pci.c.
bloat-o-meter gives chg +0.08% in text size, almost all of which is
caused by the last commit (drm/i915: make device info a pointer to
static const data). IMO the first 14-15 commits separating
static/runtime info are worth it even if we decide the text size penalty
is too much to switch to using a pointer for static device info.
BR,
Jani.
Jani Nikula (16):
drm/i915: use GRAPHICS_VER() instead of accessing match_info directly
drm/i915: combine device info printing into one
drm/i915: add initial runtime info into device info
drm/i915: move fbc_mask to runtime info
drm/i915: move page_sizes to runtime info
drm/i915: move ppgtt_type and ppgtt_size to runtime info
drm/i915: move has_pooled_eu to runtime info
drm/i915: move memory_regions to runtime info
drm/i915: move platform_engine_mask to runtime info
drm/i915: move graphics.ver and graphics.rel to runtime info
drm/i915: move pipe_mask and cpu_transcoder_mask to runtime info
drm/i915: move has_hdcp to runtime info
drm/i915: move has_dmc to runtime info
drm/i915: move has_dsc to runtime info
drm/i915: stop resetting display info to zero for no display
drm/i915: make device info a pointer to static const data
drivers/gpu/drm/i915/display/intel_display.h | 4 +-
drivers/gpu/drm/i915/display/intel_fbc.c | 6 +-
drivers/gpu/drm/i915/display/intel_hdcp.c | 4 +-
drivers/gpu/drm/i915/display/intel_vdsc.c | 2 +-
.../drm/i915/display/skl_universal_plane.c | 2 +-
drivers/gpu/drm/i915/gem/i915_gem_pages.c | 2 +-
.../gpu/drm/i915/gem/selftests/huge_pages.c | 18 +-
drivers/gpu/drm/i915/gt/intel_engine_cs.c | 2 +-
drivers/gpu/drm/i915/gt/intel_ppgtt.c | 2 +-
drivers/gpu/drm/i915/gt/intel_sseu.c | 5 +-
drivers/gpu/drm/i915/i915_debugfs.c | 3 +-
drivers/gpu/drm/i915/i915_driver.c | 23 +-
drivers/gpu/drm/i915/i915_drv.h | 35 ++-
drivers/gpu/drm/i915/i915_gem.c | 3 +-
drivers/gpu/drm/i915/i915_gpu_error.c | 3 +-
drivers/gpu/drm/i915/i915_pci.c | 230 +++++++++---------
drivers/gpu/drm/i915/intel_device_info.c | 73 +++---
drivers/gpu/drm/i915/intel_device_info.h | 82 ++++---
drivers/gpu/drm/i915/intel_uncore.c | 2 +-
.../gpu/drm/i915/selftests/mock_gem_device.c | 8 +-
20 files changed, 255 insertions(+), 254 deletions(-)
--
2.30.2
WARNING: multiple messages have this Message-ID (diff)
From: Jani Nikula <jani.nikula@intel.com>
To: intel-gfx@lists.freedesktop.org
Cc: jani.nikula@intel.com
Subject: [Intel-gfx] [PATCH 00/16] drm/i915: stop modifying "const" device info
Date: Mon, 20 Jun 2022 11:37:59 +0300 [thread overview]
Message-ID: <cover.1655712106.git.jani.nikula@intel.com> (raw)
Message-ID: <20220620083759.8UuLeClwmAo6GCrDMCwUJ7Y_oHSwNoZm9HAdaIZu3OI@z> (raw)
Move any device info that gets modified runtime into runtime info,
making device info a const pointer. Finally throw mkwrite_device_info()
into the curb.
The data size increases by sizeof(struct intel_runtime_info) for each
struct intel_device_info in i915_pci.c.
bloat-o-meter gives chg +0.08% in text size, almost all of which is
caused by the last commit (drm/i915: make device info a pointer to
static const data). IMO the first 14-15 commits separating
static/runtime info are worth it even if we decide the text size penalty
is too much to switch to using a pointer for static device info.
BR,
Jani.
Jani Nikula (16):
drm/i915: use GRAPHICS_VER() instead of accessing match_info directly
drm/i915: combine device info printing into one
drm/i915: add initial runtime info into device info
drm/i915: move fbc_mask to runtime info
drm/i915: move page_sizes to runtime info
drm/i915: move ppgtt_type and ppgtt_size to runtime info
drm/i915: move has_pooled_eu to runtime info
drm/i915: move memory_regions to runtime info
drm/i915: move platform_engine_mask to runtime info
drm/i915: move graphics.ver and graphics.rel to runtime info
drm/i915: move pipe_mask and cpu_transcoder_mask to runtime info
drm/i915: move has_hdcp to runtime info
drm/i915: move has_dmc to runtime info
drm/i915: move has_dsc to runtime info
drm/i915: stop resetting display info to zero for no display
drm/i915: make device info a pointer to static const data
drivers/gpu/drm/i915/display/intel_display.h | 4 +-
drivers/gpu/drm/i915/display/intel_fbc.c | 6 +-
drivers/gpu/drm/i915/display/intel_hdcp.c | 4 +-
drivers/gpu/drm/i915/display/intel_vdsc.c | 2 +-
.../drm/i915/display/skl_universal_plane.c | 2 +-
drivers/gpu/drm/i915/gem/i915_gem_pages.c | 2 +-
.../gpu/drm/i915/gem/selftests/huge_pages.c | 18 +-
drivers/gpu/drm/i915/gt/intel_engine_cs.c | 2 +-
drivers/gpu/drm/i915/gt/intel_ppgtt.c | 2 +-
drivers/gpu/drm/i915/gt/intel_sseu.c | 5 +-
drivers/gpu/drm/i915/i915_debugfs.c | 3 +-
drivers/gpu/drm/i915/i915_driver.c | 23 +-
drivers/gpu/drm/i915/i915_drv.h | 35 ++-
drivers/gpu/drm/i915/i915_gem.c | 3 +-
drivers/gpu/drm/i915/i915_gpu_error.c | 3 +-
drivers/gpu/drm/i915/i915_pci.c | 230 +++++++++---------
drivers/gpu/drm/i915/intel_device_info.c | 73 +++---
drivers/gpu/drm/i915/intel_device_info.h | 82 ++++---
drivers/gpu/drm/i915/intel_uncore.c | 2 +-
.../gpu/drm/i915/selftests/mock_gem_device.c | 8 +-
20 files changed, 255 insertions(+), 254 deletions(-)
--
2.30.2
next reply other threads:[~2022-06-20 8:38 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-06-20 8:37 Jani Nikula [this message]
2022-06-20 8:37 ` [Intel-gfx] [PATCH 00/16] drm/i915: stop modifying "const" device info Jani Nikula
2022-06-20 8:37 ` [Intel-gfx] [PATCH 01/16] drm/i915: use GRAPHICS_VER() instead of accessing match_info directly Jani Nikula
2022-06-20 8:38 ` Jani Nikula
2022-06-23 20:00 ` Ville Syrjälä
2022-06-28 14:10 ` Jani Nikula
2022-06-20 8:38 ` [Intel-gfx] [PATCH 02/16] drm/i915: combine device info printing into one Jani Nikula
2022-06-20 8:38 ` [Intel-gfx] [PATCH 03/16] drm/i915: add initial runtime info into device info Jani Nikula
2022-06-23 20:05 ` Ville Syrjälä
2022-06-20 8:38 ` [Intel-gfx] [PATCH 04/16] drm/i915: move fbc_mask to runtime info Jani Nikula
2022-06-20 8:38 ` [Intel-gfx] [PATCH 05/16] drm/i915: move page_sizes " Jani Nikula
2022-06-20 8:38 ` [Intel-gfx] [PATCH 06/16] drm/i915: move ppgtt_type and ppgtt_size " Jani Nikula
2022-06-20 8:49 ` [Intel-gfx] [PATCH 07/16] drm/i915: move has_pooled_eu " Jani Nikula
2022-06-20 8:49 ` [Intel-gfx] [PATCH 08/16] drm/i915: move memory_regions " Jani Nikula
2022-06-20 8:49 ` [Intel-gfx] [PATCH 09/16] drm/i915: move platform_engine_mask " Jani Nikula
2022-06-20 8:49 ` [Intel-gfx] [PATCH 10/16] drm/i915: move graphics.ver and graphics.rel " Jani Nikula
2022-06-20 8:49 ` [Intel-gfx] [PATCH 11/16] drm/i915: move pipe_mask and cpu_transcoder_mask " Jani Nikula
2022-06-20 8:49 ` [Intel-gfx] [PATCH 12/16] drm/i915: move has_hdcp " Jani Nikula
2022-06-20 8:49 ` [Intel-gfx] [PATCH 13/16] drm/i915: move has_dmc " Jani Nikula
2022-06-20 8:49 ` [Intel-gfx] [PATCH 14/16] drm/i915: move has_dsc " Jani Nikula
2022-06-20 8:49 ` [Intel-gfx] [PATCH 15/16] drm/i915: stop resetting display info to zero for no display Jani Nikula
2022-06-20 8:49 ` [Intel-gfx] [PATCH 16/16] drm/i915: make device info a pointer to static const data Jani Nikula
2022-08-18 9:57 ` Maarten Lankhorst
2022-08-18 10:48 ` Jani Nikula
2022-08-18 11:05 ` Maarten Lankhorst
2022-06-20 8:54 ` [Intel-gfx] [PATCH 00/16] drm/i915: stop modifying "const" device info Jani Nikula
2022-06-20 14:16 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for " Patchwork
2022-06-20 14:43 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2022-06-20 19:45 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
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=cover.1655712106.git.jani.nikula@intel.com \
--to=jani.nikula@intel.com \
--cc=intel-gfx@lists.freedesktop.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.