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
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
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
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
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