Hi Andy,

> > > -         nr_pages = min_array(((unsigned long[]) {
> > > -                                 folio_nr_pages(folio),
> > > -                                 page_count - i,
> > > -                                 max_segment / PAGE_SIZE,
> > > -                               }), 3);
> > > +         nr_pages = min3(folio_nr_pages(folio),
> > > +                         page_count - i, max_segment / PAGE_SIZE);
> > 
> > thanks for your patch, but there is another patch involving (and
> > fixing) this line of code[*]. You are welcome to join the review.
> 
> Thanks for pointing this out! Do you prefer my to be rebased on top?
> Otherwise one can apply this patch first.
> 
> > [*] https://patchwork.freedesktop.org/patch/746877/?series=172332&rev=1
> 
> Note, my patch doesn't change existing behaviour.

yeah... besides that patch has had some reviews to follow up;
Krzysztof will have a little conflict easy to fix.

I reviewed and pushed your branch to drm-intel-gt-next.

Thanks,
Andi

Reply via email to