* Re: Merging the xe / ttm shrinker series?
2024-11-19 13:04 ` Christian König
@ 2024-11-21 16:08 ` Thomas Hellström
2024-12-02 9:55 ` Thomas Hellström
1 sibling, 0 replies; 5+ messages in thread
From: Thomas Hellström @ 2024-11-21 16:08 UTC (permalink / raw)
To: Christian König; +Cc: dri-devel, intel-xe
On Tue, 2024-11-19 at 14:04 +0100, Christian König wrote:
> Am 19.11.24 um 12:22 schrieb Thomas Hellström:
> > Hi, Christian,
> >
> > On Sun, 2024-11-03 at 13:38 +0100, Thomas Hellström wrote:
> > > Hi, Christian,
> > >
> > > The TTM shrinker series is now at v12 with all patches R-B:d.
> > >
> > > Ack to merge through drm-xe-next?
> > >
> > > Thanks,
> > > Thomas
> > >
> > Gentle ping on this,
> > The requested change of the ttm_backup interface done and MBrost
> > has
> > extended his R-B.
>
> Yeah and you guys keep pushing stuff on my TODO list additionally to
> the
> stuff the AMD guys are pushing for.
Yes, I think perhaps we need to communicate priorities from Intel's
side so that things with high priority doesn't get stalled, although
feedback is valuable also on work that is for "when time allows". Do
you see a way we can make reviews on dma-buf, drm_exec, TTM scale a bit
better?
>
> Give me a few more days, it looked mostly good. I just couldn't wrap
> my
> head around splitting the huge pages so far.
That's fine, although moving forward I think we need a way to relax the
merge criterion at least for TTM to have the review process scale a bit
better. Could we perhaps work towards adding reviewers to TTM
MAINTAINERS so that an R-B from a reviewer is sufficient unless there
are unresolved concerns from your or another maintainer?
Thanks,
Thomas
>
> Regards,
> Christian.
>
> >
> > Thanks,
> > Thomas
> >
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: Merging the xe / ttm shrinker series?
2024-11-19 13:04 ` Christian König
2024-11-21 16:08 ` Thomas Hellström
@ 2024-12-02 9:55 ` Thomas Hellström
1 sibling, 0 replies; 5+ messages in thread
From: Thomas Hellström @ 2024-12-02 9:55 UTC (permalink / raw)
To: Christian König; +Cc: dri-devel, intel-xe
Hi, Christian,
On Tue, 2024-11-19 at 14:04 +0100, Christian König wrote:
> Am 19.11.24 um 12:22 schrieb Thomas Hellström:
> > Hi, Christian,
> >
> > On Sun, 2024-11-03 at 13:38 +0100, Thomas Hellström wrote:
> > > Hi, Christian,
> > >
> > > The TTM shrinker series is now at v12 with all patches R-B:d.
> > >
> > > Ack to merge through drm-xe-next?
> > >
> > > Thanks,
> > > Thomas
> > >
> > Gentle ping on this,
> > The requested change of the ttm_backup interface done and MBrost
> > has
> > extended his R-B.
>
> Yeah and you guys keep pushing stuff on my TODO list additionally to
> the
> stuff the AMD guys are pushing for.
>
> Give me a few more days, it looked mostly good. I just couldn't wrap
> my
> head around splitting the huge pages so far.
>
> Regards,
> Christian.
Where are we on this? I have a small update on the series as a result
of your last review comments, but are you planning to review more
patches of the series or should I send the update and prepare for
merging?
It would be good if we could close on this one.
Thanks,
Thomas
>
> >
> > Thanks,
> > Thomas
> >
>
^ permalink raw reply [flat|nested] 5+ messages in thread