WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-ia64-devel

[Xen-ia64-devel] [PATCH] initial cleanup of ivt.S

To: xen-ia64-devel <xen-ia64-devel@xxxxxxxxxxxxxxxxxxx>
Subject: [Xen-ia64-devel] [PATCH] initial cleanup of ivt.S
From: Al Stone <ahs3@xxxxxxxxx>
Date: Thu, 10 Aug 2006 17:12:38 -0600
Delivery-date: Thu, 10 Aug 2006 16:12:56 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-ia64-devel-request@lists.xensource.com?subject=help>
List-id: Discussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>
List-post: <mailto:xen-ia64-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=unsubscribe>
Organization: Hewlett-Packard
Reply-to: ahs3@xxxxxxxxx
Sender: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
This patch reflects a patch I recently posted to the linux-ia64
mailing list to do essentially the same thing for ivt.S in upstream
source.

I've reformatted the contents of the Xen version of ivt.S so that
they are now readable on 80-column displays, in accordance with 
Linux coding standards.  This is really just a first pass at 
cleaning up this code.  In subsequent passes, I see several things
need to be done:

   1) Correct any additional typos or misspellings (there were
      quite a few cleaned up in this patch).

   2) Make the formatting consistent (use the same style of
      comments everywhere, same syntax for stop bits,...).

   3) Make sure the upstream and Xen ivt.S files are consistent
      where they need to be.

   4) Do all of the TODOs in the file.

   5) Handle the FIXMEs.

   6) Go through an optimization pass.

In all of this patch, I only changed one line of assembly; there
was a 'mov r31=pr;' in entry 23 (daccess_rights) that I changed
to 'mov r31=pr;;' which is most likely what was meant.  No other
instructions were changed.  Hopefully, I didn't harm any other
white space formatting.

Tested by compiling, running as dom0 and running as domU.

Signed-off-by: Al Stone <ahs3@xxxxxxxxx>

-- 
Ciao,
al
----------------------------------------------------------------------
Al Stone                                      Alter Ego:
Open Source and Linux R&D                     Debian Developer
Hewlett-Packard Company                       http://www.debian.org
E-mail: ahs3@xxxxxxxxx                        ahs3@xxxxxxxxxx
----------------------------------------------------------------------

Attachment: ivt.S-xen-cleanup.patch
Description: Text Data

_______________________________________________
Xen-ia64-devel mailing list
Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-ia64-devel
<Prev in Thread] Current Thread [Next in Thread>