xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
* [Xen-devel] [OSSTEST PATCH] ts-xen-install: Drop gdb= parameter
@ 2019-12-19 15:07 Ian Jackson
  2019-12-19 15:10 ` Andrew Cooper
  0 siblings, 1 reply; 2+ messages in thread
From: Ian Jackson @ 2019-12-19 15:07 UTC (permalink / raw)
  To: xen-devel; +Cc: Ian Jackson

This has been there forever and I doubt anyone has ever used it.

Andrew Cooper tells me that it needs an L or H suffix so probably
doesn't work.  Recent changes to more conspicuously report command
line parsing failures highlighted this issue.

Suggested-by: Andrew Cooper <Andrew.Cooper3@citrix.com>
Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>
---
 ts-xen-install | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ts-xen-install b/ts-xen-install
index 154f78c7..d8e9d7e6 100755
--- a/ts-xen-install
+++ b/ts-xen-install
@@ -197,7 +197,7 @@ sub setupboot () {
 				: 'com1');
 
     if ( $cons eq "com1" ) {
-	$xenhopt .= " com1=$c{Baud},8n1 console=com1,vga gdb=com1";
+	$xenhopt .= " com1=$c{Baud},8n1 console=com1,vga";
     } elsif ( $cons eq "dtuart" ) {
 	$xenhopt .= " console=dtuart";
 	my $dtuart= get_host_property($ho, 'XenDTUARTPath', undef);
-- 
2.11.0


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [Xen-devel] [OSSTEST PATCH] ts-xen-install: Drop gdb= parameter
  2019-12-19 15:07 [Xen-devel] [OSSTEST PATCH] ts-xen-install: Drop gdb= parameter Ian Jackson
@ 2019-12-19 15:10 ` Andrew Cooper
  0 siblings, 0 replies; 2+ messages in thread
From: Andrew Cooper @ 2019-12-19 15:10 UTC (permalink / raw)
  To: Ian Jackson, xen-devel

On 19/12/2019 15:07, Ian Jackson wrote:
> This has been there forever and I doubt anyone has ever used it.
>
> Andrew Cooper tells me that it needs an L or H suffix so probably
> doesn't work.  Recent changes to more conspicuously report command
> line parsing failures highlighted this issue.
>
> Suggested-by: Andrew Cooper <Andrew.Cooper3@citrix.com>
> Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>

http://xenbits.xen.org/docs/unstable/misc/xen-command-line.html#gdb

It needs an L or H suffix to instruct Xen how to mux the console with
the gdb remote protocol based on the high bit on 8-bit clean line.

Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2019-12-19 15:10 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-12-19 15:07 [Xen-devel] [OSSTEST PATCH] ts-xen-install: Drop gdb= parameter Ian Jackson
2019-12-19 15:10 ` Andrew Cooper

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).