Merging the xe / ttm shrinker series?

Merging the xe / ttm shrinker series?

[Thomas Hellström]

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

Re: Merging the xe / ttm shrinker series?

[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.

Thanks,
Thomas

Re: Merging the xe / ttm shrinker series?

[Christian König]

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.

>
> Thanks,
> Thomas
>

Re: Merging the xe / ttm shrinker series?

[Thomas Hellström]

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
> > 
> 

Re: Merging the xe / ttm shrinker series?

[Thomas Hellström]

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
> > 
>