All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] MAINTAINERS: Add "B:" preferred bug reporting method
@ 2016-01-15 14:45 ` Jani Nikula
  0 siblings, 0 replies; 6+ messages in thread
From: Jani Nikula @ 2016-01-15 14:45 UTC (permalink / raw)
  To: linux-kernel
  Cc: intel-gfx, jani.nikula, Daniel Vetter, Joe Perches, Andrew Morton

Different subsystems and drivers have different preferred ways of
receiving bug reports; mailing list or bugzillas at various
locations. Add "B:" entry for specifying the preference to guide bug
reporters at the right location.

While at it, document the freedesktop.org bugzilla as the preferred
location for drm/i915 bug reports. We have more reassignment of bugs
between the kernel and userspace components than with other kernel
components, and we've thus consolidated our bug tracking there.

Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Joe Perches <joe@perches.com>
Cc: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 MAINTAINERS | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 32eda9d0be0c..a803066a9422 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -75,6 +75,8 @@ Descriptions of section entries:
 	L: Mailing list that is relevant to this area
 	W: Web-page with status/info
 	Q: Patchwork web based patch tracking system site
+	B: Preferred method for reporting bugs; bug tracking system site
+	   or "List" for mailing list.
 	T: SCM tree type and location.
 	   Type is one of: git, hg, quilt, stgit, topgit
 	S: Status, one of the following:
@@ -3655,6 +3657,7 @@ L:	intel-gfx@lists.freedesktop.org
 L:	dri-devel@lists.freedesktop.org
 W:	https://01.org/linuxgraphics/
 Q:	http://patchwork.freedesktop.org/project/intel-gfx/
+B:	https://bugs.freedesktop.org/enter_bug.cgi?product=DRI&component=DRM/Intel
 T:	git git://anongit.freedesktop.org/drm-intel
 S:	Supported
 F:	drivers/gpu/drm/i915/
-- 
2.1.4

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH] MAINTAINERS: Add "B:" preferred bug reporting method
@ 2016-01-15 14:45 ` Jani Nikula
  0 siblings, 0 replies; 6+ messages in thread
From: Jani Nikula @ 2016-01-15 14:45 UTC (permalink / raw)
  To: linux-kernel; +Cc: jani.nikula, Joe Perches, intel-gfx, Andrew Morton

Different subsystems and drivers have different preferred ways of
receiving bug reports; mailing list or bugzillas at various
locations. Add "B:" entry for specifying the preference to guide bug
reporters at the right location.

While at it, document the freedesktop.org bugzilla as the preferred
location for drm/i915 bug reports. We have more reassignment of bugs
between the kernel and userspace components than with other kernel
components, and we've thus consolidated our bug tracking there.

Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Joe Perches <joe@perches.com>
Cc: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 MAINTAINERS | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 32eda9d0be0c..a803066a9422 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -75,6 +75,8 @@ Descriptions of section entries:
 	L: Mailing list that is relevant to this area
 	W: Web-page with status/info
 	Q: Patchwork web based patch tracking system site
+	B: Preferred method for reporting bugs; bug tracking system site
+	   or "List" for mailing list.
 	T: SCM tree type and location.
 	   Type is one of: git, hg, quilt, stgit, topgit
 	S: Status, one of the following:
@@ -3655,6 +3657,7 @@ L:	intel-gfx@lists.freedesktop.org
 L:	dri-devel@lists.freedesktop.org
 W:	https://01.org/linuxgraphics/
 Q:	http://patchwork.freedesktop.org/project/intel-gfx/
+B:	https://bugs.freedesktop.org/enter_bug.cgi?product=DRI&component=DRM/Intel
 T:	git git://anongit.freedesktop.org/drm-intel
 S:	Supported
 F:	drivers/gpu/drm/i915/
-- 
2.1.4

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* ✗ Fi.CI.BAT: failure for MAINTAINERS: Add "B:" preferred bug reporting method
  2016-01-15 14:45 ` Jani Nikula
  (?)
@ 2016-01-15 15:49 ` Patchwork
  -1 siblings, 0 replies; 6+ messages in thread
From: Patchwork @ 2016-01-15 15:49 UTC (permalink / raw)
  To: Jani Nikula; +Cc: intel-gfx

== Summary ==

Built on 615fbad7f4cea73b1a8eccdcc942c8ca1a708dab drm-intel-nightly: 2016y-01m-15d-09h-46m-32s UTC integration manifest

Test gem_ctx_basic:
                pass       -> FAIL       (bdw-ultra)
Test gem_storedw_loop:
        Subgroup basic-render:
                pass       -> DMESG-WARN (skl-i7k-2) UNSTABLE
Test kms_flip:
        Subgroup basic-flip-vs-modeset:
                dmesg-warn -> PASS       (skl-i5k-2)
Test kms_force_connector_basic:
        Subgroup prune-stale-modes:
                pass       -> SKIP       (hsw-gt2)
Test kms_pipe_crc_basic:
        Subgroup read-crc-pipe-c:
                pass       -> DMESG-WARN (bdw-ultra)

bdw-nuci7        total:138  pass:128  dwarn:1   dfail:0   fail:0   skip:9  
bdw-ultra        total:138  pass:130  dwarn:1   dfail:0   fail:1   skip:6  
bsw-nuc-2        total:141  pass:115  dwarn:2   dfail:0   fail:0   skip:24 
byt-nuc          total:141  pass:123  dwarn:3   dfail:0   fail:0   skip:15 
hsw-brixbox      total:141  pass:134  dwarn:0   dfail:0   fail:0   skip:7  
hsw-gt2          total:141  pass:136  dwarn:0   dfail:0   fail:0   skip:5  
ilk-hp8440p      total:141  pass:101  dwarn:3   dfail:0   fail:0   skip:37 
ivb-t430s        total:135  pass:122  dwarn:3   dfail:4   fail:0   skip:6  
skl-i5k-2        total:141  pass:132  dwarn:1   dfail:0   fail:0   skip:8  
skl-i7k-2        total:141  pass:131  dwarn:2   dfail:0   fail:0   skip:8  
snb-dellxps      total:141  pass:122  dwarn:5   dfail:0   fail:0   skip:14 
snb-x220t        total:141  pass:122  dwarn:5   dfail:0   fail:1   skip:13 

Results at /archive/results/CI_IGT_test/Patchwork_1198/

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] MAINTAINERS: Add "B:" preferred bug reporting method
  2016-01-15 14:45 ` Jani Nikula
  (?)
  (?)
@ 2016-01-16  0:22 ` Joe Perches
  2016-01-16 11:39     ` Jani Nikula
  -1 siblings, 1 reply; 6+ messages in thread
From: Joe Perches @ 2016-01-16  0:22 UTC (permalink / raw)
  To: Jani Nikula, linux-kernel; +Cc: intel-gfx, Daniel Vetter, Andrew Morton

On Fri, 2016-01-15 at 16:45 +0200, Jani Nikula wrote:
> Different subsystems and drivers have different preferred ways of
> receiving bug reports; mailing list or bugzillas at various
> locations. Add "B:" entry for specifying the preference to guide bug
> reporters at the right location.
[]
> diff --git a/MAINTAINERS b/MAINTAINERS
[]
> @@ -75,6 +75,8 @@ Descriptions of section entries:
>  	L: Mailing list that is relevant to this area
>  	W: Web-page with status/info
>  	Q: Patchwork web based patch tracking system site
> +	B: Preferred method for reporting bugs; bug tracking system site
> +	   or "List" for mailing list.
>  	T: SCM tree type and location.
>  	   Type is one of: git, hg, quilt, stgit, topgit
>  	S: Status, one of the following:
> @@ -3655,6 +3657,7 @@ L:	intel-gfx@lists.freedesktop.org
>  L:	dri-devel@lists.freedesktop.org
>  W:	https://01.org/linuxgraphics/
>  Q:	http://patchwork.freedesktop.org/project/intel-gfx/
> +B:	https://bugs.freedesktop.org/enter_bug.cgi?product=DRI&component=DRM/Intel

This requires a LoginID & password

Maybe useful to show the open bugs too:
https://bugs.freedesktop.org/buglist.cgi?component=DRM%2FIntel&product=DRI&resolution=---

Maybe the get_maintainer script should be updated.
Something like: (untested)
---
 scripts/get_maintainer.pl | 17 +++++++++++++++--
 1 file changed, 15 insertions(+), 2 deletions(-)

diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
index 1873421..bbe5337 100755
--- a/scripts/get_maintainer.pl
+++ b/scripts/get_maintainer.pl
@@ -47,6 +47,7 @@ my $output_rolestats = 1;
 my $output_section_maxlen = 50;
 my $scm = 0;
 my $web = 0;
+my $bug = 0;
 my $subsystem = 0;
 my $status = 0;
 my $keywords = 1;
@@ -239,6 +240,7 @@ if (!GetOptions(
 		'status!' => \$status,
 		'scm!' => \$scm,
 		'web!' => \$web,
+		'bug!' => \$bug,
 		'pattern-depth=i' => \$pattern_depth,
 		'k|keywords!' => \$keywords,
 		'sections!' => \$sections,
@@ -276,12 +278,13 @@ if ($sections) {
     $status = 0;
     $subsystem = 0;
     $web = 0;
+    $bug = 0;
     $keywords = 0;
     $interactive = 0;
 } else {
-    my $selections = $email + $scm + $status + $subsystem + $web;
+    my $selections = $email + $scm + $status + $subsystem + $web + $bug;
     if ($selections == 0) {
-	die "$P:  Missing required option: email, scm, status, subsystem or web\n";
+	die "$P:  Missing required option: email, scm, status, subsystem, web or bug\n";
     }
 }
 
@@ -505,6 +508,7 @@ my %hash_list_to;
 my @list_to = ();
 my @scm = ();
 my @web = ();
+my @bug = ();
 my @subsystem = ();
 my @status = ();
 my %deduplicate_name_hash = ();
@@ -537,6 +541,11 @@ if ($web) {
     output(@web);
 }
 
+if ($bug) {
+    @bug = uniq(@bug);
+    output(@bug);
+}
+
 exit($exit);
 
 sub ignore_email_address {
@@ -593,6 +602,7 @@ sub get_maintainers {
     @list_to = ();
     @scm = ();
     @web = ();
+    @bug = ();
     @subsystem = ();
     @status = ();
     %deduplicate_name_hash = ();
@@ -802,6 +812,7 @@ MAINTAINER field selection options:
   --status => print status if any
   --subsystem => print subsystem name if any
   --web => print website(s) if any
+  --bug => print bug reporting mechanism(s) if any
 
 Output type options:
   --separator [, ] => separator for multiple entries on 1 line
@@ -1129,6 +1140,8 @@ sub add_categories {
 		push(@scm, $pvalue);
 	    } elsif ($ptype eq "W") {
 		push(@web, $pvalue);
+	    } elsif ($ptype eq "B") {
+		push(@bug, $pvalue);
 	    } elsif ($ptype eq "S") {
 		push(@status, $pvalue);
 	    }

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] MAINTAINERS: Add "B:" preferred bug reporting method
  2016-01-16  0:22 ` [PATCH] " Joe Perches
@ 2016-01-16 11:39     ` Jani Nikula
  0 siblings, 0 replies; 6+ messages in thread
From: Jani Nikula @ 2016-01-16 11:39 UTC (permalink / raw)
  To: Joe Perches, linux-kernel; +Cc: intel-gfx, Daniel Vetter, Andrew Morton

On Sat, 16 Jan 2016, Joe Perches <joe@perches.com> wrote:
> On Fri, 2016-01-15 at 16:45 +0200, Jani Nikula wrote:
>> Different subsystems and drivers have different preferred ways of
>> receiving bug reports; mailing list or bugzillas at various
>> locations. Add "B:" entry for specifying the preference to guide bug
>> reporters at the right location.
> []
>> diff --git a/MAINTAINERS b/MAINTAINERS
> []
>> @@ -75,6 +75,8 @@ Descriptions of section entries:
>>  	L: Mailing list that is relevant to this area
>>  	W: Web-page with status/info
>>  	Q: Patchwork web based patch tracking system site
>> +	B: Preferred method for reporting bugs; bug tracking system site
>> +	   or "List" for mailing list.
>>  	T: SCM tree type and location.
>>  	   Type is one of: git, hg, quilt, stgit, topgit
>>  	S: Status, one of the following:
>> @@ -3655,6 +3657,7 @@ L:	intel-gfx@lists.freedesktop.org
>>  L:	dri-devel@lists.freedesktop.org
>>  W:	https://01.org/linuxgraphics/
>>  Q:	http://patchwork.freedesktop.org/project/intel-gfx/
>> +B:	https://bugs.freedesktop.org/enter_bug.cgi?product=DRI&component=DRM/Intel
>
> This requires a LoginID & password

It does, and I expect basically any bug reporting tool to do so.

I was divided about linking to the form requiring login directly, but
this helps the reporter to the right product and component, which may
not be obvious. Anything to make the bug reporters' lives easier.

> Maybe useful to show the open bugs too:
> https://bugs.freedesktop.org/buglist.cgi?component=DRM%2FIntel&product=DRI&resolution=---

In the end, I don't mind whether it's the URL I suggested, the URL you
suggested, or some combination, as long as it directs the bug reporter
to https://bugs.freedesktop.org/.

The main idea is just to make the life of bug reporters and maintainers
easier by documenting where the maintainers want the bugs reported.

BR,
Jani.

>
> Maybe the get_maintainer script should be updated.
> Something like: (untested)
> ---
>  scripts/get_maintainer.pl | 17 +++++++++++++++--
>  1 file changed, 15 insertions(+), 2 deletions(-)
>
> diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
> index 1873421..bbe5337 100755
> --- a/scripts/get_maintainer.pl
> +++ b/scripts/get_maintainer.pl
> @@ -47,6 +47,7 @@ my $output_rolestats = 1;
>  my $output_section_maxlen = 50;
>  my $scm = 0;
>  my $web = 0;
> +my $bug = 0;
>  my $subsystem = 0;
>  my $status = 0;
>  my $keywords = 1;
> @@ -239,6 +240,7 @@ if (!GetOptions(
>  		'status!' => \$status,
>  		'scm!' => \$scm,
>  		'web!' => \$web,
> +		'bug!' => \$bug,
>  		'pattern-depth=i' => \$pattern_depth,
>  		'k|keywords!' => \$keywords,
>  		'sections!' => \$sections,
> @@ -276,12 +278,13 @@ if ($sections) {
>      $status = 0;
>      $subsystem = 0;
>      $web = 0;
> +    $bug = 0;
>      $keywords = 0;
>      $interactive = 0;
>  } else {
> -    my $selections = $email + $scm + $status + $subsystem + $web;
> +    my $selections = $email + $scm + $status + $subsystem + $web + $bug;
>      if ($selections == 0) {
> -	die "$P:  Missing required option: email, scm, status, subsystem or web\n";
> +	die "$P:  Missing required option: email, scm, status, subsystem, web or bug\n";
>      }
>  }
>  
> @@ -505,6 +508,7 @@ my %hash_list_to;
>  my @list_to = ();
>  my @scm = ();
>  my @web = ();
> +my @bug = ();
>  my @subsystem = ();
>  my @status = ();
>  my %deduplicate_name_hash = ();
> @@ -537,6 +541,11 @@ if ($web) {
>      output(@web);
>  }
>  
> +if ($bug) {
> +    @bug = uniq(@bug);
> +    output(@bug);
> +}
> +
>  exit($exit);
>  
>  sub ignore_email_address {
> @@ -593,6 +602,7 @@ sub get_maintainers {
>      @list_to = ();
>      @scm = ();
>      @web = ();
> +    @bug = ();
>      @subsystem = ();
>      @status = ();
>      %deduplicate_name_hash = ();
> @@ -802,6 +812,7 @@ MAINTAINER field selection options:
>    --status => print status if any
>    --subsystem => print subsystem name if any
>    --web => print website(s) if any
> +  --bug => print bug reporting mechanism(s) if any
>  
>  Output type options:
>    --separator [, ] => separator for multiple entries on 1 line
> @@ -1129,6 +1140,8 @@ sub add_categories {
>  		push(@scm, $pvalue);
>  	    } elsif ($ptype eq "W") {
>  		push(@web, $pvalue);
> +	    } elsif ($ptype eq "B") {
> +		push(@bug, $pvalue);
>  	    } elsif ($ptype eq "S") {
>  		push(@status, $pvalue);
>  	    }

-- 
Jani Nikula, Intel Open Source Technology Center

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] MAINTAINERS: Add "B:" preferred bug reporting method
@ 2016-01-16 11:39     ` Jani Nikula
  0 siblings, 0 replies; 6+ messages in thread
From: Jani Nikula @ 2016-01-16 11:39 UTC (permalink / raw)
  To: Joe Perches, linux-kernel; +Cc: intel-gfx, Andrew Morton

On Sat, 16 Jan 2016, Joe Perches <joe@perches.com> wrote:
> On Fri, 2016-01-15 at 16:45 +0200, Jani Nikula wrote:
>> Different subsystems and drivers have different preferred ways of
>> receiving bug reports; mailing list or bugzillas at various
>> locations. Add "B:" entry for specifying the preference to guide bug
>> reporters at the right location.
> []
>> diff --git a/MAINTAINERS b/MAINTAINERS
> []
>> @@ -75,6 +75,8 @@ Descriptions of section entries:
>>  	L: Mailing list that is relevant to this area
>>  	W: Web-page with status/info
>>  	Q: Patchwork web based patch tracking system site
>> +	B: Preferred method for reporting bugs; bug tracking system site
>> +	   or "List" for mailing list.
>>  	T: SCM tree type and location.
>>  	   Type is one of: git, hg, quilt, stgit, topgit
>>  	S: Status, one of the following:
>> @@ -3655,6 +3657,7 @@ L:	intel-gfx@lists.freedesktop.org
>>  L:	dri-devel@lists.freedesktop.org
>>  W:	https://01.org/linuxgraphics/
>>  Q:	http://patchwork.freedesktop.org/project/intel-gfx/
>> +B:	https://bugs.freedesktop.org/enter_bug.cgi?product=DRI&component=DRM/Intel
>
> This requires a LoginID & password

It does, and I expect basically any bug reporting tool to do so.

I was divided about linking to the form requiring login directly, but
this helps the reporter to the right product and component, which may
not be obvious. Anything to make the bug reporters' lives easier.

> Maybe useful to show the open bugs too:
> https://bugs.freedesktop.org/buglist.cgi?component=DRM%2FIntel&product=DRI&resolution=---

In the end, I don't mind whether it's the URL I suggested, the URL you
suggested, or some combination, as long as it directs the bug reporter
to https://bugs.freedesktop.org/.

The main idea is just to make the life of bug reporters and maintainers
easier by documenting where the maintainers want the bugs reported.

BR,
Jani.

>
> Maybe the get_maintainer script should be updated.
> Something like: (untested)
> ---
>  scripts/get_maintainer.pl | 17 +++++++++++++++--
>  1 file changed, 15 insertions(+), 2 deletions(-)
>
> diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
> index 1873421..bbe5337 100755
> --- a/scripts/get_maintainer.pl
> +++ b/scripts/get_maintainer.pl
> @@ -47,6 +47,7 @@ my $output_rolestats = 1;
>  my $output_section_maxlen = 50;
>  my $scm = 0;
>  my $web = 0;
> +my $bug = 0;
>  my $subsystem = 0;
>  my $status = 0;
>  my $keywords = 1;
> @@ -239,6 +240,7 @@ if (!GetOptions(
>  		'status!' => \$status,
>  		'scm!' => \$scm,
>  		'web!' => \$web,
> +		'bug!' => \$bug,
>  		'pattern-depth=i' => \$pattern_depth,
>  		'k|keywords!' => \$keywords,
>  		'sections!' => \$sections,
> @@ -276,12 +278,13 @@ if ($sections) {
>      $status = 0;
>      $subsystem = 0;
>      $web = 0;
> +    $bug = 0;
>      $keywords = 0;
>      $interactive = 0;
>  } else {
> -    my $selections = $email + $scm + $status + $subsystem + $web;
> +    my $selections = $email + $scm + $status + $subsystem + $web + $bug;
>      if ($selections == 0) {
> -	die "$P:  Missing required option: email, scm, status, subsystem or web\n";
> +	die "$P:  Missing required option: email, scm, status, subsystem, web or bug\n";
>      }
>  }
>  
> @@ -505,6 +508,7 @@ my %hash_list_to;
>  my @list_to = ();
>  my @scm = ();
>  my @web = ();
> +my @bug = ();
>  my @subsystem = ();
>  my @status = ();
>  my %deduplicate_name_hash = ();
> @@ -537,6 +541,11 @@ if ($web) {
>      output(@web);
>  }
>  
> +if ($bug) {
> +    @bug = uniq(@bug);
> +    output(@bug);
> +}
> +
>  exit($exit);
>  
>  sub ignore_email_address {
> @@ -593,6 +602,7 @@ sub get_maintainers {
>      @list_to = ();
>      @scm = ();
>      @web = ();
> +    @bug = ();
>      @subsystem = ();
>      @status = ();
>      %deduplicate_name_hash = ();
> @@ -802,6 +812,7 @@ MAINTAINER field selection options:
>    --status => print status if any
>    --subsystem => print subsystem name if any
>    --web => print website(s) if any
> +  --bug => print bug reporting mechanism(s) if any
>  
>  Output type options:
>    --separator [, ] => separator for multiple entries on 1 line
> @@ -1129,6 +1140,8 @@ sub add_categories {
>  		push(@scm, $pvalue);
>  	    } elsif ($ptype eq "W") {
>  		push(@web, $pvalue);
> +	    } elsif ($ptype eq "B") {
> +		push(@bug, $pvalue);
>  	    } elsif ($ptype eq "S") {
>  		push(@status, $pvalue);
>  	    }

-- 
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2016-01-16 11:40 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-01-15 14:45 [PATCH] MAINTAINERS: Add "B:" preferred bug reporting method Jani Nikula
2016-01-15 14:45 ` Jani Nikula
2016-01-15 15:49 ` ✗ Fi.CI.BAT: failure for " Patchwork
2016-01-16  0:22 ` [PATCH] " Joe Perches
2016-01-16 11:39   ` Jani Nikula
2016-01-16 11:39     ` Jani Nikula

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.