All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mukesh Rathor <mukesh.rathor@oracle.com>
To: Jan Beulich <JBeulich@suse.com>
Cc: xen-devel@lists.xenproject.org
Subject: Re: [V1 PATCH 2/3] pvh: fix pirq path for pvh
Date: Tue, 25 Feb 2014 12:44:59 -0800	[thread overview]
Message-ID: <20140225124459.792471cf@mantra.us.oracle.com> (raw)
In-Reply-To: <530C7048020000780011F0D9@nat28.tlf.novell.com>

On Tue, 25 Feb 2014 09:28:24 +0000
"Jan Beulich" <JBeulich@suse.com> wrote:

> >>> On 25.02.14 at 02:03, Mukesh Rathor <mukesh.rathor@oracle.com>
> >>> wrote:
> > Just like hvm, pirq eoi shared page is not there for pvh. pvh should
> > not touch any pv_domain fields.
> 
> While the latter is true, wasn't it that IRQ handling wise PVH is
> using PV mechanisms? In which case the EOI map page would be of
> interest, and rather than guarding the accesses you ought to move
> the field out of pv_domain.

It would be, but my proposals to move the fields out in earlier patches
had been turned down by you. So, while I work on another solution, this
would help. It's on my list to work on. As soon as dom0 pvh patches
are in, I intend to work on fixmes. I don't understand very well whats
going on with pirq eoi, so need to study it both in xen and linux, 
and focussing on one thing at a time really helps mortals like me :) ... 

thanks,
Mukesh

  reply	other threads:[~2014-02-25 20:45 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-25  1:03 [V1 PATCH 0/3] pvh: misc bug fixes Mukesh Rathor
2014-02-25  1:03 ` [V1 PATCH 1/3] pvh: early return from hvm_hap_nested_page_fault Mukesh Rathor
2014-02-25  1:03   ` [V1 PATCH 2/3] pvh: fix pirq path for pvh Mukesh Rathor
2014-02-25  1:03     ` [V1 PATCH 3/3] pvh: disallow PHYSDEVOP_pirq_eoi_gmfn_v2/v1 Mukesh Rathor
     [not found]       ` <530C7362020000780011F0F0@nat28.tlf.novell.com>
     [not found]         ` <530C7663020000780011F104@nat28.tlf.novell.com>
     [not found]           ` <20140225152514.GA4322@konrad-lan.dumpdata.com>
     [not found]             ` <530CD069020000780011F3D1@nat28.tlf.novell.com>
     [not found]               ` <20140225123314.334b7fec@mantra.us.oracle.com>
     [not found]                 ` <530DC4A6020000780011F6C0@nat28.tlf.novell.com>
     [not found]                   ` <20140227190649.47d4eb7a@mantra.us.oracle.com>
     [not found]                     ` <5310524802000078001201B8@nat28.tlf.novell.com>
2014-03-03 12:30                       ` George Dunlap
2014-03-04  7:44                         ` Jan Beulich
2014-02-25  9:28     ` [V1 PATCH 2/3] pvh: fix pirq path for pvh Jan Beulich
2014-02-25 20:44       ` Mukesh Rathor [this message]
2014-02-26  9:44         ` Jan Beulich
2014-02-25  9:26   ` [V1 PATCH 1/3] pvh: early return from hvm_hap_nested_page_fault Jan Beulich
2014-02-27 10:56   ` Tim Deegan
2014-02-28  2:25     ` Mukesh Rathor

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=20140225124459.792471cf@mantra.us.oracle.com \
    --to=mukesh.rathor@oracle.com \
    --cc=JBeulich@suse.com \
    --cc=xen-devel@lists.xenproject.org \
    /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.