All of lore.kernel.org
 help / color / mirror / Atom feed
From: Computer Enthusiastic <computer.enthusiastic@gmail.com>
To: Salvatore Bonaccorso <carnil@debian.org>
Cc: nouveau@lists.freedesktop.org, linux-kernel@vger.kernel.org,
	dri-devel@lists.freedesktop.org, Ben Skeggs <bskeggs@redhat.com>,
	stable@vger.kernel.org
Subject: Re: [Nouveau] [PATCH] nouveau: explicitly wait on the fence in nouveau_bo_move_m2mf
Date: Fri, 30 Sep 2022 23:09:59 +0200	[thread overview]
Message-ID: <CAHSpYy1mcTns0JS6eivjK82CZ9_ajSwH-H7gtDwCkNyfvihaAw@mail.gmail.com> (raw)
In-Reply-To: <YymrJSfXe4LaXmkA@eldamar.lan>

[-- Attachment #1: Type: text/plain, Size: 1432 bytes --]

Hello,

Il giorno mar 20 set 2022 alle ore 13:59 Salvatore Bonaccorso
<carnil@debian.org> ha scritto:
[..]
> Computer Enthusiastic, can you verify the problem as well in a
> non-Debian patched upstream kernel directly from the 5.10.y series
> (latest 5.10.144) and verify the fix there?
>
> Regards,
> Salvatore

I've tested the vanilla kernel 5.10.145 (it was the latest one week
ago) without Debian kernel patches, but using the kernel config file
from the latest kernel for Debian Stable:
- without the Karol's patch: it always fails both suspend to ram and
hibernate to disk with the usual behavior (a very long time to suspend
or hibernate, then it fails on resume with a garbled screen)
- with the Karol's patch: it succeeds both suspend and hibernate and
it correctly resumes afterwards.

The kernel was tested using the following graphic adapter:
Graphics:  Device-1: NVIDIA G96CM [GeForce 9600M GT] driver: nouveau v: kernel
          Device-2: Suyin Acer HD Crystal Eye webcam type: USB driver:
uvcvideo
          Display: x11 server: X.Org 1.20.11 driver: loaded:
modesetting unloaded: fbdev,vesa
          resolution: 1280x800~60Hz
          OpenGL: renderer: NV96 v: 3.3 Mesa 20.3.5

Therefore, 5.10.y series of the kernel need to be patched to work
correctly at least with the aforementioned graphic card.

The script I used to compile the kernel are attached for further
reference and verification.

Hope that helps.

[-- Attachment #2: vanilla-kernel-build-5.10.145 --]
[-- Type: application/octet-stream, Size: 572 bytes --]

# Download source code
wget -nc https://cdn.kernel.org/pub/linux/kernel/v5.x/linux-5.10.145.tar.xz
tar xf linux-5.10.145.tar.xz

# Automate subversion index
SUBVERSION_INDEX="1"

# Delete from previous builds
cd linux-5.10.145
rm -rf ./debian
rm -rf ../linux.orig/
rm -rf ../linux-upstream*

cp /boot/config-5.10.0-18-amd64 .config
make olddefconfig

scripts/config --disable SYSTEM_TRUSTED_KEYRING
scripts/config --set-str SYSTEM_TRUSTED_KEYS ''

# Build kernel
time make -j 8 deb-pkg LOCALVERSION=-vanilla KDEB_PKGVERSION=$(make kernelversion)-$SUBVERSION_INDEX

exit 0

[-- Attachment #3: vanilla-kernel-build-5.10.145-patched --]
[-- Type: application/octet-stream, Size: 781 bytes --]

# Download source code
wget -nc https://cdn.kernel.org/pub/linux/kernel/v5.x/linux-5.10.145.tar.xz
tar xf linux-5.10.145.tar.xz

# get patch
wget nc -O nouveau.patch https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/patch/?id=3640cdccbe75b8922e5bfc0191dd37e3aaa24833

# Automate subversion index
SUBVERSION_INDEX="1"

# Delete from previous builds
cd linux-5.10.145
rm -rf ./debian
rm -rf ../linux.orig/
rm -rf ../linux-upstream*

cp /boot/config-5.10.0-18-amd64 .config
make olddefconfig

scripts/config --disable SYSTEM_TRUSTED_KEYRING
scripts/config --set-str SYSTEM_TRUSTED_KEYS ''

# Apply patch
patch -p 1 < ../nouveau.patch || exit 1

# Build kernel
time make -j 8 deb-pkg LOCALVERSION=-patched KDEB_PKGVERSION=$(make kernelversion)-$SUBVERSION_INDEX

exit 0

WARNING: multiple messages have this Message-ID (diff)
From: Computer Enthusiastic <computer.enthusiastic@gmail.com>
To: Salvatore Bonaccorso <carnil@debian.org>
Cc: Karol Herbst <kherbst@redhat.com>,
	nouveau@lists.freedesktop.org, linux-kernel@vger.kernel.org,
	dri-devel@lists.freedesktop.org, Ben Skeggs <bskeggs@redhat.com>,
	stable@vger.kernel.org
Subject: Re: [PATCH] nouveau: explicitly wait on the fence in nouveau_bo_move_m2mf
Date: Fri, 30 Sep 2022 23:09:59 +0200	[thread overview]
Message-ID: <CAHSpYy1mcTns0JS6eivjK82CZ9_ajSwH-H7gtDwCkNyfvihaAw@mail.gmail.com> (raw)
In-Reply-To: <YymrJSfXe4LaXmkA@eldamar.lan>

[-- Attachment #1: Type: text/plain, Size: 1432 bytes --]

Hello,

Il giorno mar 20 set 2022 alle ore 13:59 Salvatore Bonaccorso
<carnil@debian.org> ha scritto:
[..]
> Computer Enthusiastic, can you verify the problem as well in a
> non-Debian patched upstream kernel directly from the 5.10.y series
> (latest 5.10.144) and verify the fix there?
>
> Regards,
> Salvatore

I've tested the vanilla kernel 5.10.145 (it was the latest one week
ago) without Debian kernel patches, but using the kernel config file
from the latest kernel for Debian Stable:
- without the Karol's patch: it always fails both suspend to ram and
hibernate to disk with the usual behavior (a very long time to suspend
or hibernate, then it fails on resume with a garbled screen)
- with the Karol's patch: it succeeds both suspend and hibernate and
it correctly resumes afterwards.

The kernel was tested using the following graphic adapter:
Graphics:  Device-1: NVIDIA G96CM [GeForce 9600M GT] driver: nouveau v: kernel
          Device-2: Suyin Acer HD Crystal Eye webcam type: USB driver:
uvcvideo
          Display: x11 server: X.Org 1.20.11 driver: loaded:
modesetting unloaded: fbdev,vesa
          resolution: 1280x800~60Hz
          OpenGL: renderer: NV96 v: 3.3 Mesa 20.3.5

Therefore, 5.10.y series of the kernel need to be patched to work
correctly at least with the aforementioned graphic card.

The script I used to compile the kernel are attached for further
reference and verification.

Hope that helps.

[-- Attachment #2: vanilla-kernel-build-5.10.145 --]
[-- Type: application/octet-stream, Size: 572 bytes --]

# Download source code
wget -nc https://cdn.kernel.org/pub/linux/kernel/v5.x/linux-5.10.145.tar.xz
tar xf linux-5.10.145.tar.xz

# Automate subversion index
SUBVERSION_INDEX="1"

# Delete from previous builds
cd linux-5.10.145
rm -rf ./debian
rm -rf ../linux.orig/
rm -rf ../linux-upstream*

cp /boot/config-5.10.0-18-amd64 .config
make olddefconfig

scripts/config --disable SYSTEM_TRUSTED_KEYRING
scripts/config --set-str SYSTEM_TRUSTED_KEYS ''

# Build kernel
time make -j 8 deb-pkg LOCALVERSION=-vanilla KDEB_PKGVERSION=$(make kernelversion)-$SUBVERSION_INDEX

exit 0

[-- Attachment #3: vanilla-kernel-build-5.10.145-patched --]
[-- Type: application/octet-stream, Size: 781 bytes --]

# Download source code
wget -nc https://cdn.kernel.org/pub/linux/kernel/v5.x/linux-5.10.145.tar.xz
tar xf linux-5.10.145.tar.xz

# get patch
wget nc -O nouveau.patch https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/patch/?id=3640cdccbe75b8922e5bfc0191dd37e3aaa24833

# Automate subversion index
SUBVERSION_INDEX="1"

# Delete from previous builds
cd linux-5.10.145
rm -rf ./debian
rm -rf ../linux.orig/
rm -rf ../linux-upstream*

cp /boot/config-5.10.0-18-amd64 .config
make olddefconfig

scripts/config --disable SYSTEM_TRUSTED_KEYRING
scripts/config --set-str SYSTEM_TRUSTED_KEYS ''

# Apply patch
patch -p 1 < ../nouveau.patch || exit 1

# Build kernel
time make -j 8 deb-pkg LOCALVERSION=-patched KDEB_PKGVERSION=$(make kernelversion)-$SUBVERSION_INDEX

exit 0

WARNING: multiple messages have this Message-ID (diff)
From: Computer Enthusiastic <computer.enthusiastic@gmail.com>
To: Salvatore Bonaccorso <carnil@debian.org>
Cc: Karol Herbst <kherbst@redhat.com>,
	linux-kernel@vger.kernel.org, Ben Skeggs <bskeggs@redhat.com>,
	Lyude Paul <lyude@redhat.com>,
	dri-devel@lists.freedesktop.org, nouveau@lists.freedesktop.org,
	stable@vger.kernel.org
Subject: Re: [PATCH] nouveau: explicitly wait on the fence in nouveau_bo_move_m2mf
Date: Fri, 30 Sep 2022 23:09:59 +0200	[thread overview]
Message-ID: <CAHSpYy1mcTns0JS6eivjK82CZ9_ajSwH-H7gtDwCkNyfvihaAw@mail.gmail.com> (raw)
In-Reply-To: <YymrJSfXe4LaXmkA@eldamar.lan>

[-- Attachment #1: Type: text/plain, Size: 1432 bytes --]

Hello,

Il giorno mar 20 set 2022 alle ore 13:59 Salvatore Bonaccorso
<carnil@debian.org> ha scritto:
[..]
> Computer Enthusiastic, can you verify the problem as well in a
> non-Debian patched upstream kernel directly from the 5.10.y series
> (latest 5.10.144) and verify the fix there?
>
> Regards,
> Salvatore

I've tested the vanilla kernel 5.10.145 (it was the latest one week
ago) without Debian kernel patches, but using the kernel config file
from the latest kernel for Debian Stable:
- without the Karol's patch: it always fails both suspend to ram and
hibernate to disk with the usual behavior (a very long time to suspend
or hibernate, then it fails on resume with a garbled screen)
- with the Karol's patch: it succeeds both suspend and hibernate and
it correctly resumes afterwards.

The kernel was tested using the following graphic adapter:
Graphics:  Device-1: NVIDIA G96CM [GeForce 9600M GT] driver: nouveau v: kernel
          Device-2: Suyin Acer HD Crystal Eye webcam type: USB driver:
uvcvideo
          Display: x11 server: X.Org 1.20.11 driver: loaded:
modesetting unloaded: fbdev,vesa
          resolution: 1280x800~60Hz
          OpenGL: renderer: NV96 v: 3.3 Mesa 20.3.5

Therefore, 5.10.y series of the kernel need to be patched to work
correctly at least with the aforementioned graphic card.

The script I used to compile the kernel are attached for further
reference and verification.

Hope that helps.

[-- Attachment #2: vanilla-kernel-build-5.10.145 --]
[-- Type: application/octet-stream, Size: 572 bytes --]

# Download source code
wget -nc https://cdn.kernel.org/pub/linux/kernel/v5.x/linux-5.10.145.tar.xz
tar xf linux-5.10.145.tar.xz

# Automate subversion index
SUBVERSION_INDEX="1"

# Delete from previous builds
cd linux-5.10.145
rm -rf ./debian
rm -rf ../linux.orig/
rm -rf ../linux-upstream*

cp /boot/config-5.10.0-18-amd64 .config
make olddefconfig

scripts/config --disable SYSTEM_TRUSTED_KEYRING
scripts/config --set-str SYSTEM_TRUSTED_KEYS ''

# Build kernel
time make -j 8 deb-pkg LOCALVERSION=-vanilla KDEB_PKGVERSION=$(make kernelversion)-$SUBVERSION_INDEX

exit 0

[-- Attachment #3: vanilla-kernel-build-5.10.145-patched --]
[-- Type: application/octet-stream, Size: 781 bytes --]

# Download source code
wget -nc https://cdn.kernel.org/pub/linux/kernel/v5.x/linux-5.10.145.tar.xz
tar xf linux-5.10.145.tar.xz

# get patch
wget nc -O nouveau.patch https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/patch/?id=3640cdccbe75b8922e5bfc0191dd37e3aaa24833

# Automate subversion index
SUBVERSION_INDEX="1"

# Delete from previous builds
cd linux-5.10.145
rm -rf ./debian
rm -rf ../linux.orig/
rm -rf ../linux-upstream*

cp /boot/config-5.10.0-18-amd64 .config
make olddefconfig

scripts/config --disable SYSTEM_TRUSTED_KEYRING
scripts/config --set-str SYSTEM_TRUSTED_KEYS ''

# Apply patch
patch -p 1 < ../nouveau.patch || exit 1

# Build kernel
time make -j 8 deb-pkg LOCALVERSION=-patched KDEB_PKGVERSION=$(make kernelversion)-$SUBVERSION_INDEX

exit 0

  reply	other threads:[~2022-09-30 21:10 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-08-19 20:09 [PATCH] nouveau: explicitly wait on the fence in nouveau_bo_move_m2mf Karol Herbst
2022-08-19 20:09 ` Karol Herbst
2022-08-19 20:09 ` [Nouveau] " Karol Herbst
2022-08-22 21:15 ` Lyude Paul
2022-08-22 21:15   ` Lyude Paul
2022-08-22 21:15   ` [Nouveau] " Lyude Paul
2022-09-20 10:42 ` Salvatore Bonaccorso
2022-09-20 10:42   ` Salvatore Bonaccorso
2022-09-20 10:42   ` [Nouveau] " Salvatore Bonaccorso
2022-09-20 11:36   ` Karol Herbst
2022-09-20 11:36     ` Karol Herbst
2022-09-20 11:36     ` [Nouveau] " Karol Herbst
2022-09-20 11:59     ` Salvatore Bonaccorso
2022-09-20 11:59       ` Salvatore Bonaccorso
2022-09-20 11:59       ` [Nouveau] " Salvatore Bonaccorso
2022-09-30 21:09       ` Computer Enthusiastic [this message]
2022-09-30 21:09         ` Computer Enthusiastic
2022-09-30 21:09         ` Computer Enthusiastic
2022-11-01  6:44 ` [Nouveau] " Computer Enthusiastic
2022-11-01 10:46   ` Karol Herbst
2022-12-23  9:43     ` Computer Enthusiastic
2023-01-28 14:49       ` Computer Enthusiastic
2023-01-28 14:49         ` Computer Enthusiastic
2023-01-28 17:51         ` Greg KH
2023-01-28 17:51           ` Greg KH
2023-01-28 19:49           ` Salvatore Bonaccorso
2023-01-28 19:49             ` Salvatore Bonaccorso
2023-01-29 21:36             ` Computer Enthusiastic
2023-01-29 21:36               ` Computer Enthusiastic
2023-01-30 10:05               ` Greg KH
2023-01-30 10:05                 ` Greg KH
2023-01-30 22:27                 ` Lyude Paul
2023-01-30 22:27                   ` Lyude Paul
2023-02-01 19:24                   ` Computer Enthusiastic
2023-02-01 19:24                     ` Computer Enthusiastic
2022-11-19  5:20 ` Computer Enthusiastic
2022-11-19  5:20   ` Computer Enthusiastic

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=CAHSpYy1mcTns0JS6eivjK82CZ9_ajSwH-H7gtDwCkNyfvihaAw@mail.gmail.com \
    --to=computer.enthusiastic@gmail.com \
    --cc=bskeggs@redhat.com \
    --cc=carnil@debian.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nouveau@lists.freedesktop.org \
    --cc=stable@vger.kernel.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.