All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mukesh Rathor <mukesh.rathor@oracle.com>
To: Ian Campbell <Ian.Campbell@eu.citrix.com>
Cc: "xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>,
	Keir Fraser <Keir.Fraser@eu.citrix.com>,
	Jan Beulich <JBeulich@novell.com>
Subject: Re: hypercall_xlat_continuation()
Date: Mon, 01 Jun 2009 17:03:25 -0700	[thread overview]
Message-ID: <4A246C4D.7000605@oracle.com> (raw)
In-Reply-To: <1243590859.5997.59.camel@zakaz.uk.xensource.com>



Ian Campbell wrote:
> On Thu, 2009-05-28 at 23:01 -0400, Mukesh Rathor wrote:
>> yeah, looks like privcmd_ioctl_32() only fixes the wrapper struct. The
>> PFN array still is 32bit pfn's, and privcmd_ioctl() expects 64bits. So,
>> in a dilemma now, not sure if I should fix it up in privcmd_ioctl_32()
>> or change privcmd_ioctl() which will take some time to reverse engineer.
>> Not sure how many things I'll discover, if it's too many, it may
>> not be worth it in the end.
> 
> FWIW here are my very skanky patches from ages ago (patch names are the
> originals if that gives a clue to my opinion of them even then ;-)).
> 
> I don't even recall if they worked properly (or at all), I think I
> remember starting guests so long as they didn't use blktap (which has
> issues with the user<->kernel ring protocol in this scenario).
> 
> There's an outside change there might be something in them which isn't
> complete rubbish.
> 
> Ian.
> 

Hi Ian,

Looks like you also put in quite a bit of effort into it. I'm
realizing that it is more work than it appeared in the beginning,
and probably not worth the gains in the end. So I should also give up.
learnt few new things along the way tho :).

Thanks,
Mukesh

  reply	other threads:[~2009-06-02  0:03 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-22  2:57 hypercall_xlat_continuation() Mukesh Rathor
2009-05-22 16:29 ` hypercall_xlat_continuation() Keir Fraser
2009-05-22 21:58   ` hypercall_xlat_continuation() Mukesh Rathor
2009-05-23 10:17     ` hypercall_xlat_continuation() Keir Fraser
2009-05-23 22:44       ` hypercall_xlat_continuation() Ian Campbell
2009-05-28  2:35         ` hypercall_xlat_continuation() Mukesh Rathor
2009-05-29  3:01           ` hypercall_xlat_continuation() Mukesh Rathor
2009-05-29  9:54             ` hypercall_xlat_continuation() Ian Campbell
2009-06-02  0:03               ` Mukesh Rathor [this message]
2009-05-26 18:43       ` hypercall_xlat_continuation() Mukesh Rathor
2009-05-27  8:02         ` hypercall_xlat_continuation() Jan Beulich
2009-05-27  8:04           ` hypercall_xlat_continuation() Keir Fraser
2009-05-27  8:22             ` hypercall_xlat_continuation() Jan Beulich
2009-05-27  8:58               ` hypercall_xlat_continuation() Keir Fraser
2009-05-23 22:36 ` hypercall_xlat_continuation() Ian Campbell
2009-05-25 11:47   ` hypercall_xlat_continuation() Jan Beulich
2009-05-25 13:07     ` hypercall_xlat_continuation() Keir Fraser

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=4A246C4D.7000605@oracle.com \
    --to=mukesh.rathor@oracle.com \
    --cc=Ian.Campbell@eu.citrix.com \
    --cc=JBeulich@novell.com \
    --cc=Keir.Fraser@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.