All of lore.kernel.org
 help / color / mirror / Atom feed
From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: Sheng Yang <sheng@linux.intel.com>
Cc: Jeremy Fitzhardinge <jeremy@goop.org>,
	xen-devel@lists.xensource.com,
	Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Subject: Re: [PATCH 0 of 5] PV on HVM Xen
Date: Wed, 17 Mar 2010 10:18:54 -0400	[thread overview]
Message-ID: <20100317141854.GA5802@phenom.dumpdata.com> (raw)
In-Reply-To: <201003171738.37830.sheng@linux.intel.com>

On Wed, Mar 17, 2010 at 05:38:37PM +0800, Sheng Yang wrote:
> On Monday 15 March 2010 20:22:23 Stefano Stabellini wrote:
> > On Mon, 15 Mar 2010, Sheng Yang wrote:
> > > Hi Stefano
> > >
> > > I've checked your patches. And one point puzzled me(I am looking into
> > > pv_ops dom0 code): seems if it depends on PIRQ, it would still require to
> > > do EOI(then result in vmexit) for edge triggered interrupt? (through
> > > xen_pirq_chip->end). And unmask is still a heavy work required
> > > vmexit?(seems it need twice vmexit, even heavier than current HVM
> > > solution)
> > 
> > Only when pirq_needs_eoi(irq) returns true, and this happens when
> > PIRQ_NEEDS_EOI is set.
> > In my patch series I am making sure PIRQ_NEEDS_EOI is never set for any
> > pirq on HVM.
> > 
> OK.
> 
> I would like to work on the PIRQ porting upstream Linux(I would try ACPI but 
> not sure if it would make code much more complex, e.g. cpu hotplug, PCI 
> hotplug... A simple version would be just using PIRQ instead of VIRQ in my 
> patches, but PIRQ is flexible, I would see if I can do more with it). And we 
> may not get a version exactly the same as pv_ops dom0 code in the end... I 

That is OK. There is no problems in ripping out the dom0 code and
building on top of your code.

  reply	other threads:[~2010-03-17 14:18 UTC|newest]

Thread overview: 68+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-03-10 15:46 [PATCH 0 of 5] PV on HVM Xen Stefano Stabellini
2010-03-10 17:33 ` [Xen-devel] " Pasi Kärkkäinen
2010-03-10 17:55   ` Stefano Stabellini
2010-03-10 19:45     ` Jeremy Fitzhardinge
2010-03-12  3:23 ` Sheng Yang
2010-03-12  3:23   ` Sheng Yang
2010-03-12 10:42   ` [Xen-devel] " Stefano Stabellini
2010-03-12 16:00     ` Stefano Stabellini
2010-03-15  4:05       ` Sheng Yang
2010-03-15  8:29         ` Sheng Yang
2010-03-15 12:22           ` Stefano Stabellini
2010-03-17  9:38             ` Sheng Yang
2010-03-17 14:18               ` Konrad Rzeszutek Wilk [this message]
2010-03-17 15:21               ` Stefano Stabellini
2010-03-17 16:13                 ` Jeremy Fitzhardinge
2010-03-18  1:30                   ` Sheng Yang
2010-03-19 20:38                     ` Jeremy Fitzhardinge
2010-03-22  6:26                       ` MSI proposal and work transfer...(was: Re: [PATCH 0 of 5] PV on HVM Xen) Sheng Yang
2010-03-23 20:47                         ` Jeremy Fitzhardinge
2010-03-24  8:19                           ` Sheng Yang
2010-03-23 23:16                         ` Stefano Stabellini
2010-03-24  8:25                           ` Sheng Yang
2010-03-18  2:19                 ` [PATCH 0 of 5] PV on HVM Xen Sheng Yang
2010-03-18 16:42                   ` Jeremy Fitzhardinge
2010-03-17 16:13               ` Jeremy Fitzhardinge
2010-03-15 12:28         ` Stefano Stabellini
2010-03-15 23:08           ` Jeremy Fitzhardinge
2010-03-15 23:24             ` Frank van der Linden
2010-03-16  0:32             ` Dan Magenheimer
2010-03-16  6:09               ` Sheng Yang
2010-03-16 16:46                 ` Dan Magenheimer
2010-03-16 11:07             ` Stefano Stabellini
2010-03-16 17:23               ` Jeremy Fitzhardinge
2010-03-16 17:32                 ` Stefano Stabellini
2010-03-16 17:41                   ` Jeremy Fitzhardinge
2010-03-16 18:06                     ` Stefano Stabellini
2010-03-16 18:26                       ` Jeremy Fitzhardinge
2010-03-16 18:37                         ` Stefano Stabellini
2010-03-17  8:51                           ` Sheng Yang
2010-03-17  9:18                             ` Sheng Yang
2010-03-17 15:17                               ` Stefano Stabellini
2010-03-17 18:20                                 ` Ian Campbell
2010-03-18  1:42                                   ` Sheng Yang
2010-03-18  1:35                                 ` Sheng Yang
2010-03-18 14:22                                   ` Stefano Stabellini
2010-03-18 16:50                                     ` Jeremy Fitzhardinge
2010-03-18 17:30                                 ` Jeremy Fitzhardinge
2010-03-12 21:53     ` [Xen-devel] " Jeremy Fitzhardinge
  -- strict thread matches above, loose matches on Subject: below --
2010-03-12 11:44 Boris Derzhavets
2010-03-12 11:59 ` Stefano Stabellini
2010-03-12 15:01   ` Boris Derzhavets
2010-03-12 15:08     ` Stefano Stabellini
2010-03-12 15:09       ` Boris Derzhavets
2010-03-12 20:10     ` Jeremy Fitzhardinge
2010-03-12 21:14       ` Boris Derzhavets
2010-03-12 21:22         ` Jeremy Fitzhardinge
2010-03-12 21:28           ` Boris Derzhavets
2010-03-12 21:25       ` Boris Derzhavets
2010-03-12 21:29         ` Jeremy Fitzhardinge
2010-03-12 22:08           ` Boris Derzhavets
2010-03-12 22:11             ` Jeremy Fitzhardinge
2010-03-12 22:13               ` Boris Derzhavets
2010-03-12 22:22                 ` Jeremy Fitzhardinge
2010-03-15 15:53                   ` Stefano Stabellini
2010-03-15 16:02                     ` Boris Derzhavets
2010-03-15 17:27                   ` Stefano Stabellini
2010-03-15 17:49                     ` Boris Derzhavets
2010-03-15 18:01                       ` Stefano Stabellini

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=20100317141854.GA5802@phenom.dumpdata.com \
    --to=konrad.wilk@oracle.com \
    --cc=jeremy@goop.org \
    --cc=sheng@linux.intel.com \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=xen-devel@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.