Re: [PATCH] drm/ttm: don't set page->mapping

2020-11-26 Thread Jason Gunthorpe
On Wed, Nov 25, 2020 at 05:28:32PM +0100, Daniel Vetter wrote: > On Wed, Nov 25, 2020 at 5:25 PM Daniel Vetter wrote: > > > > Random observation while trying to review Christian's patch series to > > stop looking at struct page for dma-buf imports. > > > > This was originally added in > > > >

Re: [PATCH] drm/ttm: don't set page->mapping

2020-11-26 Thread Jason Gunthorpe
On Wed, Nov 25, 2020 at 06:11:29PM +, Christoph Hellwig wrote: > On Wed, Nov 25, 2020 at 02:06:06PM -0400, Jason Gunthorpe wrote: > > It uses a empty 'cover-letter' commit and automatically transforms it > > into exactly the right stuff. Keeps track of everything you send in > > git, and there

Re: [PATCH] drm/ttm: don't set page->mapping

2020-11-25 Thread Daniel Stone
Hi, On Wed, 25 Nov 2020 at 18:06, Jason Gunthorpe wrote: > On Wed, Nov 25, 2020 at 05:28:32PM +0100, Daniel Vetter wrote: > > Apologies again, this shouldn't have been included. But at least I > > have an idea now why this patch somehow was included in the git > > send-email. Lovely interface

Re: [PATCH] drm/ttm: don't set page->mapping

2020-11-25 Thread Daniel Vetter
On Wed, Nov 25, 2020 at 5:25 PM Daniel Vetter wrote: > > Random observation while trying to review Christian's patch series to > stop looking at struct page for dma-buf imports. > > This was originally added in > > commit 58aa6622d32af7d2c08d45085f44c54554a16ed7 > Author: Thomas Hellstrom >

[PATCH] drm/ttm: don't set page->mapping

2020-11-25 Thread Daniel Vetter
Random observation while trying to review Christian's patch series to stop looking at struct page for dma-buf imports. This was originally added in commit 58aa6622d32af7d2c08d45085f44c54554a16ed7 Author: Thomas Hellstrom Date: Fri Jan 3 11:47:23 2014 +0100 drm/ttm: Correctly set page

Re: [PATCH] drm/ttm: don't set page->mapping

2020-11-20 Thread Daniel Vetter
On Fri, Nov 20, 2020 at 11:08:31AM +0100, Christian König wrote: > Am 20.11.20 um 11:05 schrieb Daniel Vetter: > > On Fri, Nov 20, 2020 at 11:04 AM Christian König > > wrote: > > > Am 20.11.20 um 10:54 schrieb Daniel Vetter: > > > > Random observation while trying to review Christian's patch

Re: [PATCH] drm/ttm: don't set page->mapping

2020-11-20 Thread Christian König
Am 20.11.20 um 11:05 schrieb Daniel Vetter: On Fri, Nov 20, 2020 at 11:04 AM Christian König wrote: Am 20.11.20 um 10:54 schrieb Daniel Vetter: Random observation while trying to review Christian's patch series to stop looking at struct page for dma-buf imports. This was originally added in

Re: [PATCH] drm/ttm: don't set page->mapping

2020-11-20 Thread Daniel Vetter
On Fri, Nov 20, 2020 at 11:04 AM Christian König wrote: > > Am 20.11.20 um 10:54 schrieb Daniel Vetter: > > Random observation while trying to review Christian's patch series to > > stop looking at struct page for dma-buf imports. > > > > This was originally added in > > > > commit

Re: [PATCH] drm/ttm: don't set page->mapping

2020-11-20 Thread Christian König
Am 20.11.20 um 10:54 schrieb Daniel Vetter: Random observation while trying to review Christian's patch series to stop looking at struct page for dma-buf imports. This was originally added in commit 58aa6622d32af7d2c08d45085f44c54554a16ed7 Author: Thomas Hellstrom Date: Fri Jan 3 11:47:23

[PATCH] drm/ttm: don't set page->mapping

2020-11-20 Thread Daniel Vetter
Random observation while trying to review Christian's patch series to stop looking at struct page for dma-buf imports. This was originally added in commit 58aa6622d32af7d2c08d45085f44c54554a16ed7 Author: Thomas Hellstrom Date: Fri Jan 3 11:47:23 2014 +0100 drm/ttm: Correctly set page

Re: [PATCH] drm/ttm: don't set page->mapping

2020-11-06 Thread Christian König
Am 05.11.20 um 17:37 schrieb Daniel Vetter: [SNIP] NAK, we use this to determine if a pages belongs to the driver or not in amdgpu for example. Mostly used for debugging, but I would really like to keep that. Can you pls point me at that code? A quick grep hasn't really found much at all.

Re: [PATCH] drm/ttm: don't set page->mapping

2020-11-05 Thread Daniel Vetter
On Thu, Nov 5, 2020 at 4:15 PM Christian König wrote: > > Am 05.11.20 um 15:38 schrieb Daniel Vetter: > > On Thu, Nov 5, 2020 at 3:31 PM Daniel Vetter wrote: > >> On Thu, Nov 5, 2020 at 2:22 PM Christian König > >> wrote: > >>> Am 05.11.20 um 14:20 schrieb Daniel Vetter: > On Thu, Nov 05,

Re: [PATCH] drm/ttm: don't set page->mapping

2020-11-05 Thread Christian König
Am 05.11.20 um 15:38 schrieb Daniel Vetter: On Thu, Nov 5, 2020 at 3:31 PM Daniel Vetter wrote: On Thu, Nov 5, 2020 at 2:22 PM Christian König wrote: Am 05.11.20 um 14:20 schrieb Daniel Vetter: On Thu, Nov 05, 2020 at 01:56:22PM +0100, Christian König wrote: Am 05.11.20 um 13:50 schrieb

Re: [PATCH] drm/ttm: don't set page->mapping

2020-11-05 Thread Daniel Vetter
On Thu, Nov 5, 2020 at 3:31 PM Daniel Vetter wrote: > > On Thu, Nov 5, 2020 at 2:22 PM Christian König > wrote: > > > > Am 05.11.20 um 14:20 schrieb Daniel Vetter: > > > On Thu, Nov 05, 2020 at 01:56:22PM +0100, Christian König wrote: > > >> Am 05.11.20 um 13:50 schrieb Daniel Vetter: > > >>>

Re: [PATCH] drm/ttm: don't set page->mapping

2020-11-05 Thread Daniel Vetter
On Thu, Nov 5, 2020 at 2:22 PM Christian König wrote: > > Am 05.11.20 um 14:20 schrieb Daniel Vetter: > > On Thu, Nov 05, 2020 at 01:56:22PM +0100, Christian König wrote: > >> Am 05.11.20 um 13:50 schrieb Daniel Vetter: > >>> On Thu, Nov 05, 2020 at 01:29:50PM +0100, Christian König wrote: >

Re: [PATCH] drm/ttm: don't set page->mapping

2020-11-05 Thread Christian König
Am 05.11.20 um 14:20 schrieb Daniel Vetter: On Thu, Nov 05, 2020 at 01:56:22PM +0100, Christian König wrote: Am 05.11.20 um 13:50 schrieb Daniel Vetter: On Thu, Nov 05, 2020 at 01:29:50PM +0100, Christian König wrote: Am 05.11.20 um 10:11 schrieb Daniel Vetter: On Thu, Nov 5, 2020 at 9:00 AM

Re: [PATCH] drm/ttm: don't set page->mapping

2020-11-05 Thread Daniel Vetter
On Thu, Nov 05, 2020 at 01:56:22PM +0100, Christian König wrote: > Am 05.11.20 um 13:50 schrieb Daniel Vetter: > > On Thu, Nov 05, 2020 at 01:29:50PM +0100, Christian König wrote: > > > Am 05.11.20 um 10:11 schrieb Daniel Vetter: > > > > On Thu, Nov 5, 2020 at 9:00 AM Christian König > > > >

Re: [PATCH] drm/ttm: don't set page->mapping

2020-11-05 Thread Christian König
Am 05.11.20 um 13:50 schrieb Daniel Vetter: On Thu, Nov 05, 2020 at 01:29:50PM +0100, Christian König wrote: Am 05.11.20 um 10:11 schrieb Daniel Vetter: On Thu, Nov 5, 2020 at 9:00 AM Christian König wrote: Am 04.11.20 um 17:50 schrieb Daniel Vetter: Random observation while trying to

Re: [PATCH] drm/ttm: don't set page->mapping

2020-11-05 Thread Daniel Vetter
On Thu, Nov 05, 2020 at 01:29:50PM +0100, Christian König wrote: > Am 05.11.20 um 10:11 schrieb Daniel Vetter: > > On Thu, Nov 5, 2020 at 9:00 AM Christian König > > wrote: > > > Am 04.11.20 um 17:50 schrieb Daniel Vetter: > > > > Random observation while trying to review Christian's patch

Re: [PATCH] drm/ttm: don't set page->mapping

2020-11-05 Thread Christian König
Am 05.11.20 um 10:11 schrieb Daniel Vetter: On Thu, Nov 5, 2020 at 9:00 AM Christian König wrote: Am 04.11.20 um 17:50 schrieb Daniel Vetter: Random observation while trying to review Christian's patch series to stop looking at struct page for dma-buf imports. This was originally added in

Re: [PATCH] drm/ttm: don't set page->mapping

2020-11-05 Thread Daniel Vetter
On Thu, Nov 5, 2020 at 9:00 AM Christian König wrote: > > Am 04.11.20 um 17:50 schrieb Daniel Vetter: > > Random observation while trying to review Christian's patch series to > > stop looking at struct page for dma-buf imports. > > > > This was originally added in > > > > commit

Re: [PATCH] drm/ttm: don't set page->mapping

2020-11-05 Thread Christian König
Am 04.11.20 um 17:50 schrieb Daniel Vetter: Random observation while trying to review Christian's patch series to stop looking at struct page for dma-buf imports. This was originally added in commit 58aa6622d32af7d2c08d45085f44c54554a16ed7 Author: Thomas Hellstrom Date: Fri Jan 3 11:47:23

[PATCH] drm/ttm: don't set page->mapping

2020-11-04 Thread Daniel Vetter
Random observation while trying to review Christian's patch series to stop looking at struct page for dma-buf imports. This was originally added in commit 58aa6622d32af7d2c08d45085f44c54554a16ed7 Author: Thomas Hellstrom Date: Fri Jan 3 11:47:23 2014 +0100 drm/ttm: Correctly set page