[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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


  • To: xen-ia64-devel <xen-ia64-devel@xxxxxxxxxxxxxxxxxxx>
  • From: Al Stone <ahs3@xxxxxxxxx>
  • Date: Thu, 10 Aug 2006 17:12:38 -0600
  • Delivery-date: Thu, 10 Aug 2006 16:12:56 -0700
  • List-id: Discussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>

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

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.