All of lore.kernel.org
 help / color / mirror / Atom feed
* linux c/s 376
@ 2008-01-18  9:35 Jan Beulich
  2008-01-18  9:43 ` Keir Fraser
  0 siblings, 1 reply; 6+ messages in thread
From: Jan Beulich @ 2008-01-18  9:35 UTC (permalink / raw)
  To: xen-devel

What's the point of modifying arch/x86_64/ia32/ia32entry.S, but not
arch/x86_64/ia32/ia32entry-xen.S in the changeset?

Thanks, Jan

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: linux c/s 376
  2008-01-18  9:35 Jan Beulich
@ 2008-01-18  9:43 ` Keir Fraser
  2008-01-18  9:53   ` Jan Beulich
  0 siblings, 1 reply; 6+ messages in thread
From: Keir Fraser @ 2008-01-18  9:43 UTC (permalink / raw)
  To: Jan Beulich, xen-devel

Yeah, my mistake.

 -- Keir

On 18/1/08 09:35, "Jan Beulich" <jbeulich@novell.com> wrote:

> What's the point of modifying arch/x86_64/ia32/ia32entry.S, but not
> arch/x86_64/ia32/ia32entry-xen.S in the changeset?
> 
> Thanks, Jan
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: linux c/s 376
  2008-01-18  9:43 ` Keir Fraser
@ 2008-01-18  9:53   ` Jan Beulich
  2008-01-18 11:45     ` Keir Fraser
  0 siblings, 1 reply; 6+ messages in thread
From: Jan Beulich @ 2008-01-18  9:53 UTC (permalink / raw)
  To: Keir Fraser; +Cc: xen-devel

Btw., what are this and c/s 375 needed for? Jan

>>> Keir Fraser <Keir.Fraser@cl.cam.ac.uk> 18.01.08 10:43 >>>
Yeah, my mistake.

 -- Keir

On 18/1/08 09:35, "Jan Beulich" <jbeulich@novell.com> wrote:

> What's the point of modifying arch/x86_64/ia32/ia32entry.S, but not
> arch/x86_64/ia32/ia32entry-xen.S in the changeset?
> 
> Thanks, Jan
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com 
> http://lists.xensource.com/xen-devel 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: linux c/s 376
  2008-01-18  9:53   ` Jan Beulich
@ 2008-01-18 11:45     ` Keir Fraser
  0 siblings, 0 replies; 6+ messages in thread
From: Keir Fraser @ 2008-01-18 11:45 UTC (permalink / raw)
  To: Jan Beulich; +Cc: xen-devel


Needed to make Fedora Core 8 boot:
http://lists.xensource.com/archives/html/xen-devel/2008-01/msg00196.html

I assume others who applied the patches referenced in that email must have
correctly applied the patch to ia32entry-xen.S, or it would make no sense
that it fixes the problem.

 -- Keir

On 18/1/08 09:53, "Jan Beulich" <jbeulich@novell.com> wrote:

> Btw., what are this and c/s 375 needed for? Jan
> 
>>>> Keir Fraser <Keir.Fraser@cl.cam.ac.uk> 18.01.08 10:43 >>>
> Yeah, my mistake.
> 
>  -- Keir
> 
> On 18/1/08 09:35, "Jan Beulich" <jbeulich@novell.com> wrote:
> 
>> What's the point of modifying arch/x86_64/ia32/ia32entry.S, but not
>> arch/x86_64/ia32/ia32entry-xen.S in the changeset?
>> 
>> Thanks, Jan
>> 
>> 
>> _______________________________________________
>> Xen-devel mailing list
>> Xen-devel@lists.xensource.com
>> http://lists.xensource.com/xen-devel
> 
> 
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: linux c/s 376
@ 2008-01-18 13:03 Deuce
  2008-01-18 13:27 ` Keir Fraser
  0 siblings, 1 reply; 6+ messages in thread
From: Deuce @ 2008-01-18 13:03 UTC (permalink / raw)
  To: Keir Fraser, Jan Beulich; +Cc: xen-devel

I guess it makes no sense...

I was unaware of ia32entry-xen.S and just applied the patches verbatim.  But it does indeed allow the system to boot.

Jason

----- Original Message ----
> From: Keir Fraser <Keir.Fraser@cl.cam.ac.uk>
> To: Jan Beulich <jbeulich@novell.com>
> Cc: xen-devel@lists.xensource.com
> Sent: Friday, January 18, 2008 6:45:06 AM
> Subject: Re: [Xen-devel] linux c/s 376
> 
> 
> Needed to make Fedora Core 8 boot:
> http://lists.xensource.com/archives/html/xen-devel/2008-01/msg00196.html
> 
> I assume others who applied the patches referenced in that email
> must
> 
 have
> correctly applied the patch to ia32entry-xen.S, or it would make
> no
> 
 sense
> that it fixes the problem.
> 
>  -- Keir
> 
> On 18/1/08 09:53, "Jan Beulich"  wrote:
> 
> > Btw., what are this and c/s 375 needed for? Jan
> > 
> >>>> Keir Fraser  18.01.08 10:43 >>>
> > Yeah, my mistake.
> > 
> >  -- Keir
> > 
> > On 18/1/08 09:35, "Jan Beulich"  wrote:
> > 
> >> What's the point of modifying arch/x86_64/ia32/ia32entry.S, but not
> >> arch/x86_64/ia32/ia32entry-xen.S in the changeset?
> >> 
> >> Thanks, Jan
> >> 
> >> 
> >> _______________________________________________
> >> Xen-devel mailing list
> >> Xen-devel@lists.xensource.com
> >> http://lists.xensource.com/xen-devel
> > 
> > 
> > 
> 
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel
> 




      ____________________________________________________________________________________
Be a better friend, newshound, and 
know-it-all with Yahoo! Mobile.  Try it now.  http://mobile.yahoo.com/;_ylt=Ahu06i62sR8HDtDypao8Wcj9tAcJ 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: linux c/s 376
  2008-01-18 13:03 linux c/s 376 Deuce
@ 2008-01-18 13:27 ` Keir Fraser
  0 siblings, 0 replies; 6+ messages in thread
From: Keir Fraser @ 2008-01-18 13:27 UTC (permalink / raw)
  To: Deuce, Jan Beulich; +Cc: xen-devel

Oh, it does makes sense. The unistd.h change works for 64-bit applications.
It's just that by not fixing ia32entry-xen.S the ppoll/pselect syscalls are
unavailable to 32-bit apps on 64-bit Linux.

 -- Keir


On 18/1/08 13:03, "Deuce" <the_deuce@yahoo.com> wrote:

> I guess it makes no sense...
> 
> I was unaware of ia32entry-xen.S and just applied the patches verbatim.  But
> it does indeed allow the system to boot.
> 
> Jason
> 
> ----- Original Message ----
>> From: Keir Fraser <Keir.Fraser@cl.cam.ac.uk>
>> To: Jan Beulich <jbeulich@novell.com>
>> Cc: xen-devel@lists.xensource.com
>> Sent: Friday, January 18, 2008 6:45:06 AM
>> Subject: Re: [Xen-devel] linux c/s 376
>> 
>> 
>> Needed to make Fedora Core 8 boot:
>> http://lists.xensource.com/archives/html/xen-devel/2008-01/msg00196.html
>> 
>> I assume others who applied the patches referenced in that email
>> must
>> 
>  have
>> correctly applied the patch to ia32entry-xen.S, or it would make
>> no
>> 
>  sense
>> that it fixes the problem.
>> 
>>  -- Keir
>> 
>> On 18/1/08 09:53, "Jan Beulich"  wrote:
>> 
>>> Btw., what are this and c/s 375 needed for? Jan
>>> 
>>>>>> Keir Fraser  18.01.08 10:43 >>>
>>> Yeah, my mistake.
>>> 
>>>  -- Keir
>>> 
>>> On 18/1/08 09:35, "Jan Beulich"  wrote:
>>> 
>>>> What's the point of modifying arch/x86_64/ia32/ia32entry.S, but not
>>>> arch/x86_64/ia32/ia32entry-xen.S in the changeset?
>>>> 
>>>> Thanks, Jan
>>>> 
>>>> 
>>>> _______________________________________________
>>>> Xen-devel mailing list
>>>> Xen-devel@lists.xensource.com
>>>> http://lists.xensource.com/xen-devel
>>> 
>>> 
>>> 
>> 
>> 
>> 
>> _______________________________________________
>> Xen-devel mailing list
>> Xen-devel@lists.xensource.com
>> http://lists.xensource.com/xen-devel
>> 
> 
> 
> 
> 
>       
> ______________________________________________________________________________
> ______
> Be a better friend, newshound, and
> know-it-all with Yahoo! Mobile.  Try it now.
> http://mobile.yahoo.com/;_ylt=Ahu06i62sR8HDtDypao8Wcj9tAcJ
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2008-01-18 13:27 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-01-18 13:03 linux c/s 376 Deuce
2008-01-18 13:27 ` Keir Fraser
  -- strict thread matches above, loose matches on Subject: below --
2008-01-18  9:35 Jan Beulich
2008-01-18  9:43 ` Keir Fraser
2008-01-18  9:53   ` Jan Beulich
2008-01-18 11:45     ` Keir Fraser

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.