All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Goirand <thomas@goirand.fr>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: George Dunlap <george.dunlap@eu.citrix.com>,
	"jens.axboe@oracle.com" <jens.axboe@oracle.com>,
	"npiggin@suse.de" <npiggin@suse.de>,
	Dan Magenheimer <dan.magenheimer@oracle.com>,
	"xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>,
	"wimcoekaerts@wimmekes.net" <wimcoekaerts@wimmekes.net>,
	"gregkh@suse.de" <gregkh@suse.de>,
	ksrinivasan <ksrinivasan@novell.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"x86@kernel.org" <x86@kernel.org>,
	"jeremy@goop.org" <jeremy@goop.org>,
	David Miller <davem@davemloft.net>,
	Ian Pratt <Ian.Pratt@eu.citrix.com>,
	Stephen Spector <stephen.spector@citrix.com>,
	"avi@redhat.com" <avi@redhat.com>,
	"EAnderson@novell.com" <EAnderson@novell.com>,
	"kurt.hackel@oracle.com" <kurt.hackel@oracle.com>,
	Thomas Gleixner <tglx@linutronix.de>,
	"xen-users@lists.xensource.com" <xen-users@lists.xensource.com>,
	"mingo@elte.hu" <mingo@elte.hu>,
	Keir Fraser <Keir.Fraser@eu.citrix.com>
Subject: Re: [Xen-users] Re: Xen is a feature
Date: Thu, 04 Jun 2009 22:02:03 +0800	[thread overview]
Message-ID: <4A27D3DB.6060704@goirand.fr> (raw)
In-Reply-To: <alpine.LFD.2.01.0906021033230.3351@localhost.localdomain>

Linus Torvalds wrote:
> Seriously.
> 
> If it was just the local APIC, fine. But it may be just the local APIC 
> code this time around, next time it will be something else. It's been TLB, 
> it's been entry_*.S, it's been all over. Some of them are performance 
> issues.
> 
> I dunno. I just do know that I pointed out the statistics for how 
> mindlessly incestuous the Xen patches have historically been to Jeremy. He 
> admitted it. I've not seen _anybody_ say that things will improve. 
> 
> Xen has been painful. If you give maintainers pain, don't expect them to 
> love you or respect you.
> 
> So I would really suggest that Xen people should look at _why_ they are 
> giving maintainers so much pain.
> 
> 		Linus

Seriously, reading this is discouraging. I had to stop myself
criticizing too much this opinion here, but it's kind of hard to read
"mindless", "painful" and such considering the consequences of the
current state.

As time passes, it's becoming more and more unmaintainable to manage the
dom0 patch on one side, and the mainline kernel on the other, even for a
user/admin point of view. THIS is years of mindless and painful
administration/patching tasks. We've all bee waiting too long already.
We need the Xen dom0 "feature" NOW! Not tomorrow, not in one week, not
in 10 years...

As a developer myself (not on the kernel though), I can perfectly
understand the standpoint about ugliness of the code. However, refusing
to merge gives bad headaches to hundreds of people trying to deal and
maintain productions with the issues it creates.

I stand on Steven Rostedt's side (and many others too). Merging WILL
make it possible to have Xen going the way you wish. Otherwise, it's
again a cathedral type of development. Keir Fraser and others seems to
be willing to do the changes in the API if needed. It's just not right
to tell they don't want to. And if there is such need for ABI/API
compatibility, why not just add a config option "compatibility to old
style Xen (dirty hugly slow feature)" if there are some issues?

Now, about merging the Xen hypervisor, that's another discussion that
can happen later on, IMHO. What's URGENT (I insist here) is dom0 support
(including with 64 bits).

Thomas

  parent reply	other threads:[~2009-06-04 14:20 UTC|newest]

Thread overview: 104+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-12 23:25 [GIT PULL] Xen APIC hooks (with io_apic_ops) Jeremy Fitzhardinge
2009-05-12 23:25 ` [PATCH 01/17] xen/dom0: handle acpi lapic parsing in Xen dom0 Jeremy Fitzhardinge
2009-05-12 23:25 ` [PATCH 02/17] x86: add io_apic_ops to allow interception Jeremy Fitzhardinge
2009-05-25  3:54   ` Ingo Molnar
2009-05-27  7:17     ` Jeremy Fitzhardinge
2009-05-12 23:25 ` [PATCH 03/17] xen: implement io_apic_ops Jeremy Fitzhardinge
2009-05-12 23:25 ` [PATCH 04/17] xen: create dummy ioapic mapping Jeremy Fitzhardinge
2009-05-12 23:25 ` [PATCH 05/17] xen: implement pirq type event channels Jeremy Fitzhardinge
2009-05-12 23:25 ` [PATCH 06/17] x86/io_apic: add get_nr_irqs_gsi() Jeremy Fitzhardinge
2009-05-12 23:25 ` [PATCH 07/17] xen/apic: identity map gsi->irqs Jeremy Fitzhardinge
2009-05-12 23:25 ` [PATCH 08/17] xen: direct irq registration to pirq event channels Jeremy Fitzhardinge
2009-05-12 23:25 ` [PATCH 09/17] xen: bind pirq to vector and event channel Jeremy Fitzhardinge
2009-05-12 23:25 ` [PATCH 10/17] xen: pre-initialize legacy irqs early Jeremy Fitzhardinge
2009-05-12 23:25 ` [PATCH 11/17] xen: don't setup acpi interrupt unless there is one Jeremy Fitzhardinge
2009-05-12 23:25 ` [PATCH 12/17] xen: use acpi_get_override_irq() to get triggering for legacy irqs Jeremy Fitzhardinge
2009-05-12 23:25 ` [PATCH 13/17] xen: initialize irq 0 too Jeremy Fitzhardinge
2009-05-12 23:25 ` [PATCH 14/17] xen: dynamically allocate irq & event structures Jeremy Fitzhardinge
2009-05-12 23:25 ` [PATCH 15/17] xen: set pirq name to something useful Jeremy Fitzhardinge
2009-05-12 23:25 ` [PATCH 16/17] xen: fix legacy irq setup, make ioapic-less machines work Jeremy Fitzhardinge
2009-05-12 23:25 ` [PATCH 17/17] xen: disable MSI Jeremy Fitzhardinge
2009-05-19 12:35 ` [GIT PULL] Xen APIC hooks (with io_apic_ops) Ingo Molnar
2009-05-20 17:57   ` Jeremy Fitzhardinge
2009-05-25  4:10     ` Ingo Molnar
2009-05-26 12:46       ` [Xen-devel] " George Dunlap
2009-05-26 18:26         ` Avi Kivity
2009-05-26 19:18           ` Dan Magenheimer
2009-05-26 19:41             ` Avi Kivity
2009-05-28  0:13             ` Ingo Molnar
2009-05-28  0:49               ` Jeremy Fitzhardinge
2009-05-28  3:47               ` Dan Magenheimer
2009-05-28 14:26               ` George Dunlap
2009-05-29  0:45               ` Xen is a feature Jeremy Fitzhardinge
2009-05-29  1:27                 ` Greg KH
2009-05-29  4:05                 ` David Miller
2009-05-29  6:37                   ` Jaswinder Singh Rajput
2009-05-29  6:51                     ` David Miller
2009-05-29 12:01                   ` George Dunlap
2009-05-29 14:14                     ` Pasi Kärkkäinen
2009-05-29 21:29                       ` David Miller
     [not found]                     ` <87tz33ep1b.fsf@basil.nowhere.org>
2009-05-29 21:31                       ` [Xen-devel] " Jeremy Fitzhardinge
2009-05-29 23:09                       ` Nakajima, Jun
2009-05-29 23:26                         ` Jeremy Fitzhardinge
2009-06-02 15:23                     ` Thomas Gleixner
2009-06-02 16:41                       ` George Dunlap
2009-06-02 17:28                         ` Chris Friesen
2009-06-02 17:46                         ` Linus Torvalds
2009-06-02 18:02                           ` Linus Torvalds
2009-06-02 18:59                             ` Avi Kivity
2009-06-07  9:13                               ` Ingo Molnar
2009-06-07 10:01                                 ` Avi Kivity
2009-06-07 10:35                                   ` Ingo Molnar
2009-06-07 12:46                                     ` Avi Kivity
2009-06-07 13:02                                       ` Jaswinder Singh Rajput
2009-06-04 14:02                           ` Thomas Goirand [this message]
2009-06-02 18:59                         ` Thomas Gleixner
2009-06-03 19:49                       ` Bill Davidsen
2009-06-03 20:20                         ` Thomas Gleixner
2009-06-03 22:37                           ` Bill Davidsen
2009-06-03 23:29                             ` Frans Pop
2009-06-04 13:21                               ` George Dunlap
2009-06-04 15:10                                 ` Theodore Tso
2009-06-04 15:31                                 ` Chris Friesen
2009-06-05  4:14                               ` Bill Davidsen
2009-06-05  4:55                                 ` Chris Friesen
2009-06-02 22:40                     ` Steven Rostedt
2009-06-02 23:28                       ` Merge Xen (the hypervisor) into Linux Ingo Molnar
2009-06-03  0:00                         ` Dan Magenheimer
2009-06-03  0:32                           ` Thomas Gleixner
2009-06-03  2:43                           ` Theodore Tso
2009-06-03  3:42                             ` Steven Rostedt
2009-06-03  4:49                               ` Dan Magenheimer
2009-06-03  4:58                                 ` David Miller
2009-06-03  5:07                                   ` Steven Rostedt
2009-06-03  5:22                                 ` Steven Rostedt
2009-06-03 12:03                                   ` George Dunlap
2009-06-03 19:05                                     ` Theodore Tso
     [not found]                                       ` <4A27CF94.1050903@gmx.de>
2009-06-04 14:03                                         ` [Xen-users] " Steven Rostedt
2009-06-03  7:28                             ` Gerd Hoffmann
2009-06-03  8:47                               ` Alan Cox
2009-06-03  9:09                                 ` Gerd Hoffmann
2009-06-03  9:20                                   ` Keir Fraser
2009-06-03 11:15                                   ` Theodore Tso
2009-06-03 11:39                                     ` Keir Fraser
2009-06-03 11:41                                     ` Gerd Hoffmann
2009-06-03  1:00                         ` Joel Becker
2009-06-03  2:00                           ` david
2009-06-03  7:59                           ` Alan Cox
2009-06-03  8:07                         ` Christian Tramnitz
2009-06-04 18:53                           ` Linus Torvalds
2009-06-05  0:09                             ` Samuel Thibault
2009-06-05  0:18                               ` David Miller
2009-06-05  0:54                               ` Linus Torvalds
2009-06-03 17:31                         ` Chris Friesen
2009-06-03 17:36                           ` Alan Cox
2009-06-02 23:41                       ` Xen is a feature Thomas Gleixner
2009-05-30  2:19                 ` [Xen-devel] " Andy Burns
2009-05-26 21:19         ` [Xen-devel] Re: [GIT PULL] Xen APIC hooks (with io_apic_ops) Gerd Hoffmann
2009-05-27 10:14           ` George Dunlap
2009-05-24 20:10   ` Avi Kivity
2009-05-25  3:51     ` Ingo Molnar
2009-05-25  4:55       ` Avi Kivity
2009-05-25  5:06         ` Ingo Molnar
2009-05-25  5:12           ` Avi Kivity
2009-05-25  5:19             ` Ingo Molnar

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4A27D3DB.6060704@goirand.fr \
    --to=thomas@goirand.fr \
    --cc=EAnderson@novell.com \
    --cc=Ian.Pratt@eu.citrix.com \
    --cc=Keir.Fraser@eu.citrix.com \
    --cc=avi@redhat.com \
    --cc=dan.magenheimer@oracle.com \
    --cc=davem@davemloft.net \
    --cc=george.dunlap@eu.citrix.com \
    --cc=gregkh@suse.de \
    --cc=jens.axboe@oracle.com \
    --cc=jeremy@goop.org \
    --cc=ksrinivasan@novell.com \
    --cc=kurt.hackel@oracle.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=npiggin@suse.de \
    --cc=stephen.spector@citrix.com \
    --cc=tglx@linutronix.de \
    --cc=torvalds@linux-foundation.org \
    --cc=wimcoekaerts@wimmekes.net \
    --cc=x86@kernel.org \
    --cc=xen-devel@lists.xensource.com \
    --cc=xen-users@lists.xensource.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.