Re: [PATCH] drm/mm: revert "Break long searches in fragmented address spaces"

2020-04-06 Thread Christian König
[Adding Nirmoy, setting bunch of people to BCC] This bubbled up in our internal testing as well. Nirmoy now wants to take a look at it. Am 01.04.20 um 11:17 schrieb Christian König: Am 01.04.20 um 10:53 schrieb Chris Wilson: Quoting Christian König (2020-04-01 08:29:34) Am 31.03.20 um

Re: [PATCH] drm/mm: revert "Break long searches in fragmented address spaces"

2020-04-01 Thread Christian König
Am 01.04.20 um 10:53 schrieb Chris Wilson: Quoting Christian König (2020-04-01 08:29:34) Am 31.03.20 um 15:19 schrieb Chris Wilson: Quoting Daniel Vetter (2020-03-31 11:38:50) On Tue, Mar 31, 2020 at 11:20 AM Chris Wilson wrote: Quoting Daniel Vetter (2020-03-31 10:16:18) On Tue, Mar 31,

Re: [PATCH] drm/mm: revert "Break long searches in fragmented address spaces"

2020-04-01 Thread Chris Wilson
Quoting Christian König (2020-04-01 08:29:34) > Am 31.03.20 um 15:19 schrieb Chris Wilson: > > Quoting Daniel Vetter (2020-03-31 11:38:50) > >> On Tue, Mar 31, 2020 at 11:20 AM Chris Wilson > >> wrote: > >>> Quoting Daniel Vetter (2020-03-31 10:16:18) > On Tue, Mar 31, 2020 at 10:59:45AM

Re: [PATCH] drm/mm: revert "Break long searches in fragmented address spaces"

2020-04-01 Thread Christian König
Am 31.03.20 um 15:19 schrieb Chris Wilson: Quoting Daniel Vetter (2020-03-31 11:38:50) On Tue, Mar 31, 2020 at 11:20 AM Chris Wilson wrote: Quoting Daniel Vetter (2020-03-31 10:16:18) On Tue, Mar 31, 2020 at 10:59:45AM +0200, Christian König wrote: A not so gentle ping, since this pretty

Re: [PATCH] drm/mm: revert "Break long searches in fragmented address spaces"

2020-03-31 Thread Chris Wilson
Quoting Daniel Vetter (2020-03-31 11:38:50) > On Tue, Mar 31, 2020 at 11:20 AM Chris Wilson > wrote: > > Quoting Daniel Vetter (2020-03-31 10:16:18) > > > On Tue, Mar 31, 2020 at 10:59:45AM +0200, Christian König wrote: > > > > A not so gentle ping, since this pretty much broke all TTM based >

Re: [PATCH] drm/mm: revert "Break long searches in fragmented address spaces"

2020-03-31 Thread Christian König
Am 31.03.20 um 12:38 schrieb Daniel Vetter: On Tue, Mar 31, 2020 at 11:20 AM Chris Wilson wrote: Quoting Daniel Vetter (2020-03-31 10:16:18) On Tue, Mar 31, 2020 at 10:59:45AM +0200, Christian König wrote: A not so gentle ping, since this pretty much broke all TTM based drivers. Could we

Re: [PATCH] drm/mm: revert "Break long searches in fragmented address spaces"

2020-03-31 Thread Daniel Vetter
On Tue, Mar 31, 2020 at 11:20 AM Chris Wilson wrote: > Quoting Daniel Vetter (2020-03-31 10:16:18) > > On Tue, Mar 31, 2020 at 10:59:45AM +0200, Christian König wrote: > > > A not so gentle ping, since this pretty much broke all TTM based drivers. > > > > > > Could we revert this for now? > > > >

Re: [PATCH] drm/mm: revert "Break long searches in fragmented address spaces"

2020-03-31 Thread Chris Wilson
Quoting Daniel Vetter (2020-03-31 10:16:18) > On Tue, Mar 31, 2020 at 10:59:45AM +0200, Christian König wrote: > > A not so gentle ping, since this pretty much broke all TTM based drivers. > > > > Could we revert this for now? > > Always ack for revert. > > Acked-by: Daniel Vetter So you

Re: [PATCH] drm/mm: revert "Break long searches in fragmented address spaces"

2020-03-31 Thread Chris Wilson
Quoting Christian König (2020-03-31 09:59:45) > A not so gentle ping, since this pretty much broke all TTM based drivers. > > Could we revert this for now? Ping??? https://patchwork.freedesktop.org/patch/358535/?series=74984=1 -Chris ___ dri-devel

Re: [PATCH] drm/mm: revert "Break long searches in fragmented address spaces"

2020-03-31 Thread Daniel Vetter
On Tue, Mar 31, 2020 at 10:59:45AM +0200, Christian König wrote: > A not so gentle ping, since this pretty much broke all TTM based drivers. > > Could we revert this for now? Always ack for revert. Acked-by: Daniel Vetter Needs to go to drm-misc-next-fixes, and then maybe also ask for a

Re: [PATCH] drm/mm: revert "Break long searches in fragmented address spaces"

2020-03-31 Thread Christian König
A not so gentle ping, since this pretty much broke all TTM based drivers. Could we revert this for now? Thanks, Christian. Am 30.03.20 um 14:34 schrieb Christian König: This reverts commit 7be1b9b8e9d1e9ef0342d2e001f44eec4030aa4d. The drm_mm is supposed to work in atomic context, so calling

Re: [PATCH] drm/mm: revert "Break long searches in fragmented address spaces"

2020-03-30 Thread Chris Wilson
Quoting Christian König (2020-03-30 13:34:25) > This reverts commit 7be1b9b8e9d1e9ef0342d2e001f44eec4030aa4d. > > The drm_mm is supposed to work in atomic context, so calling schedule() > or in this case cond_resched() is illegal. https://patchwork.freedesktop.org/patch/358535/?series=74984=1

[PATCH] drm/mm: revert "Break long searches in fragmented address spaces"

2020-03-30 Thread Christian König
This reverts commit 7be1b9b8e9d1e9ef0342d2e001f44eec4030aa4d. The drm_mm is supposed to work in atomic context, so calling schedule() or in this case cond_resched() is illegal. Signed-off-by: Christian König --- drivers/gpu/drm/drm_mm.c | 8 +--- 1 file changed, 1 insertion(+), 7