[Qemu-devel] [PATCH for-3.1 1/2] MAINTAINERS: Any file with "riscv" in the name is a RISC-V file

[Qemu-devel] [PATCH for-3.1 1/2] MAINTAINERS: Any file with "riscv" in the name is a RISC-V file

[Palmer Dabbelt]

There was a recent thread about unmaintained files in QEMU where it was
noted that a few RISC-V related files had slipped through the patterns
in our MAINTAINERS file entry.  This patch adds some more aggressive
pattern matching in an attempt to rectify this situation.

Signed-off-by: Palmer Dabbelt <palmer@sifive.com>
---
 MAINTAINERS | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 1032406c5607..e89adc81d5d5 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -255,6 +255,8 @@ F: target/riscv/
 F: hw/riscv/
 F: include/hw/riscv/
 F: disas/riscv.c
+K: riscv
+N: riscv
 
 S390
 M: Richard Henderson <rth@twiddle.net>
-- 
2.18.1

[Qemu-devel] [PATCH for-3.1 2/2] MAINTAINERS: Mark RISC-V as Supported

[Palmer Dabbelt]

There's three of us that are paid to work on this.

Signed-off-by: Palmer Dabbelt <palmer@sifive.com>
---
 MAINTAINERS | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index e89adc81d5d5..d5ec5e74f328 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -250,7 +250,7 @@ M: Alistair Francis <Alistair.Francis@wdc.com>
 M: Sagar Karandikar <sagark@eecs.berkeley.edu>
 M: Bastian Koppelmann <kbastian@mail.uni-paderborn.de>
 L: qemu-riscv@nongnu.org
-S: Maintained
+S: Supported
 F: target/riscv/
 F: hw/riscv/
 F: include/hw/riscv/
-- 
2.18.1

Re: [Qemu-devel] [PATCH for-3.1 2/2] MAINTAINERS: Mark RISC-V as Supported

[Alistair Francis]

On Wed, Nov 21, 2018 at 3:55 PM Palmer Dabbelt <palmer@sifive.com> wrote:
>
> There's three of us that are paid to work on this.
>
> Signed-off-by: Palmer Dabbelt <palmer@sifive.com>

Reviewed-by: Alistair Francis <alistair.francis@wdc.com>

Alistair

> ---
>  MAINTAINERS | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index e89adc81d5d5..d5ec5e74f328 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -250,7 +250,7 @@ M: Alistair Francis <Alistair.Francis@wdc.com>
>  M: Sagar Karandikar <sagark@eecs.berkeley.edu>
>  M: Bastian Koppelmann <kbastian@mail.uni-paderborn.de>
>  L: qemu-riscv@nongnu.org
> -S: Maintained
> +S: Supported
>  F: target/riscv/
>  F: hw/riscv/
>  F: include/hw/riscv/
> --
> 2.18.1
>
>

Re: [Qemu-devel] [PATCH for-3.1 1/2] MAINTAINERS: Any file with "riscv" in the name is a RISC-V file

[Philippe Mathieu-Daudé]

On 22/11/18 0:46, Palmer Dabbelt wrote:
> There was a recent thread about unmaintained files in QEMU where it was
> noted that a few RISC-V related files had slipped through the patterns
> in our MAINTAINERS file entry.  This patch adds some more aggressive
> pattern matching in an attempt to rectify this situation.
> 
> Signed-off-by: Palmer Dabbelt <palmer@sifive.com>
> ---
>   MAINTAINERS | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 1032406c5607..e89adc81d5d5 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -255,6 +255,8 @@ F: target/riscv/
>   F: hw/riscv/
>   F: include/hw/riscv/
>   F: disas/riscv.c
> +K: riscv
> +N: riscv

QEMU doesn't support the 'N' tag.
I see from the Linux kernel:

   N: Files and directories with regex patterns.
      N:   [^a-z]tegra     all files whose path contains the word tegra
      One pattern per line.  Multiple N: lines acceptable.
      scripts/get_maintainer.pl has different behavior for files that
      match F: pattern and matches of N: patterns.  By default,
      get_maintainer will not look at git log history when an F: pattern
      match occurs.  When an N: match occurs, git log history is used
      to also notify the people that have git commit signatures.

Should we add this feature to checkscript?

>   
>   S390
>   M: Richard Henderson <rth@twiddle.net>
> 

Re: [Qemu-devel] [PATCH for-3.1 1/2] MAINTAINERS: Any file with "riscv" in the name is a RISC-V file

[Palmer Dabbelt]

On Wed, 21 Nov 2018 16:29:09 PST (-0800), philmd@redhat.com wrote:
> On 22/11/18 0:46, Palmer Dabbelt wrote:
>> There was a recent thread about unmaintained files in QEMU where it was
>> noted that a few RISC-V related files had slipped through the patterns
>> in our MAINTAINERS file entry.  This patch adds some more aggressive
>> pattern matching in an attempt to rectify this situation.
>>
>> Signed-off-by: Palmer Dabbelt <palmer@sifive.com>
>> ---
>>   MAINTAINERS | 2 ++
>>   1 file changed, 2 insertions(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 1032406c5607..e89adc81d5d5 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -255,6 +255,8 @@ F: target/riscv/
>>   F: hw/riscv/
>>   F: include/hw/riscv/
>>   F: disas/riscv.c
>> +K: riscv
>> +N: riscv
>
> QEMU doesn't support the 'N' tag.
> I see from the Linux kernel:
>
>    N: Files and directories with regex patterns.
>       N:   [^a-z]tegra     all files whose path contains the word tegra
>       One pattern per line.  Multiple N: lines acceptable.
>       scripts/get_maintainer.pl has different behavior for files that
>       match F: pattern and matches of N: patterns.  By default,
>       get_maintainer will not look at git log history when an F: pattern
>       match occurs.  When an N: match occurs, git log history is used
>       to also notify the people that have git commit signatures.
>
> Should we add this feature to checkscript?

Ah, I didn't check because I just copied this from our Linux entry.  It's 
pretty much the core of what I'm trying to do here so if it's easy then that'd 
be great.

Unfortunately I don't know how to do so.

>
>>
>>   S390
>>   M: Richard Henderson <rth@twiddle.net>
>>

Re: [Qemu-devel] [PATCH for-3.1 1/2] MAINTAINERS: Any file with "riscv" in the name is a RISC-V file

[Thomas Huth]

On 2018-11-22 02:44, Palmer Dabbelt wrote:
> On Wed, 21 Nov 2018 16:29:09 PST (-0800), philmd@redhat.com wrote:
>> On 22/11/18 0:46, Palmer Dabbelt wrote:
>>> There was a recent thread about unmaintained files in QEMU where it was
>>> noted that a few RISC-V related files had slipped through the patterns
>>> in our MAINTAINERS file entry.  This patch adds some more aggressive
>>> pattern matching in an attempt to rectify this situation.
>>>
>>> Signed-off-by: Palmer Dabbelt <palmer@sifive.com>
>>> ---
>>>   MAINTAINERS | 2 ++
>>>   1 file changed, 2 insertions(+)
>>>
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>> index 1032406c5607..e89adc81d5d5 100644
>>> --- a/MAINTAINERS
>>> +++ b/MAINTAINERS
>>> @@ -255,6 +255,8 @@ F: target/riscv/
>>>   F: hw/riscv/
>>>   F: include/hw/riscv/
>>>   F: disas/riscv.c
>>> +K: riscv

Please be very careful with the "K:" lines ... you could suddenly end up
being a "maintainer" for files that you did not expect. For example,
after applying your patch, I guess the following commands will tell you
that you're the maintainer of these files:

 script/get_maintainer.pl -f MAINTAINERS
 script/get_maintainer.pl -f qapi/misc.json
 script/get_maintainer.pl -f configure
 ...

For this reason, we've limited the "K:" lines in MAINTAINERS so far to
files that contain the regex "^Subject:", too (so that they only match
patches with the keyword in the subject).

>>> +N: riscv
>>
>> QEMU doesn't support the 'N' tag.
>> I see from the Linux kernel:
>>
>>    N: Files and directories with regex patterns.
>>       N:   [^a-z]tegra     all files whose path contains the word tegra
>>       One pattern per line.  Multiple N: lines acceptable.
>>       scripts/get_maintainer.pl has different behavior for files that
>>       match F: pattern and matches of N: patterns.  By default,
>>       get_maintainer will not look at git log history when an F: pattern
>>       match occurs.  When an N: match occurs, git log history is used
>>       to also notify the people that have git commit signatures.
>>
>> Should we add this feature to checkscript?
> 
> Ah, I didn't check because I just copied this from our Linux entry. 
> It's pretty much the core of what I'm trying to do here so if it's easy
> then that'd be great.
> 
> Unfortunately I don't know how to do so.

Try to export the commits eb90d0855 and bbbe96ed8 from the Linux kernel
git and see whether the patches can be applied to our version of
get_maintainers.pl, too.

 HTH,
  Thomas