Hello
Since some days on next, nouveau fail to load:
[ 2.754087] nouveau 0000:02:00.0: vgaarb: deactivate vga console
[ 2.761260] Console: switching to colour dummy device 80x25
[ 2.766888] nouveau 0000:02:00.0: NVIDIA MCP77/MCP78 (0aa480a2)
[ 2.783954] nouveau 0000:02:00.0: bios: version 62.77.2a.00.04
[ 2.810122] nouveau 0000:02:00.0: fb: 256 MiB stolen system memory
[ 3.484031] nouveau 0000:02:00.0: DRM: VRAM: 256 MiB
[ 3.488993] nouveau 0000:02:00.0: DRM: GART: 1048576 MiB
[ 3.494308] nouveau 0000:02:00.0: DRM: TMDS table version 2.0
[ 3.500052] nouveau 0000:02:00.0: DRM: DCB version 4.0
[ 3.505192] nouveau 0000:02:00.0: DRM: DCB outp 00: 01000300 0000001e
[ 3.511632] nouveau 0000:02:00.0: DRM: DCB outp 01: 01011332 00020010
[ 3.518074] nouveau 0000:02:00.0: DRM: DCB conn 00: 00000100
[ 3.523728] nouveau 0000:02:00.0: DRM: DCB conn 01: 00001261
[ 3.529455] nouveau 0000:02:00.0: DRM: failed to initialise sync subsystem, -28
[ 3.545946] nouveau: probe of 0000:02:00.0 failed with error -28
I bisected it to:
git bisect start
# good: [62fb9874f5da54fdb243003b386128037319b219] Linux 5.13
git bisect good 62fb9874f5da54fdb243003b386128037319b219
# bad: [fb0ca446157a86b75502c1636b0d81e642fe6bf1] Add linux-next specific files for 20210701
git bisect bad fb0ca446157a86b75502c1636b0d81e642fe6bf1
# good: [f63c4fda987a19b1194cc45cb72fd5bf968d9d90] Merge remote-tracking branch 'rdma/for-next'
git bisect good f63c4fda987a19b1194cc45cb72fd5bf968d9d90
# bad: [49c8769be0b910d4134eba07cae5d9c71b861c4a] Merge remote-tracking branch 'drm/drm-next'
git bisect bad 49c8769be0b910d4134eba07cae5d9c71b861c4a
# good: [4e3db44a242a4e2afe33b59793898ecbb61d478e] Merge tag 'wireless-drivers-next-2021-06-25' of git://git.kernel.org/pub/scm/linux/kernel/git/kvalo/wireless-drivers-next
git bisect good 4e3db44a242a4e2afe33b59793898ecbb61d478e
# bad: [5745d647d5563d3e9d32013ad4e5c629acff04d7] Merge tag 'amd-drm-next-5.14-2021-06-02' of https://gitlab.freedesktop.org/agd5f/linux into drm-next
git bisect bad 5745d647d5563d3e9d32013ad4e5c629acff04d7
# bad: [c99c4d0ca57c978dcc2a2f41ab8449684ea154cc] Merge tag 'amd-drm-next-5.14-2021-05-19' of https://gitlab.freedesktop.org/agd5f/linux into drm-next
git bisect bad c99c4d0ca57c978dcc2a2f41ab8449684ea154cc
# bad: [ae25ec2fc6c5a9e5767bf1922cd648501d0f914c] Merge tag 'drm-misc-next-2021-05-17' of git://anongit.freedesktop.org/drm/drm-misc into drm-next
git bisect bad ae25ec2fc6c5a9e5767bf1922cd648501d0f914c
# bad: [cac80e71cfb0b00202d743c6e90333c45ba77cc5] drm/vkms: rename cursor to plane on ops of planes composition
git bisect bad cac80e71cfb0b00202d743c6e90333c45ba77cc5
# good: [178bdba84c5f0ad14de384fc7f15fba0e272919d] drm/ttm/ttm_device: Demote kernel-doc abuses
git bisect good 178bdba84c5f0ad14de384fc7f15fba0e272919d
# bad: [3f3a6524f6065fd3d130515e012f63eac74d96da] drm/dp: Clarify DP AUX registration time
git bisect bad 3f3a6524f6065fd3d130515e012f63eac74d96da
# bad: [6dd7efc437611db16d432e0030f72d0c7e890127] drm/gud: cleanup coding style a bit
git bisect bad 6dd7efc437611db16d432e0030f72d0c7e890127
# bad: [13b29cc3a722c2c0bc9ab9f72f9047d55d08a2f9] drm/mxsfb: Don't select DRM_KMS_FB_HELPER
git bisect bad 13b29cc3a722c2c0bc9ab9f72f9047d55d08a2f9
# bad: [d02117f8efaa5fbc37437df1ae955a147a2a424a] drm/ttm: remove special handling for non GEM drivers
git bisect bad d02117f8efaa5fbc37437df1ae955a147a2a424a
# good: [13ea9aa1e7d891e950230e82f1dd2c84e5debcff] drm/ttm: fix error handling if no BO can be swapped out v4
git bisect good 13ea9aa1e7d891e950230e82f1dd2c84e5debcff
# first bad commit: [d02117f8efaa5fbc37437df1ae955a147a2a424a] drm/ttm: remove special handling for non GEM drivers
Reverting the patch permit to have nouveau works again.
Regards
I am also hitting this with upstream. Reverting d02117f8efaa ("drm/ttm: remove
special handling for non GEM drivers") also fixed it for me.
The change log for that commit reads:
drm/ttm: remove special handling for non GEM drivers
vmwgfx is the only driver actually using this. Move the handling into
the driver instead.
I wonder if Nouveau might actually have been using this somehow too?
- Alistair
On Sunday, 4 July 2021 5:21:29 AM AEST Corentin Labbe wrote:
> Hello
>
> Since some days on next, nouveau fail to load:
> [ 2.754087] nouveau 0000:02:00.0: vgaarb: deactivate vga console
> [ 2.761260] Console: switching to colour dummy device 80x25
> [ 2.766888] nouveau 0000:02:00.0: NVIDIA MCP77/MCP78 (0aa480a2)
> [ 2.783954] nouveau 0000:02:00.0: bios: version 62.77.2a.00.04
> [ 2.810122] nouveau 0000:02:00.0: fb: 256 MiB stolen system memory
> [ 3.484031] nouveau 0000:02:00.0: DRM: VRAM: 256 MiB
> [ 3.488993] nouveau 0000:02:00.0: DRM: GART: 1048576 MiB
> [ 3.494308] nouveau 0000:02:00.0: DRM: TMDS table version 2.0
> [ 3.500052] nouveau 0000:02:00.0: DRM: DCB version 4.0
> [ 3.505192] nouveau 0000:02:00.0: DRM: DCB outp 00: 01000300 0000001e
> [ 3.511632] nouveau 0000:02:00.0: DRM: DCB outp 01: 01011332 00020010
> [ 3.518074] nouveau 0000:02:00.0: DRM: DCB conn 00: 00000100
> [ 3.523728] nouveau 0000:02:00.0: DRM: DCB conn 01: 00001261
> [ 3.529455] nouveau 0000:02:00.0: DRM: failed to initialise sync
subsystem, -28
> [ 3.545946] nouveau: probe of 0000:02:00.0 failed with error -28
>
> I bisected it to:
> git bisect start
> # good: [62fb9874f5da54fdb243003b386128037319b219] Linux 5.13
> git bisect good 62fb9874f5da54fdb243003b386128037319b219
> # bad: [fb0ca446157a86b75502c1636b0d81e642fe6bf1] Add linux-next specific
files for 20210701
> git bisect bad fb0ca446157a86b75502c1636b0d81e642fe6bf1
> # good: [f63c4fda987a19b1194cc45cb72fd5bf968d9d90] Merge remote-tracking
branch 'rdma/for-next'
> git bisect good f63c4fda987a19b1194cc45cb72fd5bf968d9d90
> # bad: [49c8769be0b910d4134eba07cae5d9c71b861c4a] Merge remote-tracking
branch 'drm/drm-next'
> git bisect bad 49c8769be0b910d4134eba07cae5d9c71b861c4a
> # good: [4e3db44a242a4e2afe33b59793898ecbb61d478e] Merge tag 'wireless-
drivers-next-2021-06-25' of git://git.kernel.org/pub/scm/linux/kernel/git/
kvalo/wireless-drivers-next
> git bisect good 4e3db44a242a4e2afe33b59793898ecbb61d478e
> # bad: [5745d647d5563d3e9d32013ad4e5c629acff04d7] Merge tag 'amd-drm-
next-5.14-2021-06-02' of https://gitlab.freedesktop.org/agd5f/linux into drm-
next
> git bisect bad 5745d647d5563d3e9d32013ad4e5c629acff04d7
> # bad: [c99c4d0ca57c978dcc2a2f41ab8449684ea154cc] Merge tag 'amd-drm-
next-5.14-2021-05-19' of https://gitlab.freedesktop.org/agd5f/linux into drm-
next
> git bisect bad c99c4d0ca57c978dcc2a2f41ab8449684ea154cc
> # bad: [ae25ec2fc6c5a9e5767bf1922cd648501d0f914c] Merge tag 'drm-misc-
next-2021-05-17' of git://anongit.freedesktop.org/drm/drm-misc into drm-next
> git bisect bad ae25ec2fc6c5a9e5767bf1922cd648501d0f914c
> # bad: [cac80e71cfb0b00202d743c6e90333c45ba77cc5] drm/vkms: rename cursor to
plane on ops of planes composition
> git bisect bad cac80e71cfb0b00202d743c6e90333c45ba77cc5
> # good: [178bdba84c5f0ad14de384fc7f15fba0e272919d] drm/ttm/ttm_device:
Demote kernel-doc abuses
> git bisect good 178bdba84c5f0ad14de384fc7f15fba0e272919d
> # bad: [3f3a6524f6065fd3d130515e012f63eac74d96da] drm/dp: Clarify DP AUX
registration time
> git bisect bad 3f3a6524f6065fd3d130515e012f63eac74d96da
> # bad: [6dd7efc437611db16d432e0030f72d0c7e890127] drm/gud: cleanup coding
style a bit
> git bisect bad 6dd7efc437611db16d432e0030f72d0c7e890127
> # bad: [13b29cc3a722c2c0bc9ab9f72f9047d55d08a2f9] drm/mxsfb: Don't select
DRM_KMS_FB_HELPER
> git bisect bad 13b29cc3a722c2c0bc9ab9f72f9047d55d08a2f9
> # bad: [d02117f8efaa5fbc37437df1ae955a147a2a424a] drm/ttm: remove special
handling for non GEM drivers
> git bisect bad d02117f8efaa5fbc37437df1ae955a147a2a424a
> # good: [13ea9aa1e7d891e950230e82f1dd2c84e5debcff] drm/ttm: fix error
handling if no BO can be swapped out v4
> git bisect good 13ea9aa1e7d891e950230e82f1dd2c84e5debcff
> # first bad commit: [d02117f8efaa5fbc37437df1ae955a147a2a424a] drm/ttm:
remove special handling for non GEM drivers
>
> Reverting the patch permit to have nouveau works again.
>
> Regards
>
>
Hi guys,
yes nouveau was using the same functionality for internal BOs without
noticing it. This is fixes by the following commit:
commit d098775ed44021293b1962dea61efb19297b8d02
Author: Christian König <[email protected]>
Date: Wed Jun 9 19:25:56 2021 +0200
drm/nouveau: init the base GEM fields for internal BOs
TTMs buffer objects are based on GEM objects for quite a while
and rely on initializing those fields before initializing the TTM BO.
Nouveau now doesn't init the GEM object for internally allocated BOs,
so make sure that we at least initialize some necessary fields.
Could be that the patch needs to be send to stable as well.
Regards,
Christian.
Am 06.07.21 um 04:44 schrieb Alistair Popple:
> I am also hitting this with upstream. Reverting d02117f8efaa ("drm/ttm: remove
> special handling for non GEM drivers") also fixed it for me.
>
> The change log for that commit reads:
>
> drm/ttm: remove special handling for non GEM drivers
>
> vmwgfx is the only driver actually using this. Move the handling into
> the driver instead.
>
> I wonder if Nouveau might actually have been using this somehow too?
>
> - Alistair
>
> On Sunday, 4 July 2021 5:21:29 AM AEST Corentin Labbe wrote:
>> Hello
>>
>> Since some days on next, nouveau fail to load:
>> [ 2.754087] nouveau 0000:02:00.0: vgaarb: deactivate vga console
>> [ 2.761260] Console: switching to colour dummy device 80x25
>> [ 2.766888] nouveau 0000:02:00.0: NVIDIA MCP77/MCP78 (0aa480a2)
>> [ 2.783954] nouveau 0000:02:00.0: bios: version 62.77.2a.00.04
>> [ 2.810122] nouveau 0000:02:00.0: fb: 256 MiB stolen system memory
>> [ 3.484031] nouveau 0000:02:00.0: DRM: VRAM: 256 MiB
>> [ 3.488993] nouveau 0000:02:00.0: DRM: GART: 1048576 MiB
>> [ 3.494308] nouveau 0000:02:00.0: DRM: TMDS table version 2.0
>> [ 3.500052] nouveau 0000:02:00.0: DRM: DCB version 4.0
>> [ 3.505192] nouveau 0000:02:00.0: DRM: DCB outp 00: 01000300 0000001e
>> [ 3.511632] nouveau 0000:02:00.0: DRM: DCB outp 01: 01011332 00020010
>> [ 3.518074] nouveau 0000:02:00.0: DRM: DCB conn 00: 00000100
>> [ 3.523728] nouveau 0000:02:00.0: DRM: DCB conn 01: 00001261
>> [ 3.529455] nouveau 0000:02:00.0: DRM: failed to initialise sync
> subsystem, -28
>> [ 3.545946] nouveau: probe of 0000:02:00.0 failed with error -28
>>
>> I bisected it to:
>> git bisect start
>> # good: [62fb9874f5da54fdb243003b386128037319b219] Linux 5.13
>> git bisect good 62fb9874f5da54fdb243003b386128037319b219
>> # bad: [fb0ca446157a86b75502c1636b0d81e642fe6bf1] Add linux-next specific
> files for 20210701
>> git bisect bad fb0ca446157a86b75502c1636b0d81e642fe6bf1
>> # good: [f63c4fda987a19b1194cc45cb72fd5bf968d9d90] Merge remote-tracking
> branch 'rdma/for-next'
>> git bisect good f63c4fda987a19b1194cc45cb72fd5bf968d9d90
>> # bad: [49c8769be0b910d4134eba07cae5d9c71b861c4a] Merge remote-tracking
> branch 'drm/drm-next'
>> git bisect bad 49c8769be0b910d4134eba07cae5d9c71b861c4a
>> # good: [4e3db44a242a4e2afe33b59793898ecbb61d478e] Merge tag 'wireless-
> drivers-next-2021-06-25' of git://git.kernel.org/pub/scm/linux/kernel/git/
> kvalo/wireless-drivers-next
>> git bisect good 4e3db44a242a4e2afe33b59793898ecbb61d478e
>> # bad: [5745d647d5563d3e9d32013ad4e5c629acff04d7] Merge tag 'amd-drm-
> next-5.14-2021-06-02' of https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.freedesktop.org%2Fagd5f%2Flinux&data=04%7C01%7Cchristian.koenig%40amd.com%7C5f05fa59cd3b4432e71108d94027ede6%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637611362989756089%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=gJ98NtSRf3IyVfsCzw3DKydeMTGKIkHJNzUUhUfsWzY%3D&reserved=0 into drm-
> next
>> git bisect bad 5745d647d5563d3e9d32013ad4e5c629acff04d7
>> # bad: [c99c4d0ca57c978dcc2a2f41ab8449684ea154cc] Merge tag 'amd-drm-
> next-5.14-2021-05-19' of https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.freedesktop.org%2Fagd5f%2Flinux&data=04%7C01%7Cchristian.koenig%40amd.com%7C5f05fa59cd3b4432e71108d94027ede6%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637611362989756089%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=gJ98NtSRf3IyVfsCzw3DKydeMTGKIkHJNzUUhUfsWzY%3D&reserved=0 into drm-
> next
>> git bisect bad c99c4d0ca57c978dcc2a2f41ab8449684ea154cc
>> # bad: [ae25ec2fc6c5a9e5767bf1922cd648501d0f914c] Merge tag 'drm-misc-
> next-2021-05-17' of git://anongit.freedesktop.org/drm/drm-misc into drm-next
>> git bisect bad ae25ec2fc6c5a9e5767bf1922cd648501d0f914c
>> # bad: [cac80e71cfb0b00202d743c6e90333c45ba77cc5] drm/vkms: rename cursor to
> plane on ops of planes composition
>> git bisect bad cac80e71cfb0b00202d743c6e90333c45ba77cc5
>> # good: [178bdba84c5f0ad14de384fc7f15fba0e272919d] drm/ttm/ttm_device:
> Demote kernel-doc abuses
>> git bisect good 178bdba84c5f0ad14de384fc7f15fba0e272919d
>> # bad: [3f3a6524f6065fd3d130515e012f63eac74d96da] drm/dp: Clarify DP AUX
> registration time
>> git bisect bad 3f3a6524f6065fd3d130515e012f63eac74d96da
>> # bad: [6dd7efc437611db16d432e0030f72d0c7e890127] drm/gud: cleanup coding
> style a bit
>> git bisect bad 6dd7efc437611db16d432e0030f72d0c7e890127
>> # bad: [13b29cc3a722c2c0bc9ab9f72f9047d55d08a2f9] drm/mxsfb: Don't select
> DRM_KMS_FB_HELPER
>> git bisect bad 13b29cc3a722c2c0bc9ab9f72f9047d55d08a2f9
>> # bad: [d02117f8efaa5fbc37437df1ae955a147a2a424a] drm/ttm: remove special
> handling for non GEM drivers
>> git bisect bad d02117f8efaa5fbc37437df1ae955a147a2a424a
>> # good: [13ea9aa1e7d891e950230e82f1dd2c84e5debcff] drm/ttm: fix error
> handling if no BO can be swapped out v4
>> git bisect good 13ea9aa1e7d891e950230e82f1dd2c84e5debcff
>> # first bad commit: [d02117f8efaa5fbc37437df1ae955a147a2a424a] drm/ttm:
> remove special handling for non GEM drivers
>> Reverting the patch permit to have nouveau works again.
>>
>> Regards
>>
>>
>
>
>
Thanks Christian,
That patch fixed it for me, so I guess it just hasn't quite made it upstream
yet.
- Alistair
On Tuesday, 6 July 2021 4:58:37 PM AEST Christian K?nig wrote:
> Hi guys,
>
> yes nouveau was using the same functionality for internal BOs without
> noticing it. This is fixes by the following commit:
>
> commit d098775ed44021293b1962dea61efb19297b8d02
> Author: Christian K?nig <[email protected]>
> Date: Wed Jun 9 19:25:56 2021 +0200
>
> drm/nouveau: init the base GEM fields for internal BOs
>
> TTMs buffer objects are based on GEM objects for quite a while
> and rely on initializing those fields before initializing the TTM BO.
>
> Nouveau now doesn't init the GEM object for internally allocated BOs,
> so make sure that we at least initialize some necessary fields.
>
> Could be that the patch needs to be send to stable as well.
>
> Regards,
> Christian.
>
> Am 06.07.21 um 04:44 schrieb Alistair Popple:
> > I am also hitting this with upstream. Reverting d02117f8efaa ("drm/ttm: remove
> > special handling for non GEM drivers") also fixed it for me.
> >
> > The change log for that commit reads:
> >
> > drm/ttm: remove special handling for non GEM drivers
> >
> > vmwgfx is the only driver actually using this. Move the handling into
> > the driver instead.
> >
> > I wonder if Nouveau might actually have been using this somehow too?
> >
> > - Alistair
> >
> > On Sunday, 4 July 2021 5:21:29 AM AEST Corentin Labbe wrote:
> >> Hello
> >>
> >> Since some days on next, nouveau fail to load:
> >> [ 2.754087] nouveau 0000:02:00.0: vgaarb: deactivate vga console
> >> [ 2.761260] Console: switching to colour dummy device 80x25
> >> [ 2.766888] nouveau 0000:02:00.0: NVIDIA MCP77/MCP78 (0aa480a2)
> >> [ 2.783954] nouveau 0000:02:00.0: bios: version 62.77.2a.00.04
> >> [ 2.810122] nouveau 0000:02:00.0: fb: 256 MiB stolen system memory
> >> [ 3.484031] nouveau 0000:02:00.0: DRM: VRAM: 256 MiB
> >> [ 3.488993] nouveau 0000:02:00.0: DRM: GART: 1048576 MiB
> >> [ 3.494308] nouveau 0000:02:00.0: DRM: TMDS table version 2.0
> >> [ 3.500052] nouveau 0000:02:00.0: DRM: DCB version 4.0
> >> [ 3.505192] nouveau 0000:02:00.0: DRM: DCB outp 00: 01000300 0000001e
> >> [ 3.511632] nouveau 0000:02:00.0: DRM: DCB outp 01: 01011332 00020010
> >> [ 3.518074] nouveau 0000:02:00.0: DRM: DCB conn 00: 00000100
> >> [ 3.523728] nouveau 0000:02:00.0: DRM: DCB conn 01: 00001261
> >> [ 3.529455] nouveau 0000:02:00.0: DRM: failed to initialise sync
> > subsystem, -28
> >> [ 3.545946] nouveau: probe of 0000:02:00.0 failed with error -28
> >>
> >> I bisected it to:
> >> git bisect start
> >> # good: [62fb9874f5da54fdb243003b386128037319b219] Linux 5.13
> >> git bisect good 62fb9874f5da54fdb243003b386128037319b219
> >> # bad: [fb0ca446157a86b75502c1636b0d81e642fe6bf1] Add linux-next specific
> > files for 20210701
> >> git bisect bad fb0ca446157a86b75502c1636b0d81e642fe6bf1
> >> # good: [f63c4fda987a19b1194cc45cb72fd5bf968d9d90] Merge remote-tracking
> > branch 'rdma/for-next'
> >> git bisect good f63c4fda987a19b1194cc45cb72fd5bf968d9d90
> >> # bad: [49c8769be0b910d4134eba07cae5d9c71b861c4a] Merge remote-tracking
> > branch 'drm/drm-next'
> >> git bisect bad 49c8769be0b910d4134eba07cae5d9c71b861c4a
> >> # good: [4e3db44a242a4e2afe33b59793898ecbb61d478e] Merge tag 'wireless-
> > drivers-next-2021-06-25' of git://git.kernel.org/pub/scm/linux/kernel/git/
> > kvalo/wireless-drivers-next
> >> git bisect good 4e3db44a242a4e2afe33b59793898ecbb61d478e
> >> # bad: [5745d647d5563d3e9d32013ad4e5c629acff04d7] Merge tag 'amd-drm-
> > next-5.14-2021-06-02' of https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.freedesktop.org%2Fagd5f%2Flinux&data=04%7C01%7Cchristian.koenig%40amd.com%7C5f05fa59cd3b4432e71108d94027ede6%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637611362989756089%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=gJ98NtSRf3IyVfsCzw3DKydeMTGKIkHJNzUUhUfsWzY%3D&reserved=0 into drm-
> > next
> >> git bisect bad 5745d647d5563d3e9d32013ad4e5c629acff04d7
> >> # bad: [c99c4d0ca57c978dcc2a2f41ab8449684ea154cc] Merge tag 'amd-drm-
> > next-5.14-2021-05-19' of https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.freedesktop.org%2Fagd5f%2Flinux&data=04%7C01%7Cchristian.koenig%40amd.com%7C5f05fa59cd3b4432e71108d94027ede6%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637611362989756089%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=gJ98NtSRf3IyVfsCzw3DKydeMTGKIkHJNzUUhUfsWzY%3D&reserved=0 into drm-
> > next
> >> git bisect bad c99c4d0ca57c978dcc2a2f41ab8449684ea154cc
> >> # bad: [ae25ec2fc6c5a9e5767bf1922cd648501d0f914c] Merge tag 'drm-misc-
> > next-2021-05-17' of git://anongit.freedesktop.org/drm/drm-misc into drm-next
> >> git bisect bad ae25ec2fc6c5a9e5767bf1922cd648501d0f914c
> >> # bad: [cac80e71cfb0b00202d743c6e90333c45ba77cc5] drm/vkms: rename cursor to
> > plane on ops of planes composition
> >> git bisect bad cac80e71cfb0b00202d743c6e90333c45ba77cc5
> >> # good: [178bdba84c5f0ad14de384fc7f15fba0e272919d] drm/ttm/ttm_device:
> > Demote kernel-doc abuses
> >> git bisect good 178bdba84c5f0ad14de384fc7f15fba0e272919d
> >> # bad: [3f3a6524f6065fd3d130515e012f63eac74d96da] drm/dp: Clarify DP AUX
> > registration time
> >> git bisect bad 3f3a6524f6065fd3d130515e012f63eac74d96da
> >> # bad: [6dd7efc437611db16d432e0030f72d0c7e890127] drm/gud: cleanup coding
> > style a bit
> >> git bisect bad 6dd7efc437611db16d432e0030f72d0c7e890127
> >> # bad: [13b29cc3a722c2c0bc9ab9f72f9047d55d08a2f9] drm/mxsfb: Don't select
> > DRM_KMS_FB_HELPER
> >> git bisect bad 13b29cc3a722c2c0bc9ab9f72f9047d55d08a2f9
> >> # bad: [d02117f8efaa5fbc37437df1ae955a147a2a424a] drm/ttm: remove special
> > handling for non GEM drivers
> >> git bisect bad d02117f8efaa5fbc37437df1ae955a147a2a424a
> >> # good: [13ea9aa1e7d891e950230e82f1dd2c84e5debcff] drm/ttm: fix error
> > handling if no BO can be swapped out v4
> >> git bisect good 13ea9aa1e7d891e950230e82f1dd2c84e5debcff
> >> # first bad commit: [d02117f8efaa5fbc37437df1ae955a147a2a424a] drm/ttm:
> > remove special handling for non GEM drivers
> >> Reverting the patch permit to have nouveau works again.
> >>
> >> Regards
> >>
> >>
> >
> >
> >
>
>
On Tue, Jul 06, 2021 at 08:58:37AM +0200, Christian K?nig wrote:
> Hi guys,
>
> yes nouveau was using the same functionality for internal BOs without
> noticing it. This is fixes by the following commit:
>
> commit d098775ed44021293b1962dea61efb19297b8d02
> Author: Christian K?nig <[email protected]>
> Date:?? Wed Jun 9 19:25:56 2021 +0200
>
> ??? drm/nouveau: init the base GEM fields for internal BOs
>
> ??? TTMs buffer objects are based on GEM objects for quite a while
> ??? and rely on initializing those fields before initializing the TTM BO.
>
> ??? Nouveau now doesn't init the GEM object for internally allocated BOs,
> ??? so make sure that we at least initialize some necessary fields.
>
> Could be that the patch needs to be send to stable as well.
The regression is present in v5.14-rc1. Any idea when it will hit
upstream? I don't see it being applied to drm=next.
--
Kirill A. Shutemov
Am 14.07.21 um 14:56 schrieb Kirill A. Shutemov:
> On Tue, Jul 06, 2021 at 08:58:37AM +0200, Christian König wrote:
>> Hi guys,
>>
>> yes nouveau was using the same functionality for internal BOs without
>> noticing it. This is fixes by the following commit:
>>
>> commit d098775ed44021293b1962dea61efb19297b8d02
>> Author: Christian König <[email protected]>
>> Date: Wed Jun 9 19:25:56 2021 +0200
>>
>> drm/nouveau: init the base GEM fields for internal BOs
>>
>> TTMs buffer objects are based on GEM objects for quite a while
>> and rely on initializing those fields before initializing the TTM BO.
>>
>> Nouveau now doesn't init the GEM object for internally allocated BOs,
>> so make sure that we at least initialize some necessary fields.
>>
>> Could be that the patch needs to be send to stable as well.
> The regression is present in v5.14-rc1. Any idea when it will hit
> upstream? I don't see it being applied to drm=next.
Well that question needs to answer Dave or somebody else from the
drm-misc maintainer team.
This fix together with some others are already in drm-misc-next-fixes
waiting to be pushed upstream, but it looks like that hasn't happened yet.
Even Linus already pinged me where the fix for qxl got stuck.
Regards,
Christian.
On Wed, Jul 14, 2021 at 03:02:21PM +0200, Christian K?nig wrote:
> Am 14.07.21 um 14:56 schrieb Kirill A. Shutemov:
> > On Tue, Jul 06, 2021 at 08:58:37AM +0200, Christian K?nig wrote:
> > > Hi guys,
> > >
> > > yes nouveau was using the same functionality for internal BOs without
> > > noticing it. This is fixes by the following commit:
> > >
> > > commit d098775ed44021293b1962dea61efb19297b8d02
> > > Author: Christian K?nig <[email protected]>
> > > Date:?? Wed Jun 9 19:25:56 2021 +0200
> > >
> > > ??? drm/nouveau: init the base GEM fields for internal BOs
> > >
> > > ??? TTMs buffer objects are based on GEM objects for quite a while
> > > ??? and rely on initializing those fields before initializing the TTM BO.
> > >
> > > ??? Nouveau now doesn't init the GEM object for internally allocated BOs,
> > > ??? so make sure that we at least initialize some necessary fields.
> > >
> > > Could be that the patch needs to be send to stable as well.
> > The regression is present in v5.14-rc1. Any idea when it will hit
> > upstream? I don't see it being applied to drm=next.
>
> Well that question needs to answer Dave or somebody else from the drm-misc
> maintainer team.
>
> This fix together with some others are already in drm-misc-next-fixes
> waiting to be pushed upstream, but it looks like that hasn't happened yet.
>
> Even Linus already pinged me where the fix for qxl got stuck.
Yeah there was some missed patches. drm-misc-fixes is now in drm-fixes,
and drm-misc-next-fixes is included in drm-misc-fixes, for which Thomas
will do a pull request on Thu so it will land in -rc2.
It should also now be in linux-next.
But yes somehow bugfixes got a bit lost during the merge window.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch