Message ID | 20161118161824.5750-1-konrad.wilk@oracle.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On 18/11/16 16:18, Konrad Rzeszutek Wilk wrote: > That way we have one person who can: a) poke other maintainers > or pull them in with new drivers are introduced, b) we have > one maintainer who can shepherd the patches along instead of > depending on the REST maintainers which may be busy with > other responsibilities. > > Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> Acked-by: George Dunlap <george.dunlap@citrix.com>
On Fri, Nov 18, 2016 at 11:18:24AM -0500, Konrad Rzeszutek Wilk wrote: > That way we have one person who can: a) poke other maintainers > or pull them in with new drivers are introduced, b) we have > one maintainer who can shepherd the patches along instead of > depending on the REST maintainers which may be busy with > other responsibilities. > > Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> > --- > Cc: Andrew Cooper <andrew.cooper3@citrix.com> > Cc: George Dunlap <George.Dunlap@eu.citrix.com> > Cc: Ian Jackson <ian.jackson@eu.citrix.com> > Cc: Jan Beulich <jbeulich@suse.com> > Cc: Stefano Stabellini <sstabellini@kernel.org> > Cc: Tim Deegan <tim@xen.org> > Cc: Wei Liu <wei.liu2@citrix.com> > --- > MAINTAINERS | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index f0d0202..492f197 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -304,6 +304,11 @@ X: xen/arch/x86/acpi/lib.c > F: xen/drivers/cpufreq/ > F: xen/include/acpi/cpufreq/ > > +PUBLIC INTERFACES > +M: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> > +S: Supported > +F: xen/include/public/ Grrrr.. Looks like I need to use tabs instead of spaces! Let me repost that once folks have Acked this patch, or Nacked. > + > QEMU-DM > M: Ian Jackson <ian.jackson@eu.citrix.com> > S: Supported > -- > 2.9.3 >
On Fri, Nov 18, 2016 at 11:18:24AM -0500, Konrad Rzeszutek Wilk wrote: > That way we have one person who can: a) poke other maintainers > or pull them in with new drivers are introduced, b) we have > one maintainer who can shepherd the patches along instead of > depending on the REST maintainers which may be busy with > other responsibilities. > > Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> Acked-by: Wei Liu <wei.liu2@citrix.com>
>>> On 18.11.16 at 17:18, <konrad.wilk@oracle.com> wrote: > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -304,6 +304,11 @@ X: xen/arch/x86/acpi/lib.c > F: xen/drivers/cpufreq/ > F: xen/include/acpi/cpufreq/ > > +PUBLIC INTERFACES > +M: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> > +S: Supported > +F: xen/include/public/ I think this goes too far: Earlier discussion pinpointed to bottleneck to xen/include/public/io/, and that's what I understood would get taken care of here (and then perhaps with a slightly different title). Jan
On Fri, Nov 18, 2016 at 10:06:59AM -0700, Jan Beulich wrote: > >>> On 18.11.16 at 17:18, <konrad.wilk@oracle.com> wrote: > > --- a/MAINTAINERS > > +++ b/MAINTAINERS > > @@ -304,6 +304,11 @@ X: xen/arch/x86/acpi/lib.c > > F: xen/drivers/cpufreq/ > > F: xen/include/acpi/cpufreq/ > > > > +PUBLIC INTERFACES > > +M: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> > > +S: Supported > > +F: xen/include/public/ > > I think this goes too far: Earlier discussion pinpointed to bottleneck > to xen/include/public/io/, and that's what I understood would get Ah right! Lets change it to that. > taken care of here (and then perhaps with a slightly different title). "Czar" is the only word I heard :-) > > Jan >
On Fri, 18 Nov 2016, Konrad Rzeszutek Wilk wrote: > On Fri, Nov 18, 2016 at 10:06:59AM -0700, Jan Beulich wrote: > > >>> On 18.11.16 at 17:18, <konrad.wilk@oracle.com> wrote: > > > --- a/MAINTAINERS > > > +++ b/MAINTAINERS > > > @@ -304,6 +304,11 @@ X: xen/arch/x86/acpi/lib.c > > > F: xen/drivers/cpufreq/ > > > F: xen/include/acpi/cpufreq/ > > > > > > +PUBLIC INTERFACES > > > +M: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> > > > +S: Supported > > > +F: xen/include/public/ > > > > I think this goes too far: Earlier discussion pinpointed to bottleneck > > to xen/include/public/io/, and that's what I understood would get > > Ah right! Lets change it to that. I would write something explicit about new PV protocol designs, maybe by adding: F: docs/misc/pvdrivers but we are not using that directory yet, or by changing the title entry to: PUBLIC INTERFACES AND NEW PV DRIVERS DESIGNS > "Czar" is the only word I heard :-) :-D
On Fri, 18 Nov 2016, Stefano Stabellini wrote: > On Fri, 18 Nov 2016, Konrad Rzeszutek Wilk wrote: > > On Fri, Nov 18, 2016 at 10:06:59AM -0700, Jan Beulich wrote: > > > >>> On 18.11.16 at 17:18, <konrad.wilk@oracle.com> wrote: > > > > --- a/MAINTAINERS > > > > +++ b/MAINTAINERS > > > > @@ -304,6 +304,11 @@ X: xen/arch/x86/acpi/lib.c > > > > F: xen/drivers/cpufreq/ > > > > F: xen/include/acpi/cpufreq/ > > > > > > > > +PUBLIC INTERFACES > > > > +M: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> > > > > +S: Supported > > > > +F: xen/include/public/ > > > > > > I think this goes too far: Earlier discussion pinpointed to bottleneck > > > to xen/include/public/io/, and that's what I understood would get > > > > Ah right! Lets change it to that. > > I would write something explicit about new PV protocol designs, maybe by > adding: > > F: docs/misc/pvdrivers > > but we are not using that directory yet, or by changing the title entry to: > > PUBLIC INTERFACES AND NEW PV DRIVERS DESIGNS And of course you can add Acked-by: Stefano Stabellini <sstabellini@kernel.org> > > "Czar" is the only word I heard :-) > > :-D >
On 18/11/16 17:57, Stefano Stabellini wrote: > On Fri, 18 Nov 2016, Konrad Rzeszutek Wilk wrote: >> On Fri, Nov 18, 2016 at 10:06:59AM -0700, Jan Beulich wrote: >>>>>> On 18.11.16 at 17:18, <konrad.wilk@oracle.com> wrote: >>>> --- a/MAINTAINERS >>>> +++ b/MAINTAINERS >>>> @@ -304,6 +304,11 @@ X: xen/arch/x86/acpi/lib.c >>>> F: xen/drivers/cpufreq/ >>>> F: xen/include/acpi/cpufreq/ >>>> >>>> +PUBLIC INTERFACES >>>> +M: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> >>>> +S: Supported >>>> +F: xen/include/public/ >>> >>> I think this goes too far: Earlier discussion pinpointed to bottleneck >>> to xen/include/public/io/, and that's what I understood would get >> >> Ah right! Lets change it to that. > > I would write something explicit about new PV protocol designs, maybe by > adding: > > F: docs/misc/pvdrivers > > but we are not using that directory yet, or by changing the title entry to: > > PUBLIC INTERFACES AND NEW PV DRIVERS DESIGNS Id s/NEW//; but otherwise this sounds like a useful title. -George
>>> On 18.11.16 at 18:45, <konrad.wilk@oracle.com> wrote: > On Fri, Nov 18, 2016 at 10:06:59AM -0700, Jan Beulich wrote: >> >>> On 18.11.16 at 17:18, <konrad.wilk@oracle.com> wrote: >> > --- a/MAINTAINERS >> > +++ b/MAINTAINERS >> > @@ -304,6 +304,11 @@ X: xen/arch/x86/acpi/lib.c >> > F: xen/drivers/cpufreq/ >> > F: xen/include/acpi/cpufreq/ >> > >> > +PUBLIC INTERFACES >> > +M: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> >> > +S: Supported >> > +F: xen/include/public/ >> >> I think this goes too far: Earlier discussion pinpointed to bottleneck >> to xen/include/public/io/, and that's what I understood would get > > Ah right! Lets change it to that. >> taken care of here (and then perhaps with a slightly different title). > > "Czar" is the only word I heard :-) "Czar" is fine; I meant the "public API" part. Jan
Hey Konrad, it looks like you forgot about this. Cheers, Stefano On Fri, 18 Nov 2016, Konrad Rzeszutek Wilk wrote: > That way we have one person who can: a) poke other maintainers > or pull them in with new drivers are introduced, b) we have > one maintainer who can shepherd the patches along instead of > depending on the REST maintainers which may be busy with > other responsibilities. > > Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> > --- > Cc: Andrew Cooper <andrew.cooper3@citrix.com> > Cc: George Dunlap <George.Dunlap@eu.citrix.com> > Cc: Ian Jackson <ian.jackson@eu.citrix.com> > Cc: Jan Beulich <jbeulich@suse.com> > Cc: Stefano Stabellini <sstabellini@kernel.org> > Cc: Tim Deegan <tim@xen.org> > Cc: Wei Liu <wei.liu2@citrix.com> > --- > MAINTAINERS | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index f0d0202..492f197 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -304,6 +304,11 @@ X: xen/arch/x86/acpi/lib.c > F: xen/drivers/cpufreq/ > F: xen/include/acpi/cpufreq/ > > +PUBLIC INTERFACES > +M: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> > +S: Supported > +F: xen/include/public/ > + > QEMU-DM > M: Ian Jackson <ian.jackson@eu.citrix.com> > S: Supported > -- > 2.9.3 >
diff --git a/MAINTAINERS b/MAINTAINERS index f0d0202..492f197 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -304,6 +304,11 @@ X: xen/arch/x86/acpi/lib.c F: xen/drivers/cpufreq/ F: xen/include/acpi/cpufreq/ +PUBLIC INTERFACES +M: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> +S: Supported +F: xen/include/public/ + QEMU-DM M: Ian Jackson <ian.jackson@eu.citrix.com> S: Supported
That way we have one person who can: a) poke other maintainers or pull them in with new drivers are introduced, b) we have one maintainer who can shepherd the patches along instead of depending on the REST maintainers which may be busy with other responsibilities. Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> --- Cc: Andrew Cooper <andrew.cooper3@citrix.com> Cc: George Dunlap <George.Dunlap@eu.citrix.com> Cc: Ian Jackson <ian.jackson@eu.citrix.com> Cc: Jan Beulich <jbeulich@suse.com> Cc: Stefano Stabellini <sstabellini@kernel.org> Cc: Tim Deegan <tim@xen.org> Cc: Wei Liu <wei.liu2@citrix.com> --- MAINTAINERS | 5 +++++ 1 file changed, 5 insertions(+)