xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: George Dunlap <George.Dunlap@eu.citrix.com>
To: Keir Fraser <keir.fraser@eu.citrix.com>
Cc: Kathy Hadley <Kathy.Hadley@dornerworks.com>,
	Gross <juergen.gross@fujitsu-siemens.com>,
	"xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>
Subject: Re: [PATCH 1/1] Xen ARINC 653 Scheduler (updated to add support for CPU pools)
Date: Wed, 16 Jun 2010 17:25:57 +0100	[thread overview]
Message-ID: <AANLkTilJ6x-QmciYSqsaAAC966LOF8W4SE3aqn3umY97@mail.gmail.com> (raw)
In-Reply-To: <C83EB84E.17B17%keir.fraser@eu.citrix.com>

It applied pretty cleanly for me.  I didn't try to build it though. :-)

 -George

On Wed, Jun 16, 2010 at 5:20 PM, Keir Fraser <keir.fraser@eu.citrix.com> wrote:
> On 16/06/2010 17:14, "George Dunlap" <George.Dunlap@eu.citrix.com> wrote:
>
>>>  I actually tried the xmalloc() method first.  I found that when the
>>> .adjust_global function was called, the address of the "ops" data structure
>>> passed to that function was different from the address of the "ops" data
>>> structure when the .init function was called.  I wanted to use .adjust_global
>>> to modify the data structure that was created when the .init function was
>>> called, but I could not figure out a way to get the address of the second
>>> data structure.  Suggestions?
>>
>> It's been a month or two since I trawled through the cpupools code;
>> but I seem to recall that .init is called twice -- once for the
>> "default pool" (cpupool0), and once for an actually in-use pool.
>> (Juergen, can you correct me if I'm wrong?)  Is it possible that
>> that's the difference in the pointers that you're seeing?
>
> Oh yes, that was the old behaviour. I took a hatchet to the
> scheduler/cpupool interfaces a few weeks ago and now we should only
> initialise the scheduler once, unless extra cpupools are manually created.
> The fact that Kathy is seeing two different ops structures probably
> indicates that her xen-unstable tree is very out of date. Which may also
> mean that the patch will not apply to current tip.
>
>  -- Keir
>
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel
>

  parent reply	other threads:[~2010-06-16 16:25 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-06-16 15:04 [PATCH 1/1] Xen ARINC 653 Scheduler (updated to add support for CPU pools) Kathy Hadley
2010-06-16 15:50 ` George Dunlap
2010-06-16 16:00   ` Kathy Hadley
2010-06-16 16:13     ` Keir Fraser
2010-06-16 16:14     ` George Dunlap
2010-06-16 16:20       ` Keir Fraser
2010-06-16 16:25         ` Kathy Hadley
2010-06-16 16:31           ` Keir Fraser
2010-06-16 16:40             ` Kathy Hadley
2010-06-16 16:49               ` Keir Fraser
2010-06-16 18:03                 ` Kathy Hadley
2010-06-17  7:04                   ` Keir Fraser
2010-06-17 18:16                     ` Kathy Hadley
2010-06-17 18:26                       ` Keir Fraser
2010-06-18 17:35                     ` Kathy Hadley
2010-06-18 17:49                       ` Keir Fraser
2010-06-19 11:14                       ` George Dunlap
2010-06-22 19:10                     ` Kathy Hadley
2010-06-22 19:16                       ` Keir Fraser
2010-06-23 19:57                         ` Kathy Hadley
2010-06-23 20:23                           ` Keir Fraser
2010-06-23 21:16                             ` Kathy Hadley
2010-06-23 22:36                               ` Keir Fraser
2010-06-24 12:53                                 ` Kathy Hadley
2010-06-24 13:08                                   ` Dan Magenheimer
2010-06-24 13:18                                     ` Kathy Hadley
2010-06-24 13:23                                     ` Keir Fraser
2010-06-24 13:32                                       ` Kathy Hadley
2010-06-30 20:44                                 ` Kathy Hadley
2010-06-30 20:54                                   ` Keir Fraser
2010-07-14 17:32                                     ` Kathy Hadley
2010-07-14 18:04                                       ` Keir Fraser
2010-06-16 16:25         ` George Dunlap [this message]
2010-06-17  5:02         ` Juergen Gross
2010-06-17  6:09           ` 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=AANLkTilJ6x-QmciYSqsaAAC966LOF8W4SE3aqn3umY97@mail.gmail.com \
    --to=george.dunlap@eu.citrix.com \
    --cc=Kathy.Hadley@dornerworks.com \
    --cc=juergen.gross@fujitsu-siemens.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).