[PULL] drm-xe-fixes

Hi Dave and Sima, Please pull the drm-xe-fixes for this week targeting v6.9-rc7. Two important fixes: one avoiding a use-after-free in the rebind worker and the other to make display work in ADL-N. thanks Lucas De Marchi drm-xe-fixes-2024-05-02: - Fix UAF on rebind worker - Fix ADL-N display

[PULL] drm-xe-fixes

Hi Dave and Sima Please pull the drm-xe-fixes for this week targeting v6.9-rc6. Simple fixes not really visible to end users: 2 around error paths on drm managed allocations and 1 on PF/VF relay messages that are not enabled by default. thanks Lucas De Marchi drm-xe-fixes-2024-04-25: - Fix

[PULL] drm-xe-fixes

Hi Dave and Sima, Please pull the drm-xe-fixes for this week targeting v6.9-rc5. thanks Lucas De Marchi drm-xe-fixes-2024-04-18: - Fix bo leak on error path during fb init - Fix use-after-free due to order vm is put and destroyed The following changes since commit

[PULL] drm-xe-fixes

future commit making use of that register. thanks Lucas De Marchi drm-xe-fixes-2024-04-11: - Fix double display mutex initializations - Fix u32 -> u64 implicit conversions - Fix RING_CONTEXT_CONTROL not marked as masked The following changes since commit fec50db7033ea478773b159e0e2efb135270e

[PULL] drm-xe-fixes

with the workqueue changes introduced in v6.9. That shouldn't be the normal use case but was causing CI to abort further tests. Other changes include fixes around rebinding and TLB invalidation. thanks Lucas De Marchi drm-xe-fixes-2024-04-04: - Stop using system_unbound_wq for preempt fences, as this can

[PULL] drm-xe-fixes

) - Fix build on mips - Fix wrong bound checks - Fix use of msec rather than jiffies - Remove dead code Lucas De Marchi (1): drm/xe: Fix END redefinition Matthew Auld (5

[PULL] drm-xe-next-fixes

. Lucas De Marchi drm-xe-next-fixes-2024-03-14: Driver changes: - Invalidate userptr VMA on page pin fault, allowing userspace to free userptr while still having bindings - Fail early on sysfs file creation error - Skip VMA pinning on xe_exec with num_batch_buffer == 0 The following changes

[PULL] drm-xe-next-fixes

handling page fault on a VM not in fault mode - Drop use of "grouped target" feature in Makefile since that's only available in make >= 4.3 thanks, Lucas De Marchi The following changes since commit c6d6a82d8a9f8f9326b760accaa532b839b80140: Merge tag 'drm-misc-next-fixes-2024-02-

Re: [PULL] drm-xe-next

On Mon, Feb 26, 2024 at 10:46:43AM +, Daniel Stone wrote: Hi, On Mon, 26 Feb 2024 at 03:21, Lucas De Marchi wrote: All of this should be fixed by now: dim is used for applying and pushing patches, which has additional checks so that doesn't happen again. Still pending confirmation from

[PULL] drm-xe-next

: - Make sure intel_wakeref_t is treated as opaque type on i915-display and fix its type on xe Driver Changes: - Drop pre-production workarounds (Matt Roper) - Drop kunit tests for unsuported platforms: PVC and pre-production DG2 (Lucas De Marchi) - Start pumbling SR-IOV support with memory based

Re: [PATCH] nightly.conf: Add the xe repo to drm-tip

On Mon, Jan 08, 2024 at 05:13:51PM -0500, Rodrigo Vivi wrote: On Wed, Jan 03, 2024 at 11:59:16PM -0600, Lucas De Marchi wrote: On Wed, Jan 03, 2024 at 02:50:57PM +0100, Thomas Hellström wrote: > On Tue, 2023-12-26 at 13:30 -0500, Rodrigo Vivi wrote: > > On Fri, Dec 22, 2023 at 12:36:3

Re: [PATCH] nightly.conf: Add the xe repo to drm-tip

equest for drm- > tip, > since that branch tip is currently adapted for our CI testing. > > Cc: Rodrigo Vivi > Cc: Lucas De Marchi > Cc: Oded Gabbay > Cc: daniel.vet...@ffwll.ch > Cc: Maarten Lankhorst > Cc: dim-tools@lists.freedesktop.org > Cc: dri-de.

Re: [PATCH] nightly.conf: Add the xe repo to drm-tip

On Fri, Dec 22, 2023 at 12:36:39PM +0100, Thomas Hellström wrote: Add the xe repo to drm-tip and the dim tools. For now use the sha1 of the first drm-xe-next pull request for drm-tip, since that branch tip is currently adapted for our CI testing. Cc: Rodrigo Vivi Cc: Lucas De Marchi Cc: Oded

Re: [MAINTAINER TOOLS] docs: updated rules for topic/core-for-CI commit management

it's easier to see what was done in the branch since we don't have to procure the right CI tag in which things got changed. ... I actually came here to ask: wasn't gitlab supposed to be used for patches in maintainer-tools? Lucas De Marchi

Re: [PULL] drm-intel-gt-next

-p1 with 064030837c5b being the commit in drm-rerere. Cc'ing Matt Roper Lucas De Marchi We opted to sync the branches via drm-next pulls and backmerges, but I'm afraid that means you'd have to use the fixups when merging. I guess we failed to communicate that. The alternative would've been cross-merges within

Re: [PATCH] dim: allow to add link to the current HEAD

On Mon, Nov 18, 2019 at 12:16:19PM -0800, Rodrigo Vivi wrote: On Nov 18, 2019, at 12:14 PM, Lucas De Marchi <[1]lucas.demar...@intel.com> wrote: On Mon, Nov 18, 2019 at 04:57:34PM +0200, Jani Nikula wrote: On Fri, 15 Nov 2019, Lucas De Marchi <[2]lucas.demar...@

Re: [PATCH] dim: allow to add link to the current HEAD

On Mon, Nov 18, 2019 at 04:57:34PM +0200, Jani Nikula wrote: On Fri, 15 Nov 2019, Lucas De Marchi wrote: Add method add-link-head that applies the link to head without any additional check. This is useful either to reduce typing or to add a link while in the middle of an interactive rebase

[PATCH] dim: allow to add link to the current HEAD

Add method add-link-head that applies the link to head without any additional check. This is useful either to reduce typing or to add a link while in the middle of an interactive rebase. Signed-off-by: Lucas De Marchi --- dim | 23 +++ 1 file changed, 15 insertions(+), 8

Re: [PATCH 0/4] dim setup improvements

On Mon, Mar 11, 2019 at 12:08:42PM +0100, Daniel Vetter wrote: On Mon, Mar 04, 2019 at 03:07:38PM -0800, Lucas De Marchi wrote: On Mon, Mar 04, 2019 at 11:12:21PM +0200, Jani Nikula wrote: > On Mon, 04 Mar 2019, Lucas De Marchi wrote: > > We had some small problems when calling

Re: [PATCH 2/2] doc: restructure Getting Started page

On Fri, Mar 08, 2019 at 02:14:03PM -0800, Rodrigo Vivi wrote: On Tue, Mar 05, 2019 at 11:44:00AM -0800, Lucas De Marchi wrote: Restructure this pages so it first gives a minimal set of commands to get the environment setup and then explains the other parts. We tailor the commands so the user

[PATCH 1/2] doc: move "Getting Started" to be the first page

I myself missed that we had this page for a long time since it's not positioned as something to be the first thing to do. Signed-off-by: Lucas De Marchi --- index.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/index.rst b/index.rst index 14086bb..ecc721a 100644

Re: [PATCH 0/4] dim setup improvements

On Mon, Mar 04, 2019 at 11:12:21PM +0200, Jani Nikula wrote: On Mon, 04 Mar 2019, Lucas De Marchi wrote: We had some small problems when calling dim setup. This should hopefully fix them all, making it easier to setup a new environment like I was doing. Now setting up the environment from

[PATCH 2/4] dim: do all setup relative to $DIM_PREFIX

Signed-off-by: Lucas De Marchi --- dim | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/dim b/dim index 8e01c66..b125003 100755 --- a/dim +++ b/dim @@ -2234,28 +2234,28 @@ function dim_setup fi fi - if [[ ! -d "$DIM_P

[PATCH 3/4] dim: update branches after setup

Otherwise we need to document it better so user doesn't try to find out why he doesn't have the remotes he was supposed to have. Signed-off-by: Lucas De Marchi --- dim | 2 ++ 1 file changed, 2 insertions(+) diff --git a/dim b/dim index b125003..bada79c 100755 --- a/dim +++ b/dim @@ -2260,6

[PATCH 1/4] dim: clone in the right dir

Signed-off-by: Lucas De Marchi Acked-by: Jani Nikula --- dim | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dim b/dim index 004ec69..8e01c66 100755 --- a/dim +++ b/dim @@ -2236,7 +2236,7 @@ function dim_setup if [[ ! -d "$DIM_PREFIX/$DIM

[PATCH 0/4] dim setup improvements

hts? Lucas De Marchi (4): dim: clone in the right dir dim: do all setup relative to $DIM_PREFIX dim: update branches after setup dim: automatically clone maintainer-tools repo dim | 32 +++- 1 file changed, 19 insertions(+), 13 deletions(-) -- 2.2

[PATCH 4/4] dim: automatically clone maintainer-tools repo

Do like it's done for other repo instead of just complain. Signed-off-by: Lucas De Marchi --- dim | 16 ++-- 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/dim b/dim index bada79c..1d3854b 100755 --- a/dim +++ b/dim @@ -76,8 +76,6 @@ dim=$(basename $0) dim_today

[PATCH] dim: clone in the right dir

Signed-off-by: Lucas De Marchi --- dim | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dim b/dim index 10f8232..b6c88b4 100755 --- a/dim +++ b/dim @@ -2213,7 +2213,7 @@ function dim_setup if [[ ! -d "$DIM_PREFIX/$DIM_REPO" ]]; then i

[PATCH] dim: do not run hooks when rebuilding tip

dim rebuild-tip was failing for me and after investigation with Daniel Vetter, the culprit was having a pre-commit git hook enabled. Let's pass --no-verify to avoid the same mistake for other people. Signed-off-by: Lucas De Marchi --- dim | 2 +- 1 file changed, 1 insertion(+), 1 deletion

Re: [PATCH] TODO: remove TODO, handle bugs and feature requests on gitlab

issues then close https://gitlab.freedesktop.org/drm/maintainer-tools/issues/1 ? > > Signed-off-by: Jani Nikula Reviewed-by: Lucas De Marchi Lucas De Marchi > --- > CONTRIBUTING.rst | 7 --- > TODO.rst | 44 > index.rst

Re: [Intel-gfx] [PATCH] dim: Add range-diff convenience wrapper

What I kind of dislike on wrappers is the inability to provide more arguments. One that would be useful here would be a --no-patch if one only wants to see status change, not the change itself. I'm still getting used to the tool so I'm always explicit: I always type both ranges to t

Re: [PATCH] RFC: CONTRIBUTING: Embrace gitlab

repo? Not sure how crazy that would be. Do you mean the public-inbox? See https://lore.kernel.org/lkml/ and https://lwn.net/Articles/748184/ Lucas De Marchi ___ dim-tools mailing list dim-tools@lists.freedesktop.org https://lists.freedesktop.org

[PATCH] CI: switch to python3 by default

Signed-off-by: Lucas De Marchi --- .gitlab-ci.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index a23c324..91cb6a2 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -1,7 +1,7 @@ image: fedora:latest before_script: - - dnf install

Re: [PATCH] build: make it compatible with python3 tools

On Thu, Aug 30, 2018 at 12:01 AM Daniel Vetter wrote: > > On Wed, Aug 29, 2018 at 01:15:17PM -0700, Lucas De Marchi wrote: > > sphinx-build and rst2man binaries on Fedora 28 have a -3 suffix when > > they are installed for python3 in order to be able to be installed in > >

Re: [PATCH] Add .gitlab-ci.yml

ection) to disable this warning. sphinx-rtd-theme (0.3.1) sphinxcontrib-websupport (1.0.1.dev20180209) $ $ cat $(which sphinx-build)| head -n 1 #!/usr/bin/python3 And make html continues to work for me... > > Reviewed-by: Lucas De Marchi > Signed-off-by: Daniel Vetter > --- >

Re: [PATCH] Add .gitlab-ci.yml

need to be installed on every run? Could we rather default to python3 or is it broken for our docs? "make html" works for me with python3 using sphinx 1.7.5 on Fedora 28. > + > +check: > +stage: build > +script: > + - make check Regardless of the comments abov

Re: [PATCH] qf: replace colon when wiggling

On Fri, Jul 13, 2018 at 11:16:12AM -0700, Rodrigo Vivi wrote: > On Fri, Jul 13, 2018 at 08:02:17AM -0700, Lucas De Marchi wrote: > > We are not writing anything after that message. > > I think I probably aimed to do something else different that > that commented out cat the

[PATCH] qf: replace colon when wiggling

We are not writing anything after that message. Signed-off-by: Lucas De Marchi --- qf | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/qf b/qf index bffe9b753fa0..2ac071029992 100755 --- a/qf +++ b/qf @@ -477,7 +477,7 @@ function qf_wiggle_push fi for file

Re: [Intel-gfx] [PATCH] dim: replace pipe commands with single sed

Now CC the right mailing list. The way dim sets up the repository you can't have individual git configs, e.g. to set a different sendemail.to for dim-tools :( Lucas De Marchi On Wed, May 9, 2018 at 2:28 PM, Lucas De Marchi <lucas.demar...@intel.com> wrote: > A single sed can d

Re: [PATCH 2/4] dim: shut up sed broken pipe noise in apply-pull

without any pipe: pull_branch=$(sed -ne '/[gG]it repository at:$/{n;n;p;}' $file) i.e, next, next, print :) Lucas De Marchi > > if [[ -z "$pull_branch" ]] ; then > echoerr "no pull request found" > -- > 2.17.0 > > ___

[PATCH] qf: add diff option to quilt refresh

-by: Lucas De Marchi <lucas.demar...@intel.com> --- qf | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/qf b/qf index 629f1b349a1f..b75e702793d8 100755 --- a/qf +++ b/qf @@ -190,7 +190,7 @@ function quilt_clean_check } function quilt_refresh { - quilt refresh --no-time

[PATCH] qf: remove non-existent argument

qf_setup expects only one argument and branch_init none. Signed-off-by: Lucas De Marchi <lucas.demar...@intel.com> --- qf | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/qf b/qf index 1d61b2ad7692..629f1b349a1f 100755 --- a/qf +++ b/qf @@ -230,7 +230,7 @@ function qf

[PATCH] qf: fix internal use of alias

maintainer-tools/qf: line 415: qf_co: command not found Signed-off-by: Lucas De Marchi <lucas.demar...@intel.com> --- qf | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/qf b/qf index d1499ada1951..4c8a0e9ecc16 100755 --- a/qf +++ b/qf @@ -412,7 +412,7 @@ fu

Re: [PATCH] qf: Make qf_pull -f forcibly clean

base from anywhere. > > v2: Remove dubious comments and use -f. > v3: Avoid git pull --ff-only breaking force so force with fetch > and hard reset > > Cc: Paulo Zanoni <paulo.r.zan...@intel.com> > Cc: Michel Thierry <michel.thie...@intel.com> > Cc: James

Re: [PATCH 2/3] qf: Make qf_pull -f forcibly clean

On Thu, Mar 08, 2018 at 05:25:44PM -0800, Rodrigo Vivi wrote: > On Thu, Mar 08, 2018 at 07:29:52AM -0800, Lucas De Marchi wrote: > > On Wed, Mar 07, 2018 at 05:42:54PM -0800, Rodrigo Vivi wrote: > > > qf_checkout implies that you need to have that baseline > > > on

Re: [PATCH 3/3] qf: Actually allow qf rebase from a clean environment.

p/drm-tip > > Always simply work to rebase from anywhere. > > v2: Implement this behavior only on -f is used to avoid losing work in > progress (Daniel). > > Fixes: 663e7293e042 ("qf: Allow rebase function to start from anywhere.") > Cc: Lucas De Marchi <l

Re: [PATCH 2/3] qf: Make qf_pull -f forcibly clean

base from anywhere. > > v2: Remove dubious comments and use -f. > > Cc: Paulo Zanoni <paulo.r.zan...@intel.com> > Cc: Michel Thierry <michel.thie...@intel.com> > Cc: James Ausmus <james.aus...@intel.com> > Cc: Lucas De Marchi <lucas.demar...@intel.com> >

Re: [PATCH 1/3] qf: Add more wiggle helpers.

; > v2: Remove -r from rm. (Lucas) > Use -z instead of '== ""' comparison. (Lucas) > Add documentation to the helpers. (Required by make check). > > Cc: Paulo Zanoni <paulo.r.zan...@intel.com> > Cc: Michel Thierry <michel.thie...@intel.com> > Cc: J