linux-next: manual merge of the drm-misc tree with the mm-stable tree
Subsystems (4)
[email protected]
maintainerHi all,
Today's linux-next merge of the drm-misc tree got a conflict in:
drivers/gpu/drm/ttm/ttm_pool.c
between commit:
23baf831a32c0 ("mm, treewide: redefine MAX_ORDER sanely")
from the mm-stable tree and commit:
56e51681246e5 ("drm/ttm: revert "Reduce the number of used allocation orders for TTM pages"")
from the drm-misc tree.
I fixed it up (see below) and can carry the fix as necessary. This is now fixed as far as linux-next is concerned, but any non trivial conflicts should be mentioned to your upstream maintainer when your tree is submitted for merging. You may also want to consider cooperating with the maintainer of the conflicting tree to minimise any particularly complex conflicts.
drivers/gpu/drm/ttm/ttm_pool.c +0 -0
[Just the version in mm]
Quoted 29 lines · On Fri, Apr 14, 2023 at 01:59:12PM +0100, [email protected] wrote:
Hi all,
Today's linux-next merge of the drm-misc tree got a conflict in:
drivers/gpu/drm/ttm/ttm_pool.cbetween commit:
23baf831a32c0("mm, treewide: redefineMAX_ORDERsanely")from the mm-stable tree and commit:
56e51681246e5("drm/ttm: revert "Reduce the number of used allocation orders for TTM pages"")from the drm-misc tree.
I fixed it up (see below) and can carry the fix as necessary. This is now fixed as far as linux-next is concerned, but any non trivial conflicts should be mentioned to your upstream maintainer when your tree is submitted for merging. You may also want to consider cooperating with the maintainer of the conflicting tree to minimise any particularly complex conflicts.
drivers/gpu/drm/ttm/ttm_pool.c +0
-0
Quoted 1 line
[Just the version in mm]
Note there was a ppc compile fail, which is why we pushed the ttm revert. That /should/ be fixed now, but would be good if you can confirm?
Thanks, Daniel
Signature · 3 lines
Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch
Mark Brown
maintainerQuoted 2 lines · Daniel Vetter
Note there was a ppc compile fail, which is why we pushed the ttm revert. That /should/ be fixed now, but would be good if you can confirm?
I don't have any PowerPC toolchains set up - I guess one of the community builders might have checked?
Mark Brown
maintainerQuoted 2 lines · Daniel Vetter
Note there was a ppc compile fail, which is why we pushed the ttm revert. That /should/ be fixed now, but would be good if you can confirm?
According to Nathan (CCed) there's still issues with the interaction with the PowerPC tree.
Quoted 7 lines · Mark Brown
Quoted 2 lines · Daniel Vetter
Note there was a ppc compile fail, which is why we pushed the ttm revert. That /should/ be fixed now, but would be good if you can confirm?
According to Nathan (CCed) there's still issues with the interaction with the PowerPC tree.
So this revert was supposed to fix this: 56e51681246e ("drm/ttm: revert "Reduce the number of used allocation orders for TTM pages"")
If there's anything left then I need to chase that asap since the merge window will open soon.
-Daniel
Signature · 3 lines
Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch
Quoted 6 lines · Daniel Vetter
Quoted 5 lines · Mark Brown
Quoted 2 lines · Daniel Vetter
Note there was a ppc compile fail, which is why we pushed the ttm revert. That /should/ be fixed now, but would be good if you can confirm?
Quoted 8 lines
Quoted 2 lines
According to Nathan (CCed) there's still issues with the interaction with the PowerPC tree.
So this revert was supposed to fix this:
56e51681246e("drm/ttm: revert "Reduce the number of used allocation orders for TTM pages"")If there's anything left then I need to chase that asap since the merge window will open soon.
I think we are talking about two different issues here. My issue is not a compilation failure, it is an incorrect merge resolution that is happening in -next because of two independent changes in the drm and powerpc tree, the thread below should have more information.
https://lore.kernel.org/[email protected]/
I do not think this is something that either tree can solve independently of each other, -next has to resolve the conflict correctly (which is what I point out in the message above) and a note of it should be passed along to Linus so it can be resolved correctly in mainline when the time comes.
Cheers,
Nathan
Quoted 21 lines · Nathan Chancellor
Quoted 6 lines · Daniel Vetter
Quoted 5 lines · Mark Brown
Quoted 2 lines · Daniel Vetter
Note there was a ppc compile fail, which is why we pushed the ttm revert. That /should/ be fixed now, but would be good if you can confirm?
Quoted 8 lines
Quoted 2 lines
According to Nathan (CCed) there's still issues with the interaction with the PowerPC tree.
So this revert was supposed to fix this:
56e51681246e("drm/ttm: revert "Reduce the number of used allocation orders for TTM pages"")If there's anything left then I need to chase that asap since the merge window will open soon.
I think we are talking about two different issues here. My issue is not a compilation failure, it is an incorrect merge resolution that is happening in -next because of two independent changes in the drm and powerpc tree, the thread below should have more information.
Quoted 7 lines
https://lore.kernel.org/[email protected]/
I do not think this is something that either tree can solve independently of each other, -next has to resolve the conflict correctly (which is what I point out in the message above) and a note of it should be passed along to Linus so it can be resolved correctly in mainline when the time comes.
Ah yes that's a different one. I think we have a note about this one already, but I'll double-check with Dave Airlie.
-Daniel
Signature · 3 lines
Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch