Re: [PATCH v11] drm: Add initial ci/ subdirectory

2023-09-19 Thread Maxime Ripard
On Mon, Sep 18, 2023 at 06:35:13PM -0300, Helen Koike wrote: > > > > I don't quite see the same picture from your side though. For example, > > > > my reading of what you've said is that flaky tests are utterly > > > > unacceptable, as are partial runs, and we shouldn't pretend otherwise. > > > >

Re: [PATCH v11] drm: Add initial ci/ subdirectory

2023-09-19 Thread Maxime Ripard
On Fri, Sep 15, 2023 at 04:08:42PM +0100, Daniel Stone wrote: > > > I don't quite see the same picture from your side though. For example, > > > my reading of what you've said is that flaky tests are utterly > > > unacceptable, as are partial runs, and we shouldn't pretend otherwise. > > > With

Re: [PATCH v11] drm: Add initial ci/ subdirectory

2023-09-18 Thread Helen Koike
On 15/09/2023 12:08, Daniel Stone wrote: Hey, On Thu, 14 Sept 2023 at 10:54, Maxime Ripard wrote: On Tue, Sep 12, 2023 at 02:16:41PM +0100, Daniel Stone wrote: Hopefully less mangled formatting this time: turns out Thunderbird + plain text is utterly unreadable, so that's one less MUA

Re: [PATCH v11] drm: Add initial ci/ subdirectory

2023-09-15 Thread Daniel Stone
Hey, On Thu, 14 Sept 2023 at 10:54, Maxime Ripard wrote: > On Tue, Sep 12, 2023 at 02:16:41PM +0100, Daniel Stone wrote: > > Hopefully less mangled formatting this time: turns out Thunderbird + > > plain text is utterly unreadable, so that's one less MUA that is > > actually usable to send email

Re: [PATCH v11] drm: Add initial ci/ subdirectory

2023-09-14 Thread Rob Clark
On Thu, Sep 14, 2023 at 2:54 AM Maxime Ripard wrote: > > Hi, > > On Tue, Sep 12, 2023 at 02:16:41PM +0100, Daniel Stone wrote: > > Hopefully less mangled formatting this time: turns out Thunderbird + > > plain text is utterly unreadable, so that's one less MUA that is > > actually usable to send

Re: [PATCH v11] drm: Add initial ci/ subdirectory

2023-09-14 Thread Maxime Ripard
Hi, On Tue, Sep 12, 2023 at 02:16:41PM +0100, Daniel Stone wrote: > Hopefully less mangled formatting this time: turns out Thunderbird + > plain text is utterly unreadable, so that's one less MUA that is > actually usable to send email to kernel lists without getting shouted > at. Sorry if it

Re: [PATCH v11] drm: Add initial ci/ subdirectory

2023-09-12 Thread Daniel Stone
Hi Maxime, Hopefully less mangled formatting this time: turns out Thunderbird + plain text is utterly unreadable, so that's one less MUA that is actually usable to send email to kernel lists without getting shouted at. On Mon, 11 Sept 2023 at 15:46, Maxime Ripard wrote: > On Mon, Sep 11, 2023 at

Re: [PATCH v11] drm: Add initial ci/ subdirectory

2023-09-11 Thread Maxime Ripard
Replying one more time, because I certainly don't want to create any hard feeling here. On Mon, Sep 11, 2023 at 03:30:55PM +0200, Michel Dänzer wrote: > By keeping those sets of expectations, we've been able to keep Mesa > pretty > clear of regressions, whilst having a very clear

Re: [PATCH v11] drm: Add initial ci/ subdirectory

2023-09-11 Thread Michel Dänzer
On 9/11/23 14:51, Maxime Ripard wrote: > On Mon, Sep 11, 2023 at 02:13:43PM +0200, Michel Dänzer wrote: >> On 9/11/23 11:34, Maxime Ripard wrote: >>> On Thu, Sep 07, 2023 at 01:40:02PM +0200, Daniel Stone wrote: Secondly, we will never be there. If we could pause for five years and sit

Re: [PATCH v11] drm: Add initial ci/ subdirectory

2023-09-11 Thread Maxime Ripard
On Mon, Sep 11, 2023 at 02:13:43PM +0200, Michel Dänzer wrote: > On 9/11/23 11:34, Maxime Ripard wrote: > > On Thu, Sep 07, 2023 at 01:40:02PM +0200, Daniel Stone wrote: > >> Yeah, this is what our experience with Mesa (in particular) has taught us. > >> > >> Having 100% of the tests pass 100% of

Re: [PATCH v11] drm: Add initial ci/ subdirectory

2023-09-11 Thread Michel Dänzer
On 9/11/23 11:34, Maxime Ripard wrote: > On Thu, Sep 07, 2023 at 01:40:02PM +0200, Daniel Stone wrote: >> Yeah, this is what our experience with Mesa (in particular) has taught us. >> >> Having 100% of the tests pass 100% of the time on 100% of the platforms is a >> great goal that everyone should

Re: [PATCH v11] drm: Add initial ci/ subdirectory

2023-09-11 Thread Maxime Ripard
Hi (Removing most of the context that got scrambled) On Thu, Sep 07, 2023 at 01:40:02PM +0200, Daniel Stone wrote: > Yeah, this is what our experience with Mesa (in particular) has taught us. > > Having 100% of the tests pass 100% of the time on 100% of the platforms is a > great goal that

Re: [PATCH v11] drm: Add initial ci/ subdirectory

2023-09-07 Thread Daniel Stone
Hi, On 04/09/2023 09:54, Daniel Vetter wrote: On Wed, 30 Aug 2023 at 17:14, Helen Koike > wrote: >> >> On 30/08/2023 11:57, Maxime Ripard wrote: >>> >>> I agree that we need a baseline, but that baseline should be >>> defined by the tests own merits, not their outcome on a >>> particular

Re: [PATCH v11] drm: Add initial ci/ subdirectory

2023-09-04 Thread Daniel Vetter
On Wed, 30 Aug 2023 at 17:14, Helen Koike wrote: > > > > On 30/08/2023 11:57, Maxime Ripard wrote: > > On Wed, Aug 30, 2023 at 10:24:49AM -0300, Helen Koike wrote: > >> Hi all, > >> > >> Thanks for you comments. > >> > >> On 30/08/2023 08:37, Maxime Ripard wrote: > >>> On Wed, Aug 30, 2023 at

Re: [PATCH v11] drm: Add initial ci/ subdirectory

2023-08-30 Thread Helen Koike
On 30/08/2023 11:57, Maxime Ripard wrote: On Wed, Aug 30, 2023 at 10:24:49AM -0300, Helen Koike wrote: Hi all, Thanks for you comments. On 30/08/2023 08:37, Maxime Ripard wrote: On Wed, Aug 30, 2023 at 01:58:31PM +0300, Jani Nikula wrote: On Wed, 30 Aug 2023, Maxime Ripard wrote: On

Re: [PATCH v11] drm: Add initial ci/ subdirectory

2023-08-30 Thread Maxime Ripard
On Wed, Aug 30, 2023 at 10:24:49AM -0300, Helen Koike wrote: > Hi all, > > Thanks for you comments. > > On 30/08/2023 08:37, Maxime Ripard wrote: > > On Wed, Aug 30, 2023 at 01:58:31PM +0300, Jani Nikula wrote: > > > On Wed, 30 Aug 2023, Maxime Ripard wrote: > > > > On Tue, Aug 22, 2023 at

Re: [PATCH v11] drm: Add initial ci/ subdirectory

2023-08-30 Thread Rob Clark
replying to a couple points on this thread On Wed, Aug 30, 2023 at 6:25 AM Helen Koike wrote: > > Hi all, > > Thanks for you comments. > > On 30/08/2023 08:37, Maxime Ripard wrote: > > On Wed, Aug 30, 2023 at 01:58:31PM +0300, Jani Nikula wrote: > >> On Wed, 30 Aug 2023, Maxime Ripard wrote: >

Re: [PATCH v11] drm: Add initial ci/ subdirectory

2023-08-30 Thread Helen Koike
Hi all, Thanks for you comments. On 30/08/2023 08:37, Maxime Ripard wrote: On Wed, Aug 30, 2023 at 01:58:31PM +0300, Jani Nikula wrote: On Wed, 30 Aug 2023, Maxime Ripard wrote: On Tue, Aug 22, 2023 at 04:26:06PM +0200, Daniel Vetter wrote: On Fri, Aug 11, 2023 at 02:19:53PM -0300, Helen

Re: [PATCH v11] drm: Add initial ci/ subdirectory

2023-08-30 Thread Maxime Ripard
On Wed, Aug 30, 2023 at 01:58:31PM +0300, Jani Nikula wrote: > On Wed, 30 Aug 2023, Maxime Ripard wrote: > > On Tue, Aug 22, 2023 at 04:26:06PM +0200, Daniel Vetter wrote: > >> On Fri, Aug 11, 2023 at 02:19:53PM -0300, Helen Koike wrote: > >> > From: Tomeu Vizoso > >> > > >> > Developers can

Re: [PATCH v11] drm: Add initial ci/ subdirectory

2023-08-30 Thread Jani Nikula
On Wed, 30 Aug 2023, Maxime Ripard wrote: > On Tue, Aug 22, 2023 at 04:26:06PM +0200, Daniel Vetter wrote: >> On Fri, Aug 11, 2023 at 02:19:53PM -0300, Helen Koike wrote: >> > From: Tomeu Vizoso >> > >> > Developers can easily execute several tests on different devices >> > by just pushing

Re: [PATCH v11] drm: Add initial ci/ subdirectory

2023-08-30 Thread Maxime Ripard
On Tue, Aug 22, 2023 at 04:26:06PM +0200, Daniel Vetter wrote: > On Fri, Aug 11, 2023 at 02:19:53PM -0300, Helen Koike wrote: > > From: Tomeu Vizoso > > > > Developers can easily execute several tests on different devices > > by just pushing their branch to their fork in a repository hosted > >