Re: [PATCH RFC 1/2] MAINTAINERS: Add "R:" designated-reviewers tag

[Joe Perches <joe@perches.com>]

On Wed, 2014-06-04 at 00:48 +0100, Ken Moffat wrote:
> On Mon, Jun 02, 2014 at 05:12:05PM -0700, Joe Perches wrote:
> > "Tested-by:" tags would be more helpful if the test
> > cases that were used were somehow sent along with the
> > signature.
[]
> Tested-by is usually used for a fix of some problem, often a
> regression.  A good commit message will explain the problem.

It seems about half of the commits with tested-by are
for regressions.

The latest commit message with your tested-by is great.

commit 18ee37a485653aa635cfab9a3710e9bcf5fbca01
Author: Daniel Vetter <daniel.vetter@ffwll.ch>
Date:   Fri May 30 16:41:23 2014 +0200

    drm/radeon: Resume fbcon last
 
[detailed explanation elided]

This commit log with your tested-by seems a bit
mysterious though:

commit 74ad54f249de39bc040cce7237b1b854a9c6f0ad
Author: Christian König <christian.koenig@amd.com>
Date:   Tue May 13 12:50:54 2014 +0200

    drm/radeon: fix typo in finding PLL params
    
    Otherwise the limit is raised to high.

As the commit that introduces this error is:

commit 3b333c55485fef0089ae7398906599d000df195e
Author: Christian König <christian.koenig@amd.com>
Date:   Thu Apr 24 18:39:59 2014 +0200

    drm/radeon: avoid high jitter with small frac divs

This is the entirety of the commit log.

The calculation that was done here:

	max(fb_div_min, (9 - (fb_div % 10)) * 20 + 60)

Doesn't give any indication why 50 is better than 60.

> This is _often_ not like userspace programs where you can write a
> testsuite to exercise the corner cases.  Kernel problems can be
> tied up with intricate details of the hardware, or equally they
> might happen only for certain usage, and for those it might not be
> at all obvious what is "special" about the affected usage.

Shrug.  Mostly where that wonderful test robot that
Fengguang Wu put together can aid in finding regressions,
it'd be nice if the tested-by: tests done could be added
somehow.  And not in the commit log itself.

It's certainly not possible for test cases to be mandatory.

cheers, Joe