]> xenbits.xensource.com Git - mini-os.git/commitdiff
mini-os: Remove \r\n conversion when writing to PV serial port
authorSimon Waterman <watermansrdev@gmail.com>
Wed, 10 May 2017 22:33:23 +0000 (23:33 +0100)
committerWei Liu <wei.liu2@citrix.com>
Thu, 11 May 2017 14:19:54 +0000 (15:19 +0100)
This change removes \n to \r\n conversion when writing to a PV serial
port so that binary protocols carried over the port are not corrupted.

The change enables new use-cases for example connectivity from the Windows
Kernel Debugger to a guest via it's IOEMU stubdom.

Signed-off-by: Simon Waterman <watermansrdev@gmail.com>
Reviewed-by: Wei Liu <wei.liu2@citrix.com>
Acked-by: Samuel Thibault <samuel.thibault@ens-lyon.org>
Acked-by: Keir Fraser <keir@xen.org>
console/console.c

index 5538bd400a19df8fa86065ee764effb714ff85a1..de94ad666dded6eb5443d490b0c6a1552a2f49cd 100644 (file)
@@ -82,10 +82,6 @@ void xencons_tx(void)
 
 void console_print(struct consfront_dev *dev, char *data, int length)
 {
-    char *curr_char, saved_char;
-    char copied_str[length+1];
-    char *copied_ptr;
-    int part_len;
     int (*ring_send_fn)(struct consfront_dev *dev, const char *data, unsigned length);
 
     if(!console_initialised)
@@ -93,30 +89,7 @@ void console_print(struct consfront_dev *dev, char *data, int length)
     else
         ring_send_fn = xencons_ring_send;
 
-    copied_ptr = copied_str;
-    memcpy(copied_ptr, data, length);
-    for(curr_char = copied_ptr; curr_char < copied_ptr+length-1; curr_char++)
-    {
-        if(*curr_char == '\n')
-        {
-            *curr_char = '\r';
-            saved_char = *(curr_char+1);
-            *(curr_char+1) = '\n';
-            part_len = curr_char - copied_ptr + 2;
-            ring_send_fn(dev, copied_ptr, part_len);
-            *(curr_char+1) = saved_char;
-            copied_ptr = curr_char+1;
-            length -= part_len - 1;
-        }
-    }
-
-    if (copied_ptr[length-1] == '\n') {
-        copied_ptr[length-1] = '\r';
-        copied_ptr[length] = '\n';
-        length++;
-    }
-    
-    ring_send_fn(dev, copied_ptr, length);
+    ring_send_fn(dev, data, length);
 }
 
 void print(int direct, const char *fmt, va_list args)