[igt-dev] [PATCH i-g-t 0/2] Revert the wrong fix for coverage-premultiplied-constant

Shankar, Uma uma.shankar at intel.com
Thu Jul 29 07:42:36 UTC 2021



> -----Original Message-----
> From: Surendrakumar Upadhyay, TejaskumarX
> <tejaskumarx.surendrakumar.upadhyay at intel.com>
> Sent: Thursday, July 29, 2021 12:09 PM
> To: Srinivas, Vidya <vidya.srinivas at intel.com>; Vudum, Lakshminarayana
> <lakshminarayana.vudum at intel.com>; igt-dev at lists.freedesktop.org; Joshi, Kunal1
> <kunal1.joshi at intel.com>
> Cc: Latvala, Petri <petri.latvala at intel.com>; Modem, Bhanuprakash
> <bhanuprakash.modem at intel.com>; Shankar, Uma <uma.shankar at intel.com>
> Subject: RE: [PATCH i-g-t 0/2] Revert the wrong fix for coverage-premultiplied-
> constant
> 
> Reviewed-by : Tejas Upadhyay <tejaskumarx.surendrakumar.upadhyay at intel.com>

Request to all, please refrain from top posting. Have all your comments inline.

Regards,
Uma Shankar
> Thanks,
> Tejas
> 
> > -----Original Message-----
> > From: Srinivas, Vidya <vidya.srinivas at intel.com>
> > Sent: 29 July 2021 12:07
> > To: Vudum, Lakshminarayana <lakshminarayana.vudum at intel.com>; igt-
> > dev at lists.freedesktop.org; Joshi, Kunal1 <kunal1.joshi at intel.com>;
> > Surendrakumar Upadhyay, TejaskumarX
> > <tejaskumarx.surendrakumar.upadhyay at intel.com>
> > Cc: Latvala, Petri <petri.latvala at intel.com>; Modem, Bhanuprakash
> > <bhanuprakash.modem at intel.com>; Shankar, Uma <uma.shankar at intel.com>
> > Subject: RE: [PATCH i-g-t 0/2] Revert the wrong fix for coverage-
> > premultiplied-constant
> >
> > @Surendrakumar Upadhyay, TejaskumarX Can you kindly review the patch
> > series - Please provide RB for full series? Thank you.
> > Kunal, kindly help merge. Thank you.
> >
> > Regards
> > Vidya
> >
> > > -----Original Message-----
> > > From: Srinivas, Vidya
> > > Sent: Saturday, July 24, 2021 9:39 PM
> > > To: Vudum, Lakshminarayana <Lakshminarayana.Vudum at intel.com>; igt-
> > > dev at lists.freedesktop.org
> > > Cc: Latvala, Petri <petri.latvala at intel.com>; Modem, Bhanuprakash
> > > <Bhanuprakash.Modem at intel.com>; Shankar, Uma
> > <uma.shankar at intel.com>
> > > Subject: RE: [PATCH i-g-t 0/2] Revert the wrong fix for coverage-
> > > premultiplied-constant
> > >
> > > Thank you so much Lakshmi. This time it has PASSED.
> > >
> > > Regards
> > > Vidya
> > >
> > > > -----Original Message-----
> > > > From: Vudum, Lakshminarayana <lakshminarayana.vudum at intel.com>
> > > > Sent: Friday, July 23, 2021 9:00 PM
> > > > To: Srinivas, Vidya <vidya.srinivas at intel.com>;
> > > > igt-dev at lists.freedesktop.org
> > > > Cc: Latvala, Petri <petri.latvala at intel.com>; Modem, Bhanuprakash
> > > > <bhanuprakash.modem at intel.com>; Shankar, Uma
> > > <uma.shankar at intel.com>
> > > > Subject: RE: [PATCH i-g-t 0/2] Revert the wrong fix for coverage-
> > > > premultiplied-constant
> > > >
> > > > Hi,
> > > >
> > > > Failures are addressed and results are re-reported.
> > > >
> > > > Lakshmi.
> > > > -----Original Message-----
> > > > From: Srinivas, Vidya <vidya.srinivas at intel.com>
> > > > Sent: Friday, July 23, 2021 1:46 AM
> > > > To: igt-dev at lists.freedesktop.org
> > > > Cc: Latvala, Petri <petri.latvala at intel.com>; Modem, Bhanuprakash
> > > > <bhanuprakash.modem at intel.com>; Shankar, Uma
> > > <uma.shankar at intel.com>;
> > > > Vudum, Lakshminarayana <lakshminarayana.vudum at intel.com>
> > > > Subject: RE: [PATCH i-g-t 0/2] Revert the wrong fix for coverage-
> > > > premultiplied-constant
> > > >
> > > > Hello Lakshmi,
> > > >
> > > > We need to revert this series. But there is a false positive.
> > > > Could you kindly help with re-run? Thank you so much.
> > > >
> > > > https://patchwork.freedesktop.org/series/92898/
> > > >
> > > > Regards
> > > > Vidya
> > > >
> > > > > -----Original Message-----
> > > > > From: Srinivas, Vidya <vidya.srinivas at intel.com>
> > > > > Sent: Thursday, July 22, 2021 8:37 PM
> > > > > To: igt-dev at lists.freedesktop.org
> > > > > Cc: Latvala, Petri <petri.latvala at intel.com>; Modem,
> > > > > Bhanuprakash <bhanuprakash.modem at intel.com>; Shankar, Uma
> > > > <uma.shankar at intel.com>;
> > > > > Srinivas, Vidya <vidya.srinivas at intel.com>
> > > > > Subject: [PATCH i-g-t 0/2] Revert the wrong fix for
> > > > > coverage-premultiplied- constant
> > > > >
> > > > > Subtest coverage-premultiplied-constant was failing due to
> > > > > buffer alignment issue. It was not due to 6bpc.
> > > > > Hence, we need to revert this patch which was wrongly submitted.
> > > > >
> > > > > Signed-off-by: Vidya Srinivas <vidya.srinivas at intel.com>
> 
> Reviewed-by : Tejas Upadhyay <tejaskumarx.surendrakumar.upadhyay at intel.com>
> 
> > > > >
> > > > >
> > > > > Vidya Srinivas (2):
> > > > >   Revert "tests/kms_plane_alpha_blend: Fix a check in is_6bpc function"
> > > > >   Revert "tests/kms_plane_alpha_blend: Skip
> > > > > coverage-vs-premult-vs-
> > > none
> > > > >     for 6bpc panels"
> > > > >
> > > > >  tests/kms_plane_alpha_blend.c | 36
> > > > > -----------------------------------
> > > > >  1 file changed, 36 deletions(-)
> > > > >
> > > > > --
> > > > > 2.32.0
> > > >
> > > >
> 



More information about the igt-dev mailing list