Mailing List Archive

[PATCH] Cleanup x86_64 boot code
This patch removes unnecessary .org directives from x84_64.S
(xen/arch/x86/boot/x86_64.S). Most of these directives already had
labels associated with them but the code did not use the labels to
reference the locations. This patch now fixes the code to use the
labels instead of the absolute addresses. This makes the code much less
brittle to changes.

Joe
<<64-org-cleanup.patch>>
Re: [PATCH] Cleanup x86_64 boot code [ In reply to ]
This functionally duplicates a patch I had submitted on Aug 23rd. Jan

>>> "Cihula, Joseph" <joseph.cihula@intel.com> 20.09.06 03:53 >>>
This patch removes unnecessary .org directives from x84_64.S
(xen/arch/x86/boot/x86_64.S). Most of these directives already had
labels associated with them but the code did not use the labels to
reference the locations. This patch now fixes the code to use the
labels instead of the absolute addresses. This makes the code much less
brittle to changes.

Joe
<<64-org-cleanup.patch>>

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel
RE: [PATCH] Cleanup x86_64 boot code [ In reply to ]
Perhaps now it will get accepted ;-)

Joe

On Wednesday, September 20, 2006 5:52 PM, Jan Beulich
<mailto:jbeulich@novell.com> wrote:

> This functionally duplicates a patch I had submitted on Aug 23rd. Jan
>
>>>> "Cihula, Joseph" <joseph.cihula@intel.com> 20.09.06 03:53 >>>
> This patch removes unnecessary .org directives from x84_64.S
> (xen/arch/x86/boot/x86_64.S). Most of these directives already had
> labels associated with them but the code did not use the labels to
> reference the locations. This patch now fixes the code to use the
> labels instead of the absolute addresses. This makes the code much
> less
> brittle to changes.
>
> Joe
> <<64-org-cleanup.patch>>

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel
RE: [PATCH] Cleanup x86_64 boot code [ In reply to ]
I suppose it's being queued until after the 3.0.3 branch off. Jan

>>> "Cihula, Joseph" <joseph.cihula@intel.com> 20.09.06 11:53 >>>
Perhaps now it will get accepted ;-)

Joe

On Wednesday, September 20, 2006 5:52 PM, Jan Beulich
<mailto:jbeulich@novell.com> wrote:

> This functionally duplicates a patch I had submitted on Aug 23rd. Jan
>
>>>> "Cihula, Joseph" <joseph.cihula@intel.com> 20.09.06 03:53 >>>
> This patch removes unnecessary .org directives from x84_64.S
> (xen/arch/x86/boot/x86_64.S). Most of these directives already had
> labels associated with them but the code did not use the labels to
> reference the locations. This patch now fixes the code to use the
> labels instead of the absolute addresses. This makes the code much
> less
> brittle to changes.
>
> Joe
> <<64-org-cleanup.patch>>

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel
Re: [PATCH] Cleanup x86_64 boot code [ In reply to ]
It reminded me to pull in a couple of your simpler patches. They probably
won't make the initial 3.0.3 cutoff though, since the branching is currently
in progress.

-- Keir

On 20/9/06 11:03, "Jan Beulich" <jbeulich@novell.com> wrote:

> I suppose it's being queued until after the 3.0.3 branch off. Jan
>
>>>> "Cihula, Joseph" <joseph.cihula@intel.com> 20.09.06 11:53 >>>
> Perhaps now it will get accepted ;-)
>
> Joe
>
> On Wednesday, September 20, 2006 5:52 PM, Jan Beulich
> <mailto:jbeulich@novell.com> wrote:
>
>> This functionally duplicates a patch I had submitted on Aug 23rd. Jan
>>
>>>>> "Cihula, Joseph" <joseph.cihula@intel.com> 20.09.06 03:53 >>>
>> This patch removes unnecessary .org directives from x84_64.S
>> (xen/arch/x86/boot/x86_64.S). Most of these directives already had
>> labels associated with them but the code did not use the labels to
>> reference the locations. This patch now fixes the code to use the
>> labels instead of the absolute addresses. This makes the code much
>> less
>> brittle to changes.
>>
>> Joe
>> <<64-org-cleanup.patch>>
>
> _______________________________________________
> 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