== Series Details ==
Series: drm/i915/mtl: Disable MC6 for MTL A step (rev2)
URL : https://patchwork.freedesktop.org/series/114833/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_12835 -> Patchwork_114833v2
Summary
---
The Wa_14017073508 require to send Media Busy/Idle mailbox while
accessing Media tile. As of now it is getting handled while __gt_unpark,
__gt_park. But there are various corner cases where forcewakes are taken
without __gt_unpark i.e. without sending Busy Mailbox especially during
register reads.
Disabling MC6 for A step as it is not feasible to extend
Wa_14017073508 in forcewake path to cover all corner cases.
Reverting the commit 8f70f1ec587da.
v2 - Combined patch1 and patch2 from rev1
Badal Nilawar (1):
drm/i915/mtl: Disable MC6 for MTL A step
drivers/gpu/drm/i915/gt/intel_gt
Sure Andi. Will be more cautious.
Radhakrishna Sripada
> -Original Message-
> From: Andi Shyti
> Sent: Thursday, March 9, 2023 3:09 PM
> To: Sripada, Radhakrishna
> Cc: Andi Shyti ; intel-gfx@lists.freedesktop.org;
> De
> Marchi, Lucas ; Roper, Matthew D
>
> Subject: Re: [Intel-gfx] [
> From: Tian, Kevin
> Sent: Friday, March 10, 2023 1:31 PM
>
> > From: Yi Liu
> > Sent: Wednesday, March 8, 2023 9:29 PM
> >
> > This is another method to issue PCI hot reset for the users that bounds
> > device to a positive iommufd value. In such case, iommufd is a proof of
> > device ownership
> -Original Message-
> From: Intel-gfx On Behalf Of Mark
> Yacoub
> Sent: Thursday, January 19, 2023 1:00 AM
> To: quic_khs...@quicinc.com; linux-arm-...@vger.kernel.org; dri-
> de...@lists.freedesktop.org; freedr...@lists.freedesktop.org;
> devicet...@vger.kernel.org; linux-ker...@vger
> From: Alexey Kardashevskiy
> Sent: Friday, March 10, 2023 10:39 AM
>
> On 27/2/23 22:11, Yi Liu wrote:
> > This adds ioctl for userspace to bind device cdev fd to iommufd.
> >
> > VFIO_DEVICE_BIND_IOMMUFD: bind device to an iommufd, hence gain
> DMA
> > control pr
> > > -static int guc_log_relay_open(struct inode *inode, struct file *file)
> > > +static int guc_log_relay_ctl_open(struct inode *inode, struct file *file)
> >
> > Again not objecting, but what is the purpose/thinking behind adding _ctl_
> > to these function names? The previous names seemed fin
> From: Tian, Kevin
> Sent: Friday, March 10, 2023 1:08 PM
>
> > From: Liu, Yi L
> > Sent: Wednesday, March 8, 2023 9:29 PM
> >
> > @@ -1319,8 +1319,14 @@ static int vfio_pci_ioctl_pci_hot_reset(struct
> > vfio_pci_core_device *vdev,
> > break;
> > }
> >
> > -
> From: Yi Liu
> Sent: Wednesday, March 8, 2023 9:29 PM
>
> This is another method to issue PCI hot reset for the users that bounds
> device to a positive iommufd value. In such case, iommufd is a proof of
> device ownership. By passing a zero-length fd array, user indicates kernel
> to do ownersh
>
> From: Sean Paul
>
> This patch moves the hdcp atomic check from i915 to drm_hdcp so other
> drivers can use it. No functional changes, just cleaned up some of the code
> when moving it over.
>
> Acked-by: Jani Nikula
> Acked-by: Jani Nikula
> Reviewed-by: Rodrigo Vivi
> Reviewed-by: Abhi
> From: Liu, Yi L
> Sent: Wednesday, March 8, 2023 9:29 PM
>
> @@ -1319,8 +1319,14 @@ static int vfio_pci_ioctl_pci_hot_reset(struct
> vfio_pci_core_device *vdev,
> break;
> }
>
> - /* Ensure the FD is a vfio group FD.*/
> - if (!vfio_f
> From: Liu, Yi L
> Sent: Wednesday, March 8, 2023 9:29 PM
>
> No function change is intended, just to make the helpers and structures
> to be prepared to accept device fds as proof of device ownership.
>
> Signed-off-by: Yi Liu
> ---
> drivers/vfio/pci/vfio_pci_core.c | 40 ---
> From: Liu, Yi L
> Sent: Wednesday, March 8, 2023 9:29 PM
>
> @@ -2429,10 +2429,23 @@ static int vfio_pci_dev_set_hot_reset(struct
> vfio_device_set *dev_set,
>
> list_for_each_entry(cur_vma, &dev_set->device_list,
> vdev.dev_set_list) {
> /*
> - * Test whether a
> From: Liu, Yi L
> Sent: Wednesday, March 8, 2023 9:29 PM
>
> @@ -1114,6 +1114,10 @@ static long vfio_device_fops_unl_ioctl(struct file
> *filep,
> struct vfio_device *device = df->device;
> int ret;
>
> + /* Paired with smp_store_release() in vfio_device_group_open() */
> +
== Series Details ==
Series: drm/i915: Ensure DSC has enough BW and stays within HW limits (rev3)
URL : https://patchwork.freedesktop.org/series/114679/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_12827_full -> Patchwork_114679v3_full
== Series Details ==
Series: drm/i915/pmu: Use common freq functions with sysfs
URL : https://patchwork.freedesktop.org/series/114942/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_12834 -> Patchwork_114942v1
Summary
--
> From: Liu, Yi L
> Sent: Wednesday, March 8, 2023 9:14 PM
>
> After making the no-DMA drivers (samples/vfio-mdev) providing iommufd
> callbacks, __vfio_register_dev() should check the presence of the iommufd
> callbacks if CONFIG_IOMMUFD is enabled.
>
> Signed-off-by: Yi Liu
> ---
> drivers/v
> From: Liu, Yi L
> Sent: Wednesday, March 8, 2023 9:14 PM
>
> @@ -32,12 +32,8 @@ int vfio_iommufd_bind(struct vfio_device *vdev,
> struct iommufd_ctx *ictx)
> return 0;
> }
>
> - /*
> - * If the driver doesn't provide this op then it means the device does
> - *
> From: Liu, Yi L
> Sent: Wednesday, March 8, 2023 9:14 PM
>
> vfio device cdev needs to return iommufd_access ID to userspace if
> bind_iommufd succeeds.
>
> Signed-off-by: Yi Liu
Reviewed-by: Kevin Tian
> From: Liu, Yi L
> Sent: Wednesday, March 8, 2023 9:14 PM
>
> @@ -449,33 +450,18 @@ iommufd_access_create(struct iommufd_ctx *ictx,
> u32 ioas_id,
> access->data = data;
> access->ops = ops;
>
> - obj = iommufd_get_object(ictx, ioas_id, IOMMUFD_OBJ_IOAS);
> - if (IS_ERR(obj))
On Thu, 09 Mar 2023 01:20:09 -0800, Tvrtko Ursulin wrote:
>
Hi Tvrtko,
> On 09/03/2023 03:46, Ashutosh Dixit wrote:
> > Expose intel_rps_read_actual_frequency_fw to read the actual freq without
> > taking forcewake for use by PMU. The code is refactored to use a common set
> > of functions across
The fallback to requested freq does not work for SLPC because SLPC does not
use 'struct intel_rps'. Also for SLPC requested freq can only be obtained
from a hw register after acquiring forcewake which we don't want to do for
PMU. Therefore remove fallback to requested freq for SLPC. The actual freq
Expose intel_rps_read_actual_frequency_fw to read the actual freq without
taking forcewake for use by PMU. The code is refactored to use a common set
of functions across sysfs and PMU. Using common functions with sysfs in PMU
solves the issues of missing support for MTL and missing support for olde
Expose intel_rps_read_actual_frequency_fw to read the actual freq without
taking forcewake for use by PMU. The code is refactored to use a common set
of functions across sysfs and PMU. Using common functions with sysfs in PMU
solves the issues of missing support for MTL and missing support for olde
On 3/9/2023 9:32 AM, Patchwork wrote:
Project List - Patchwork *Patch Details*
*Series:* Fix a couple of issues with the GSC worker timing (rev2)
*URL:* https://patchwork.freedesktop.org/series/114306/
*State:*failure
*Details:*
https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork
On Thu, 09 Mar 2023 14:34:04 -0800, Umesh Nerlige Ramappa wrote:
>
Hi Umesh,
> On Wed, Mar 08, 2023 at 10:08:13AM -0800, Dixit, Ashutosh wrote:
> > On Tue, 07 Mar 2023 12:16:10 -0800, Umesh Nerlige Ramappa wrote:
> >>
> >
> >> One or more engines map to a specific OA unit. All reports from these
On Tue, 07 Mar 2023 12:16:11 -0800, Umesh Nerlige Ramappa wrote:
>
Hi Umesh,
> -static int gen8_configure_context(struct i915_gem_context *ctx,
> +static int gen8_configure_context(struct i915_perf_stream *stream,
> + struct i915_gem_context *ctx,
>
On Wed, 2022-12-07 at 09:24 -0800, Dixit, Ashutosh wrote:
> On Tue, 06 Dec 2022 01:21:00 -0800, Alan Previn wrote:
> >
> >
alan: snip
> > intel_gt_debugfs_register_files(root, files, ARRAY_SIZE(files), uc);
> >
> > intel_guc_debugfs_register(&uc->guc, root);
>
> After moving the line a
Again, only now having time to relook at this.
Will re-rev and reconnect with Ashutosh offline since I've been absent on this
for too long.
Thanks again Ashutosh for reviewing all these back in May.
On Wed, 2022-12-07 at 08:50 -0800, Dixit, Ashutosh wrote:
> On Tue, 06 Dec 2022 01:20:59 -0800, Al
Finally got some time to relook at this series. Responses.
I'll re-rev this and re-connect with Ashutosh offline considering how long i've
been silent on this.
On Wed, 2022-12-07 at 08:43 -0800, Dixit, Ashutosh wrote:
> On Tue, 06 Dec 2022 01:20:58 -0800, Alan Previn wrote:
> >
> > diff --git a
Hi Radhakrishna,
> Since most of the comments aligned with Matt's suggestion pushed with Matt's
> r-b.
OK, but next time, please hold on a bit as I might also have had
disagreements on your answers or I want to see it tested again
with the new changes.
It's not the case as I would have r-b it a
== Series Details ==
Series: series starting with [CI,1/3] drm/i915/opregion: Fix opregion setup
during system resume on platforms without display (rev3)
URL : https://patchwork.freedesktop.org/series/114866/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_12832 -> Patchwork_11
On Tue, Mar 07, 2023 at 05:45:48PM -0800, Dixit, Ashutosh wrote:
On Tue, 07 Mar 2023 12:16:10 -0800, Umesh Nerlige Ramappa wrote:
Hi Umesh,
+ /* Defaults when class:instance is not passed */
+ class = I915_ENGINE_CLASS_RENDER;
+ instance = 0;
+
for (i = 0; i < n_pr
On Wed, Mar 08, 2023 at 10:08:13AM -0800, Dixit, Ashutosh wrote:
On Tue, 07 Mar 2023 12:16:10 -0800, Umesh Nerlige Ramappa wrote:
Hi Umesh,
One or more engines map to a specific OA unit. All reports from these
engines are captured in the OA buffer managed by this OA unit.
Current i915 OA i
Hi Nirmoy,
On Thu, Mar 09, 2023 at 05:49:05PM +0100, Nirmoy Das wrote:
> Stolen memory is not usable for MTL A0 stepping beyond
> certain access size and we have no control over userspace
> access size of /dev/fb which can be backed by stolen memory.
> So disable stolen memory backed fb by setting
Hi Nirmoy,
On Thu, Mar 09, 2023 at 05:58:52PM +0100, Nirmoy Das wrote:
> Change the function doc to reflect updated name.
>
> Cc: Andi Shyti
> Signed-off-by: Nirmoy Das
> ---
> drivers/gpu/drm/i915/gt/intel_engine_cs.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/d
== Series Details ==
Series: series starting with [CI,1/3] drm/i915/opregion: Fix opregion setup
during system resume on platforms without display (rev3)
URL : https://patchwork.freedesktop.org/series/114866/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
Fast m
On Thu, Mar 09, 2023 at 12:03:19AM +0200, Ville Syrjälä wrote:
> On Wed, Mar 08, 2023 at 11:58:58AM -0500, Rodrigo Vivi wrote:
> > uncore->lock only protects the forcewake domain itself,
> > not the register accesses.
> >
> > uncore's _fw alternatives are for cases where the domains
> > are not ne
== Series Details ==
Series: series starting with [1/2] drm/i915/display: Restore dsparb_lock. (rev2)
URL : https://patchwork.freedesktop.org/series/114868/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_12832 -> Patchwork_114868v2
==
== Series Details ==
Series: drm/i915/gt: prevent forcewake releases during BAR resize (rev5)
URL : https://patchwork.freedesktop.org/series/114836/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_12832 -> Patchwork_114836v5
On Thu, 09 Mar 2023 11:48:41 -0800, Vinay Belgaumkar wrote:
>
> The entry for gt_idle_freq_mhz does not exist.
>
> Fixes: d1ea62b3f759 (igt: Add gem_ctx_freq to exercise requesting freq via
> sysfs)
Reviewed-by: Ashutosh Dixit
>
> Cc: Umesh Nerlige Ramappa
> Signed-off-by: Vinay Belgaumkar
>
The entry for gt_idle_freq_mhz does not exist.
Fixes: d1ea62b3f759 (igt: Add gem_ctx_freq to exercise requesting freq via
sysfs)
Cc: Umesh Nerlige Ramappa
Signed-off-by: Vinay Belgaumkar
---
tests/i915/gem_ctx_freq.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/tests
== Series Details ==
Series: drm/i915/gt: Update engine_init_common documentation
URL : https://patchwork.freedesktop.org/series/114926/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_12832 -> Patchwork_114926v1
Summary
On Thu, Mar 09, 2023 at 03:41:35PM -, Patchwork wrote:
> == Series Details ==
>
> Series: drm/i915/dmc: Load DMC on MTL (rev2)
> URL : https://patchwork.freedesktop.org/series/114576/
> State : success
>
> == Summary ==
>
> CI Bug Log - changes from CI_DRM_12827_full -> Patchwork_114576v2_
On 3/9/2023 6:33 PM, Matt Roper wrote:
On Thu, Mar 09, 2023 at 05:58:52PM +0100, Nirmoy Das wrote:
Change the function doc to reflect updated name.
Cc: Andi Shyti
Signed-off-by: Nirmoy Das
---
drivers/gpu/drm/i915/gt/intel_engine_cs.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
== Series Details ==
Series: drm/i915/pmu: Use common freq functions with sysfs
URL : https://patchwork.freedesktop.org/series/114814/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_12827_full -> Patchwork_114814v1_full
Summ
Hi Andi,
> -Original Message-
> From: Andi Shyti
> Sent: Thursday, March 9, 2023 8:30 AM
> To: Sripada, Radhakrishna
> Cc: intel-gfx@lists.freedesktop.org; De Marchi, Lucas
>
> Subject: Re: [Intel-gfx] [PATCH v3 1/5] drm/i915/mtl: Fix Wa_16015201720
> implementation
>
> Hi Radhakrishna
Thank you for the Review. Pushed.
-Radhakrishna Sripada
> -Original Message-
> From: Andi Shyti
> Sent: Thursday, March 9, 2023 9:18 AM
> To: Sripada, Radhakrishna
> Cc: intel-gfx@lists.freedesktop.org; Auld, Matthew ;
> Wilson, Chris P
> Subject: Re: [Intel-gfx] [PATCH v3 4/5] drm/i91
Thank you for the Review. Pushed.
-Radhakrishna Sripada
> -Original Message-
> From: Andi Shyti
> Sent: Thursday, March 9, 2023 9:29 AM
> To: Sripada, Radhakrishna
> Cc: intel-gfx@lists.freedesktop.org; Roper, Matthew D
>
> Subject: Re: [Intel-gfx] [PATCH v3 5/5] drm/i915/display/mtl:
== Series Details ==
Series: drm/i915: disable sampler indirect state in bindless heap
URL : https://patchwork.freedesktop.org/series/114920/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_12831 -> Patchwork_114920v1
Summary
On Thu, Mar 09, 2023 at 05:58:52PM +0100, Nirmoy Das wrote:
> Change the function doc to reflect updated name.
>
> Cc: Andi Shyti
> Signed-off-by: Nirmoy Das
> ---
> drivers/gpu/drm/i915/gt/intel_engine_cs.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/d
== Series Details ==
Series: Fix a couple of issues with the GSC worker timing (rev2)
URL : https://patchwork.freedesktop.org/series/114306/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_12827_full -> Patchwork_114306v2_full
Hi,
On Wed, Mar 01, 2023 at 12:10:53PM -0800, Radhakrishna Sripada wrote:
> From: José Roberto de Souza
>
> Latch reset of phys during DC9 and when driver is unloaded to avoid
> phy reset.
>
> Specification ask us to program it closer to the step that enables
> DC9 in DC_STATE_EN but doing this
Hi Radhakrishna,
On Wed, Mar 01, 2023 at 12:10:52PM -0800, Radhakrishna Sripada wrote:
> From: Tejas Upadhyay
>
> lock the fbdev obj before calling into
> i915_vma_pin_iomap(). This helps to solve below :
>
> <7>[ 93.563308] i915 :00:02.0: [drm:intelfb_create [i915]] no BIOS fb,
> alloca
== Series Details ==
Series: drm/i915: disable sampler indirect state in bindless heap
URL : https://patchwork.freedesktop.org/series/114920/
State : warning
== Summary ==
Error: dim checkpatch failed
e4889ef6bb9b drm/i915: disable sampler indirect state in bindless heap
-:14: WARNING:TYPO_SPE
Change the function doc to reflect updated name.
Cc: Andi Shyti
Signed-off-by: Nirmoy Das
---
drivers/gpu/drm/i915/gt/intel_engine_cs.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/gt/intel_engine_cs.c
b/drivers/gpu/drm/i915/gt/intel_engine_cs.c
inde
Stolen memory is not usable for MTL A0 stepping beyond
certain access size and we have no control over userspace
access size of /dev/fb which can be backed by stolen memory.
So disable stolen memory backed fb by setting i915->dsm.usable_size
to zero.
Cc: Matthew Auld
Cc: Andi Shyti
Cc: Daniele C
On Tue, Feb 28, 2023 at 01:18:55PM -0800, Dixit, Ashutosh wrote:
> On Fri, 12 Aug 2022 11:06:58 -0700, Guenter Roeck wrote:
> >
>
> Hi Guenter/linux-hwmon,
>
>
> > On 8/12/22 10:37, Badal Nilawar wrote:
> > > From: Dale B Stimson
> > >
> > > Use i915 HWMON to display/modify dGfx power PL1 limit
Hi Radhakrishna,
On Wed, Mar 01, 2023 at 12:10:49PM -0800, Radhakrishna Sripada wrote:
> The commit 2357f2b271ad ("drm/i915/mtl: Initial display workarounds")
> extended the workaround Wa_16015201720 to MTL. However the registers
> that the original WA implemented moved for MTL.
>
> Implement the
== Series Details ==
Series: drm/i915: use ref_tracker library for tracking wakerefs (rev5)
URL : https://patchwork.freedesktop.org/series/100327/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_12831 -> Patchwork_100327v5
Su
== Series Details ==
Series: drm/i915/dmc: Load DMC on MTL (rev2)
URL : https://patchwork.freedesktop.org/series/114576/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_12827_full -> Patchwork_114576v2_full
Summary
---
== Series Details ==
Series: drm/i915: use ref_tracker library for tracking wakerefs (rev5)
URL : https://patchwork.freedesktop.org/series/100327/
State : warning
== Summary ==
Error: dim checkpatch failed
21d94d8537c5 lib/ref_tracker: add unlocked leak print helper
-:6: WARNING:COMMIT_LOG_LON
By default the indirect state sampler data (border colors) are stored
in the same heap as the SAMPLER_STATE structure. For userspace drivers
that can be 2 different heaps (dynamic state heap & bindless sampler
state heap). This means that border colors have to copied in 2
different places so that t
On Thu, 2 Mar 2023 at 00:00, Rodrigo Vivi wrote:
> On Fri, Feb 17, 2023 at 09:51:37PM +0100, Daniel Vetter wrote:
> > Hi all,
> >
> > [I thought I've sent this out earlier this week, but alas got stuck, kinda
> > bad timing now since I'm out next week but oh well]
> >
> > So xe is a quite substant
On Thu, 09 Mar 2023, Dan Carpenter wrote:
> Hello Jani Nikula,
>
> This is a semi-automatic email about new static checker warnings.
>
> The patch 1b28c1c789d0: "drm/i915/dmc: add i915_to_dmc() and
> dmc->i915 and use them" from Mar 1, 2023, leads to the following
> Smatch complaint:
>
> dri
Hello Jani Nikula,
This is a semi-automatic email about new static checker warnings.
The patch 1b28c1c789d0: "drm/i915/dmc: add i915_to_dmc() and
dmc->i915 and use them" from Mar 1, 2023, leads to the following
Smatch complaint:
drivers/gpu/drm/i915/display/intel_dmc.c:1162
intel_dmc_debu
On Thu, Mar 09, 2023 at 11:08:09AM +0530, Nilawar, Badal wrote:
>
>
> On 08-03-2023 20:55, Nilawar, Badal wrote:
> > Hi Jani,
> >
> > On 08-03-2023 16:48, Jani Nikula wrote:
> > > On Wed, 08 Mar 2023, Badal Nilawar wrote:
> > > > This reverts commit 8f70f1ec587da0b0d52d768fd8c3defbc5e5b55c.
> >
When swapping in, or under memory pressure ttm_tt_populate() may sleep
for a substantiable amount of time. Allow interrupts during the sleep.
This will also allow us to inject -EINTR errors during swapin in upcoming
patches.
Also avoid returning VM_FAULT_OOM, since that will confuse the core
mm, m
When swapping out, we will split multi-order pages both in order to
move them to the swap-cache and to be able to return memory to the
swap cache as soon as possible on a page-by-page basis.
Reduce the page max order to the system PMD size, as we can then be nicer
to the system and avoid splitting
When hitting an error, the error path forgot to unmap dma mappings and
could call set_pages_wb() on already uncached pages.
Fix this by introducing a common ttm_pool_free_range() function that
does the right thing.
v2:
- Simplify that common function (Christian König)
v3:
- Rename that common fun
I collected the, from my POW, uncontroversial patches from V1 of the TTM
shrinker series, some corrected after the initial patch submission, one
patch added from the Xe RFC ("drm/ttm: Don't print error message if
eviction was interrupted"). It would be nice to have these reviewed and
merged while r
== Series Details ==
Series: drm/ttm: let struct ttm_device_funcs be placed in rodata
URL : https://patchwork.freedesktop.org/series/114907/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_12829 -> Patchwork_114907v1
Summary
On Mon, Feb 27, 2023 at 01:20:30PM +0200, Stanislav Lisovskiy wrote:
> According to spec, we should check if output_bpp * pixel_rate is less
> than DDI clock * 72, if UHBR is used.
>
> v2: - s/pipe_config/crtc_state/ (Jani Nikula)
> - Merged previous patch into that one, to remove empty functi
== Series Details ==
Series: drm/ttm: let struct ttm_device_funcs be placed in rodata
URL : https://patchwork.freedesktop.org/series/114907/
State : warning
== Summary ==
Error: dim checkpatch failed
dd44fba80630 drm/ttm: let struct ttm_device_funcs be placed in rodata
-:6: WARNING:COMMIT_LOG_
On Thu, 09 Mar 2023, Christian König wrote:
> Am 09.03.23 um 13:37 schrieb Jani Nikula:
>> Make the struct ttm_device_funcs pointers const so the data can be placed in
>> rodata.
>>
>> Cc: Christian Koenig
>> Cc: Huang Rui
>> Signed-off-by: Jani Nikula
>
> Good idea, Reviewed-by: Christian Kön
On Thu, 09 Mar 2023, Andrzej Hajda wrote:
> On 09.03.2023 09:16, Jani Nikula wrote:
>> There's no need for any of these to be mutable, constify:
>>
>> drivers/gpu/drm/i915/gt/sysfs_engines.o: .data 0020 files.0
>> drivers/gpu/drm/i915/gt/sysfs_engines.o: .data
Am 09.03.23 um 13:37 schrieb Jani Nikula:
Make the struct ttm_device_funcs pointers const so the data can be placed in
rodata.
Cc: Christian Koenig
Cc: Huang Rui
Signed-off-by: Jani Nikula
Good idea, Reviewed-by: Christian König
Should I push it to drm-misc-next or do you need it on some
Make the struct ttm_device_funcs pointers const so the data can be placed in
rodata.
Cc: Christian Koenig
Cc: Huang Rui
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/ttm/ttm_device.c | 2 +-
include/drm/ttm/ttm_device.h | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --g
On Thu, Mar 09, 2023 at 12:09:55PM +0100, Maarten Lankhorst wrote:
>
> On 2023-03-09 12:04, Hogander, Jouni wrote:
> > On Mon, 2023-03-06 at 22:58 +0200, Ville Syrjälä wrote:
> >> On Mon, Mar 06, 2023 at 09:23:50PM +0100, Maarten Lankhorst wrote:
> >>> Hey,
> >>>
> >>> On 2023-03-06 16:23, Souza,
Thanks Stan for the review.
On 3/9/2023 4:14 PM, Lisovskiy, Stanislav wrote:
On Thu, Feb 23, 2023 at 05:25:09PM +0530, Ankit Nautiyal wrote:
While computing compressed bpp, maximum value of bits_per_pixel is
calculated that can be supported with the given link configuration
for a given mode. Av
== Series Details ==
Series: drm/ttm: Small fixes / cleanups in prep for shrinking
URL : https://patchwork.freedesktop.org/series/114774/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_12824_full -> Patchwork_114774v1_full
S
== Series Details ==
Series: drm/i915/gt: make kobj attributes const (rev2)
URL : https://patchwork.freedesktop.org/series/114898/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_12829 -> Patchwork_114898v2
Summary
---
On 2023-03-09 12:04, Hogander, Jouni wrote:
On Mon, 2023-03-06 at 22:58 +0200, Ville Syrjälä wrote:
On Mon, Mar 06, 2023 at 09:23:50PM +0100, Maarten Lankhorst wrote:
Hey,
On 2023-03-06 16:23, Souza, Jose wrote:
On Mon, 2023-03-06 at 15:16 +0100, Maarten Lankhorst wrote:
As a fallback if w
On 3/9/2023 9:16 AM, Jani Nikula wrote:
There's no need for any of these to be mutable, constify:
drivers/gpu/drm/i915/gt/sysfs_engines.o: .data 0020 files.0
drivers/gpu/drm/i915/gt/sysfs_engines.o: .data 0050 files.1
drivers/gpu/drm/i915/gt/sysfs_engines.o: .data 00
On Mon, 2023-03-06 at 22:58 +0200, Ville Syrjälä wrote:
> On Mon, Mar 06, 2023 at 09:23:50PM +0100, Maarten Lankhorst wrote:
> > Hey,
> >
> > On 2023-03-06 16:23, Souza, Jose wrote:
> > > On Mon, 2023-03-06 at 15:16 +0100, Maarten Lankhorst wrote:
> > > > As a fallback if we decide not to merge th
On 09.03.2023 09:16, Jani Nikula wrote:
There's no need for any of these to be mutable, constify:
drivers/gpu/drm/i915/gt/sysfs_engines.o: .data 0020 files.0
drivers/gpu/drm/i915/gt/sysfs_engines.o: .data 0050 files.1
drivers/gpu/drm/i915/gt/sysfs_engines.o: .data
On Thu, Feb 23, 2023 at 05:25:09PM +0530, Ankit Nautiyal wrote:
> While computing compressed bpp, maximum value of bits_per_pixel is
> calculated that can be supported with the given link configuration
> for a given mode. Avoid rounding up of this max bits_per_pixel.
> Also improve documentation fo
On Wed, 8 Mar 2023 07:52:51 -0800
Rob Clark wrote:
> From: Rob Clark
>
> This series adds a deadline hint to fences, so realtime deadlines
> such as vblank can be communicated to the fence signaller for power/
> frequency management decisions.
>
> This is partially inspired by a trick i915 do
On Thu, Mar 09, 2023 at 02:01:06PM +0530, Nautiyal, Ankit K wrote:
> Hi Ville,
>
> Thanks for the comments and suggestions. Please find my response inline:
>
> On 3/8/2023 8:56 PM, Ville Syrjälä wrote:
> > On Wed, Mar 08, 2023 at 05:10:57PM +0200, Ville Syrjälä wrote:
> >> On Mon, Feb 27, 2023 at
On 09/03/2023 09:59, Andrzej Hajda wrote:
On 09.03.2023 10:43, Tvrtko Ursulin wrote:
On 09/03/2023 09:34, Andrzej Hajda wrote:
On 09.03.2023 10:08, Tvrtko Ursulin wrote:
On 08/03/2023 15:39, Andrzej Hajda wrote:
Write-combining memory allows speculative reads by CPU.
ggtt->error_captu
== Series Details ==
Series: drm/i915/gt: make kobj attributes const
URL : https://patchwork.freedesktop.org/series/114898/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_12829 -> Patchwork_114898v1
Summary
---
**FAIL
On 09.03.2023 10:43, Tvrtko Ursulin wrote:
On 09/03/2023 09:34, Andrzej Hajda wrote:
On 09.03.2023 10:08, Tvrtko Ursulin wrote:
On 08/03/2023 15:39, Andrzej Hajda wrote:
Write-combining memory allows speculative reads by CPU.
ggtt->error_capture is WC mapped to CPU, so CPU/MMU can try
t
On 09/03/2023 09:34, Andrzej Hajda wrote:
On 09.03.2023 10:08, Tvrtko Ursulin wrote:
On 08/03/2023 15:39, Andrzej Hajda wrote:
Write-combining memory allows speculative reads by CPU.
ggtt->error_capture is WC mapped to CPU, so CPU/MMU can try
to prefetch memory beyond the error_capture, ie
On 09.03.2023 10:08, Tvrtko Ursulin wrote:
On 08/03/2023 15:39, Andrzej Hajda wrote:
Write-combining memory allows speculative reads by CPU.
ggtt->error_capture is WC mapped to CPU, so CPU/MMU can try
to prefetch memory beyond the error_capture, ie it tries
to read memory pointed by next PTE
On 09/03/2023 03:46, Ashutosh Dixit wrote:
Expose intel_rps_read_actual_frequency_fw to read the actual freq without
taking forcewake for use by PMU. The code is refactored to use a common set
of functions across sysfs and PMU. Using common functions with sysfs in PMU
solves the issues of missi
On 08/03/2023 15:39, Andrzej Hajda wrote:
Write-combining memory allows speculative reads by CPU.
ggtt->error_capture is WC mapped to CPU, so CPU/MMU can try
to prefetch memory beyond the error_capture, ie it tries
to read memory pointed by next PTE in GGTT.
If this PTE points to invalid addres
Hi Ville,
Thanks for the comments and suggestions. Please find my response inline:
On 3/8/2023 8:56 PM, Ville Syrjälä wrote:
On Wed, Mar 08, 2023 at 05:10:57PM +0200, Ville Syrjälä wrote:
On Mon, Feb 27, 2023 at 09:33:14AM +0530, Ankit Nautiyal wrote:
For YCbCr420 output, scaler is required f
On 3/9/2023 8:54 AM, Andrzej Hajda wrote:
On 08.03.2023 18:29, Das, Nirmoy wrote:
On 3/8/2023 2:36 PM, Andrzej Hajda wrote:
Tests on DG2 machines show that releasing forcewakes during BAR resize
results later in forcewake ack timeouts.
Do we have a fdo/issues url for that? Having that as Ref
== Series Details ==
Series: Enable HDCP2.x via GSC CS (rev12)
URL : https://patchwork.freedesktop.org/series/111876/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_12829 -> Patchwork_111876v12
Summary
---
**SUCCESS**
1 - 100 of 103 matches
Mail list logo
