ttm: merge ttm_backend & ttm_tt, introduce ttm dma allocator V7
On Sat, Dec 3, 2011 at 4:41 PM, Jerome Glisse wrote: > On Sat, Dec 3, 2011 at 9:52 AM, David Airlie wrote: >> >> >> - Original Message - >>> From: "Konrad Rzeszutek Wilk" >>> To: "Jerome Glisse" , dri-devel at >>> lists.freedesktop.org, "konrad wilk" , >>> thellstrom at vmware.com, airlied at gmail.com, "Jerome Glisse" >> redhat.com> >>> Sent: Friday, 2 December, 2011 2:19:01 PM >>> Subject: Re: ttm: merge ttm_backend & ttm_tt, introduce ttm dma allocator V7 >>> >>> On Fri, Nov 18, 2011 at 08:08:36PM -0500, Jerome Glisse wrote: >>> > On Fri, Nov 18, 2011 at 08:04:47PM -0500, j.glisse at gmail.com >>> > wrote: >>> > > Important fix to patch 14, fix accounting of ghost bo. When >>> > > creating a >>> > > ghost bo we don't account it, so set its acc_size to 0 so that >>> > > when >>> > > ghost is release we don't overfree. >>> > > >>> > > I wonder how i didn't run into this before. >>> > > >>> > > Patch are also at >>> > > >>> > > http://people.freedesktop.org/~glisse/ttmdma/ >>> > > >>> > > Cheers, >>> > > Jerome >>> > > >>> > >>> > Oh i forgot to add some of the reviewed by, i updated patches on >>> > fdo, >>> > i don't resend to the ml. >>> >>> Great! How should we ask Dave to pull them? Does he prefer to do it >>> via git >>> tree? In which I can create a branch with those patches and send him >>> a GIT PULL >>> email? Or is there a more convient way? >> >> If someone could suck them all into a git tree + all the Reviewed-by tags >> from the people who reviewed them it would make it a lot easier, >> >> I lost track of where this ended up as Jerome had a few balls in the air and >> I know Thomas wasn't liking some of them. >> >> So please send me a git url and I'll go review that next week. >> >> Dave. > > http://cgit.freedesktop.org/~glisse/linux/log/ > > I need to add last reviewed by from Thomas. Note this tree also has > other patches (multi ring lastest patchset) which we want for next too > i believe. > > I will update this tree on monday with all the missing reviewed by > > Cheers, > Jerome Just to be extra informative my tree has: drm/ttm: callback move_notify any time bo placement change v4 Got the reviewed by recently from Thomas drm/ttm: simplify memory accounting for ttm user v2 Thomas reviewed version1, version2 only fix a bug so i think it's safe to assume i have reviewed by Thomas for v2. Cheers, Jerome
ttm: merge ttm_backend & ttm_tt, introduce ttm dma allocator V7
On Sat, Dec 3, 2011 at 9:52 AM, David Airlie wrote: > > > - Original Message - >> From: "Konrad Rzeszutek Wilk" >> To: "Jerome Glisse" , dri-devel at >> lists.freedesktop.org, "konrad wilk" , >> thellstrom at vmware.com, airlied at gmail.com, "Jerome Glisse" > redhat.com> >> Sent: Friday, 2 December, 2011 2:19:01 PM >> Subject: Re: ttm: merge ttm_backend & ttm_tt, introduce ttm dma allocator V7 >> >> On Fri, Nov 18, 2011 at 08:08:36PM -0500, Jerome Glisse wrote: >> > On Fri, Nov 18, 2011 at 08:04:47PM -0500, j.glisse at gmail.com >> > wrote: >> > > Important fix to patch 14, fix accounting of ghost bo. When >> > > creating a >> > > ghost bo we don't account it, so set its acc_size to 0 so that >> > > when >> > > ghost is release we don't overfree. >> > > >> > > I wonder how i didn't run into this before. >> > > >> > > Patch are also at >> > > >> > > http://people.freedesktop.org/~glisse/ttmdma/ >> > > >> > > Cheers, >> > > Jerome >> > > >> > >> > Oh i forgot to add some of the reviewed by, i updated patches on >> > fdo, >> > i don't resend to the ml. >> >> Great! How should we ask Dave to pull them? Does he prefer to do it >> via git >> tree? In which I can create a branch with those patches and send him >> a GIT PULL >> email? Or is there a more convient way? > > If someone could suck them all into a git tree + all the Reviewed-by tags > from the people who reviewed them it would make it a lot easier, > > I lost track of where this ended up as Jerome had a few balls in the air and > I know Thomas wasn't liking some of them. > > So please send me a git url and I'll go review that next week. > > Dave. http://cgit.freedesktop.org/~glisse/linux/log/ I need to add last reviewed by from Thomas. Note this tree also has other patches (multi ring lastest patchset) which we want for next too i believe. I will update this tree on monday with all the missing reviewed by Cheers, Jerome
ttm: merge ttm_backend & ttm_tt, introduce ttm dma allocator V7
- Original Message - > From: "Konrad Rzeszutek Wilk" > To: "Jerome Glisse" , dri-devel at > lists.freedesktop.org, "konrad wilk" , > thellstrom at vmware.com, airlied at gmail.com, "Jerome Glisse" redhat.com> > Sent: Friday, 2 December, 2011 2:19:01 PM > Subject: Re: ttm: merge ttm_backend & ttm_tt, introduce ttm dma allocator V7 > > On Fri, Nov 18, 2011 at 08:08:36PM -0500, Jerome Glisse wrote: > > On Fri, Nov 18, 2011 at 08:04:47PM -0500, j.glisse at gmail.com > > wrote: > > > Important fix to patch 14, fix accounting of ghost bo. When > > > creating a > > > ghost bo we don't account it, so set its acc_size to 0 so that > > > when > > > ghost is release we don't overfree. > > > > > > I wonder how i didn't run into this before. > > > > > > Patch are also at > > > > > > http://people.freedesktop.org/~glisse/ttmdma/ > > > > > > Cheers, > > > Jerome > > > > > > > Oh i forgot to add some of the reviewed by, i updated patches on > > fdo, > > i don't resend to the ml. > > Great! How should we ask Dave to pull them? Does he prefer to do it > via git > tree? In which I can create a branch with those patches and send him > a GIT PULL > email? Or is there a more convient way? If someone could suck them all into a git tree + all the Reviewed-by tags from the people who reviewed them it would make it a lot easier, I lost track of where this ended up as Jerome had a few balls in the air and I know Thomas wasn't liking some of them. So please send me a git url and I'll go review that next week. Dave.
Re: ttm: merge ttm_backend ttm_tt, introduce ttm dma allocator V7
On Fri, Nov 18, 2011 at 08:08:36PM -0500, Jerome Glisse wrote: On Fri, Nov 18, 2011 at 08:04:47PM -0500, j.glisse at gmail.com wrote: Important fix to patch 14, fix accounting of ghost bo. When creating a ghost bo we don't account it, so set its acc_size to 0 so that when ghost is release we don't overfree. I wonder how i didn't run into this before. Patch are also at http://people.freedesktop.org/~glisse/ttmdma/ Cheers, Jerome Oh i forgot to add some of the reviewed by, i updated patches on fdo, i don't resend to the ml. Great! How should we ask Dave to pull them? Does he prefer to do it via git tree? In which I can create a branch with those patches and send him a GIT PULL email? Or is there a more convient way? Cheers, Jerome ___ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: ttm: merge ttm_backend ttm_tt, introduce ttm dma allocator V7
On Sat, Dec 3, 2011 at 9:52 AM, David Airlie airl...@redhat.com wrote: - Original Message - From: Konrad Rzeszutek Wilk konrad.w...@oracle.com To: Jerome Glisse j.gli...@gmail.com, dri-devel@lists.freedesktop.org, konrad wilk konrad.w...@oracle.com, thellst...@vmware.com, airl...@gmail.com, Jerome Glisse jgli...@redhat.com Sent: Friday, 2 December, 2011 2:19:01 PM Subject: Re: ttm: merge ttm_backend ttm_tt, introduce ttm dma allocator V7 On Fri, Nov 18, 2011 at 08:08:36PM -0500, Jerome Glisse wrote: On Fri, Nov 18, 2011 at 08:04:47PM -0500, j.glisse at gmail.com wrote: Important fix to patch 14, fix accounting of ghost bo. When creating a ghost bo we don't account it, so set its acc_size to 0 so that when ghost is release we don't overfree. I wonder how i didn't run into this before. Patch are also at http://people.freedesktop.org/~glisse/ttmdma/ Cheers, Jerome Oh i forgot to add some of the reviewed by, i updated patches on fdo, i don't resend to the ml. Great! How should we ask Dave to pull them? Does he prefer to do it via git tree? In which I can create a branch with those patches and send him a GIT PULL email? Or is there a more convient way? If someone could suck them all into a git tree + all the Reviewed-by tags from the people who reviewed them it would make it a lot easier, I lost track of where this ended up as Jerome had a few balls in the air and I know Thomas wasn't liking some of them. So please send me a git url and I'll go review that next week. Dave. http://cgit.freedesktop.org/~glisse/linux/log/ I need to add last reviewed by from Thomas. Note this tree also has other patches (multi ring lastest patchset) which we want for next too i believe. I will update this tree on monday with all the missing reviewed by Cheers, Jerome ___ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel
ttm: merge ttm_backend & ttm_tt, introduce ttm dma allocator V7
On Fri, Nov 18, 2011 at 08:08:36PM -0500, Jerome Glisse wrote: > On Fri, Nov 18, 2011 at 08:04:47PM -0500, j.glisse at gmail.com wrote: > > Important fix to patch 14, fix accounting of ghost bo. When creating a > > ghost bo we don't account it, so set its acc_size to 0 so that when > > ghost is release we don't overfree. > > > > I wonder how i didn't run into this before. > > > > Patch are also at > > > > http://people.freedesktop.org/~glisse/ttmdma/ > > > > Cheers, > > Jerome > > > > Oh i forgot to add some of the reviewed by, i updated patches on fdo, > i don't resend to the ml. Great! How should we ask Dave to pull them? Does he prefer to do it via git tree? In which I can create a branch with those patches and send him a GIT PULL email? Or is there a more convient way? > Cheers, > Jerome
ttm: merge ttm_backend & ttm_tt, introduce ttm dma allocator V7
On Fri, Nov 18, 2011 at 08:04:47PM -0500, j.glisse at gmail.com wrote: > Important fix to patch 14, fix accounting of ghost bo. When creating a > ghost bo we don't account it, so set its acc_size to 0 so that when > ghost is release we don't overfree. > > I wonder how i didn't run into this before. > > Patch are also at > > http://people.freedesktop.org/~glisse/ttmdma/ > > Cheers, > Jerome > Oh i forgot to add some of the reviewed by, i updated patches on fdo, i don't resend to the ml. Cheers, Jerome
ttm: merge ttm_backend & ttm_tt, introduce ttm dma allocator V7
Important fix to patch 14, fix accounting of ghost bo. When creating a ghost bo we don't account it, so set its acc_size to 0 so that when ghost is release we don't overfree. I wonder how i didn't run into this before. Patch are also at http://people.freedesktop.org/~glisse/ttmdma/ Cheers, Jerome
ttm: merge ttm_backend ttm_tt, introduce ttm dma allocator V7
Important fix to patch 14, fix accounting of ghost bo. When creating a ghost bo we don't account it, so set its acc_size to 0 so that when ghost is release we don't overfree. I wonder how i didn't run into this before. Patch are also at http://people.freedesktop.org/~glisse/ttmdma/ Cheers, Jerome ___ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel