While adding the "loading..." dots I noticed this one.
It's a minor change, but I see no reasons to copy bytes when
we can copy four of them at a time.
Yes, I tested this change (like the previous one). It's against
a clean CVS tree.
/alessandro
Index: ChangeLog
===================================================================
RCS file: /cvs/grub/ChangeLog,v
retrieving revision 1.316
diff -u -d -r1.316 ChangeLog
--- ChangeLog 2000/09/08 21:30:16 1.316
+++ ChangeLog 2000/09/08 22:55:08
@@ -1,3 +1,7 @@
+2000-09-09 Alessandro Rubini <[EMAIL PROTECTED]>
+
+ * stage2/start.S (copy_buffer): use movsl instead of movsb
+
2000-09-09 OKUJI Yoshinori <[EMAIL PROTECTED]>
From Alessandro Rubini:
Index: stage2/start.S
===================================================================
RCS file: /cvs/grub/stage2/start.S,v
retrieving revision 1.4
diff -u -d -r1.4 start.S
--- stage2/start.S 2000/09/06 03:02:39 1.4
+++ stage2/start.S 2000/09/08 22:55:09
@@ -280,7 +280,7 @@
address for next time */
/* get the copy length */
- shlw $4, %ax
+ shlw $2, %ax
movw %ax, %cx
/* save addressing regs */
@@ -296,7 +296,7 @@
/* perform copy */
rep /* sets a repeat */
- movsb /* this runs the actual copy */
+ movsl /* this runs the actual copy */
/* restore addressing regs */
popw %di