linux-kernel - RE: [PATCH] drivers/video/ep93xx-fb.c: clean up error-handling code

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1335271586.25429.19.camel@joe2Laptop>
Date:	Tue, 24 Apr 2012 07:46:26 -0500
From:	Joe Perches <joe@...ches.com>
To:	Julia Lawall <julia.lawall@...6.fr>
Cc:	Jingoo Han <jg1.han@...sung.com>,
	'Ryan Mallon' <rmallon@...il.com>,
	'Florian Tobias Schandinat' <FlorianSchandinat@....de>,
	kernel-janitors@...r.kernel.org, linux-fbdev@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: RE: [PATCH] drivers/video/ep93xx-fb.c: clean up error-handling code

On Thu, 2012-04-19 at 07:14 +0200, Julia Lawall wrote:
> I don't know thw impact of the ./.  I use the options
> 
> --nokeywords --nogit-fallback --subsystem --norolestats -f
> 
> julia
> 
> On Thu, 19 Apr 2012, Jingoo Han wrote:
> 
> >> -----Original Message-----
> >> From: Ryan Mallon
[]
> >> Oddly, scripts/get_maintainer.pl on this file doesn't return me, even
> >> though, according to git blame, I am the author of 90% of the commits.
> >> Should I have an entry in the MAINTAINERS file, or is
> >> scripts/get_maintainer.pl not working properly?
> >
> > There are optional differences in using scripts/get_maintainer.pl.
> > If you use './' ahead of file path, you will see your name.
> >
> > Without './' ahead of 'drivers/video/ep93xx-fb.c':
> > ./scripts/get_maintainer.pl --file drivers/video/ep93xx-fb.c
> > Florian Tobias Schandinat <FlorianSchandinat@....de> (maintainer:FRAMEBUFFER LAYER)
> > linux-fbdev@...r.kernel.org (open list:FRAMEBUFFER LAYER)
> > linux-kernel@...r.kernel.org (open list)
> >
> > With './' ahead of 'drivers/video/ep93xx-fb.c':
> > ./scripts/get_maintainer.pl --file ./drivers/video/ep93xx-fb.c
> > Ryan Mallon <rmallon@...il.com> (commit_signer:2/3=67%)
> > Paul Gortmaker <paul.gortmaker@...driver.com> (commit_signer:1/3=33%)
> > H Hartley Sweeten <hsweeten@...ionengravers.com> (commit_signer:1/3=33%)
> > Jesper Juhl <jj@...osbits.net> (commit_signer:1/3=33%)
> > Jiri Kosina <jkosina@...e.cz> (commit_signer:1/3=33%)
> > linux-kernel@...r.kernel.org (open list)

Prefixing "./" to the file path bypasses the normal
get_maintainers pattern check for file maintainership.

Florian Schandinat and linux-fbdev are not returned
in the second lookup above.

One of the patterns for FRAMEBUFFER is
F:	drivers/video/
which is a direct match for is ep93xx-fb.c file,
so --git is not used by default.

Adding --git may be appropriate and it also returns
Ryan's name in this case.

$ ./scripts/get_maintainer.pl -f drivers/video/ep93xx-fb.c
Florian Tobias Schandinat <FlorianSchandinat@....de> (maintainer:FRAMEBUFFER LAYER)
linux-fbdev@...r.kernel.org (open list:FRAMEBUFFER LAYER)
linux-kernel@...r.kernel.org (open list)

$ ./scripts/get_maintainer.pl --git -f drivers/video/ep93xx-fb.c
Florian Tobias Schandinat <FlorianSchandinat@....de> (maintainer:FRAMEBUFFER LAYER)
Ryan Mallon <rmallon@...il.com> (commit_signer:2/3=67%)
Paul Gortmaker <paul.gortmaker@...driver.com> (commit_signer:1/3=33%)
H Hartley Sweeten <hsweeten@...ionengravers.com> (commit_signer:1/3=33%)
Jesper Juhl <jj@...osbits.net> (commit_signer:1/3=33%)
Jiri Kosina <jkosina@...e.cz> (commit_signer:1/3=33%)
linux-fbdev@...r.kernel.org (open list:FRAMEBUFFER LAYER)
linux-kernel@...r.kernel.org (open list)

Nominally, as framebuffer maintainer, Florian
should also have "signed-off" on these commits,
but many subsystems don't actually have a
single path or person for patches and commits.

This file came into the tree via Andrew Morton.

Another way to investigate who actually does
work on an individual file is to use the
get_maintainer.pl option "--interactive".

That can give you an output like:

*  # email/list and role:stats                                        auth sign
*  1 Florian Tobias Schandinat <FlorianSchandinat@....de>             
     maintainer:FRAMEBUFFER LAYER
*  2 Ryan Mallon <rmallon@...il.com>                                     4    4
     commit_signer:2/3=67%,authored lines:645/650=99%,commits:4/8=50%
*  3 Paul Gortmaker <paul.gortmaker@...driver.com>                       1    1
     commit_signer:1/3=33%,commits:1/8=12%
*  4 H Hartley Sweeten <hsweeten@...ionengravers.com>                    0    2
     commit_signer:1/3=33%,commits:2/8=25%
*  5 Jesper Juhl <jj@...osbits.net>                                      0    1
     commit_signer:1/3=33%
*  6 Jiri Kosina <jkosina@...e.cz>                                       0    1
     commit_signer:1/3=33%
*  7 Andrew Morton <akpm@...ux-foundation.org>                           0    2
     commits:2/8=25%
*  8 Russell King <rmk+kernel@....linux.org.uk>                          0    1
     commits:1/8=12%
*  9 linux-fbdev@...r.kernel.org                                      
     open list:FRAMEBUFFER LAYER
* 10 linux-kernel@...r.kernel.org                                     
     open list

About maintainership for individual files:

If anyone thinks they are the maintainer for
a particular file, generally they should submit
a patch to MAINTAINERS adding a section with
their name, maintainership level and an
appropriate file pattern.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/