[PATCH 3/3] Revert "drm/ttm: callback move_notify any time bo placement change v4"

2012-01-05 Thread Jerome Glisse
On Wed, Jan 04, 2012 at 04:35:16PM -0500, Konrad Rzeszutek Wilk wrote: > This reverts commit 5c2680ddbe14b24771d24841dd66881f90d3d740 otherwise > when we unbind the device we get this: > sh-4.1# echo ":00:0d.0" > unbind > BUG: unable to handle kernel NULL pointer dereference at (null

Re: [PATCH 3/3] Revert "drm/ttm: callback move_notify any time bo placement change v4"

2012-01-05 Thread Jerome Glisse
On Wed, Jan 04, 2012 at 04:35:16PM -0500, Konrad Rzeszutek Wilk wrote: > This reverts commit 5c2680ddbe14b24771d24841dd66881f90d3d740 otherwise > when we unbind the device we get this: > sh-4.1# echo ":00:0d.0" > unbind > BUG: unable to handle kernel NULL pointer dereference at (null

[PATCH 3/3] Revert "drm/ttm: callback move_notify any time bo placement change v4"

2012-01-04 Thread Konrad Rzeszutek Wilk
This reverts commit 5c2680ddbe14b24771d24841dd66881f90d3d740 otherwise when we unbind the device we get this: sh-4.1# echo ":00:0d.0" > unbind BUG: unable to handle kernel NULL pointer dereference at (null) IP: [] [] nouveau_bo_move_ntfy+0x1d/0xc0 [nouveau] RSP: 0018:88012deb

[PATCH 3/3] Revert "drm/ttm: callback move_notify any time bo placement change v4"

2012-01-04 Thread Konrad Rzeszutek Wilk
This reverts commit 5c2680ddbe14b24771d24841dd66881f90d3d740 otherwise when we unbind the device we get this: sh-4.1# echo ":00:0d.0" > unbind BUG: unable to handle kernel NULL pointer dereference at (null) IP: [] [] nouveau_bo_move_ntfy+0x1d/0xc0 [nouveau] RSP: 0018:88012deb

crash with "drm/ttm: callback move_notify any time bo placement change v4" patch

2011-12-12 Thread Konrad Rzeszutek Wilk
Hey, When I use the drm-next tree without the patch it works fine. (albeit slowly - but I posted the patches for that). With the patch mentioned I get this: 00:0d.0 VGA compatible controller: nVidia Corporation C61 [GeForce 6150SE nForce 430] (rev a2) sh-4.1# cd :00:0d.0 sh-4.1# cd driver

crash with "drm/ttm: callback move_notify any time bo placement change v4" patch

2011-12-12 Thread Konrad Rzeszutek Wilk
Hey, When I use the drm-next tree without the patch it works fine. (albeit slowly - but I posted the patches for that). With the patch mentioned I get this: 00:0d.0 VGA compatible controller: nVidia Corporation C61 [GeForce 6150SE nForce 430] (rev a2) sh-4.1# cd :00:0d.0 sh-4.1# cd driver

drm/ttm: callback move_notify any time bo placement

2011-12-07 Thread Dan Carpenter
change v4 Hello Jerome Glisse, This is a semi-automatic email about new static checker warnings. The patch dc97b3409a79: "drm/ttm: callback move_notify any time bo placement change v4" from Nov 18, 2011, leads to the following Smatch complaint: drivers/gpu/drm/nouveau/nouveau

re: drm/ttm: callback move_notify any time bo placement

2011-12-07 Thread Dan Carpenter
change v4 Hello Jerome Glisse, This is a semi-automatic email about new static checker warnings. The patch dc97b3409a79: "drm/ttm: callback move_notify any time bo placement change v4" from Nov 18, 2011, leads to the following Smatch complaint: drivers/gpu/drm/nouveau/nouveau

[PATCH] drm/ttm: callback move_notify any time bo placement change v3

2011-12-01 Thread Thomas Hellstrom
Updated patch Reviewed-by: Thomas Hellstrom On 11/20/2011 10:02 PM, Jerome Glisse wrote: > On Sat, Nov 19, 2011 at 3:45 PM, Thomas Hellstrom > wrote: > >> On 11/19/2011 12:32 AM, j.glisse at gmail.com wrote: >> >>> From: Jerome Glisse >>> >>> Previously we were calling back move_not

Re: [PATCH] drm/ttm: callback move_notify any time bo placement change v3

2011-11-30 Thread Thomas Hellstrom
Updated patch Reviewed-by: Thomas Hellstrom On 11/20/2011 10:02 PM, Jerome Glisse wrote: On Sat, Nov 19, 2011 at 3:45 PM, Thomas Hellstrom wrote: On 11/19/2011 12:32 AM, j.gli...@gmail.com wrote: From: Jerome Glisse Previously we were calling back move_notify in error path when

[PATCH] drm/ttm: callback move_notify any time bo placement change v3

2011-11-20 Thread Jerome Glisse
? ? ? bdev->driver->move_notify(bo, mem); >> + >> > >> ?moved: >> ? ? ? ?if (bo->evicted) { >> ? ? ? ? ? ? ? ?ret = bdev->driver->invalidate_caches(bdev, >> bo->mem.placement); >> @@ -1872,9 +1878,12 @@ static int ttm_bo_swapout(struct

Re: [PATCH] drm/ttm: callback move_notify any time bo placement change v3

2011-11-20 Thread Jerome Glisse
  bdev->driver->move_notify(bo, mem); >> + >> > >>  moved: >>        if (bo->evicted) { >>                ret = bdev->driver->invalidate_caches(bdev, >> bo->mem.placement); >> @@ -1872,9 +1878,12 @@ static int ttm_bo_swapout(struct

[PATCH] drm/ttm: callback move_notify any time bo placement change v3

2011-11-19 Thread Thomas Hellstrom
On 11/19/2011 12:32 AM, j.glisse at gmail.com wrote: > From: Jerome Glisse > > Previously we were calling back move_notify in error path when the > bo is returned to it's original position or when destroy the bo. > When destroying the bo set the new mem placement as NULL when calling > back in the

Re: [PATCH] drm/ttm: callback move_notify any time bo placement change v3

2011-11-19 Thread Thomas Hellstrom
On 11/19/2011 12:32 AM, j.gli...@gmail.com wrote: From: Jerome Glisse Previously we were calling back move_notify in error path when the bo is returned to it's original position or when destroy the bo. When destroying the bo set the new mem placement as NULL when calling back in the driver. Upd

[PATCH] drm/ttm: callback move_notify any time bo placement change v2

2011-11-18 Thread Thomas Hellstrom
On 11/18/2011 06:32 PM, j.glisse at gmail.com wrote: > From: Jerome Glisse > > Previously we were calling back move_notify in error path when the > bo is returned to it's original position or when destroy the bo. > When destroying the bo set the new mem placement as NULL when calling > back in the

[PATCH] drm/ttm: callback move_notify any time bo placement change v3

2011-11-18 Thread j.gli...@gmail.com
From: Jerome Glisse Previously we were calling back move_notify in error path when the bo is returned to it's original position or when destroy the bo. When destroying the bo set the new mem placement as NULL when calling back in the driver. Updating nouveau to deal with NULL placement properly.

[PATCH] drm/ttm: callback move_notify any time bo placement change v3

2011-11-18 Thread j . glisse
From: Jerome Glisse Previously we were calling back move_notify in error path when the bo is returned to it's original position or when destroy the bo. When destroying the bo set the new mem placement as NULL when calling back in the driver. Updating nouveau to deal with NULL placement properly.

Re: [PATCH] drm/ttm: callback move_notify any time bo placement change v2

2011-11-18 Thread Thomas Hellstrom
On 11/18/2011 06:32 PM, j.gli...@gmail.com wrote: From: Jerome Glisse Previously we were calling back move_notify in error path when the bo is returned to it's original position or when destroy the bo. When destroying the bo set the new mem placement as NULL when calling back in the driver. Upd

[PATCH] drm/ttm: callback move_notify any time bo placement change v2

2011-11-18 Thread j.gli...@gmail.com
From: Jerome Glisse Previously we were calling back move_notify in error path when the bo is returned to it's original position or when destroy the bo. When destroying the bo set the new mem placement as NULL when calling back in the driver. Updating nouveau to deal with NULL placement properly.

[PATCH] drm/ttm: callback move_notify any time bo placement change

2011-11-18 Thread j.gli...@gmail.com
From: Jerome Glisse Previously we were calling back move_notify in error path when the bo is returned to it's original position or when destroy the bo. When destroying the bo set the new mem placement as NULL when calling back in the driver. Updating nouveau to deal with NULL placement properly.

[PATCH] drm/ttm: callback move_notify any time bo placement change v2

2011-11-18 Thread j . glisse
From: Jerome Glisse Previously we were calling back move_notify in error path when the bo is returned to it's original position or when destroy the bo. When destroying the bo set the new mem placement as NULL when calling back in the driver. Updating nouveau to deal with NULL placement properly.

[PATCH] drm/ttm: callback move_notify any time bo placement change

2011-11-18 Thread j . glisse
From: Jerome Glisse Previously we were calling back move_notify in error path when the bo is returned to it's original position or when destroy the bo. When destroying the bo set the new mem placement as NULL when calling back in the driver. Updating nouveau to deal with NULL placement properly.