* [PATCH] hw/arm/aspeed: fix connect_serial_hds_to_uarts @ 2024-12-12 13:31 ` kenneth_jia--- via 0 siblings, 0 replies; 8+ messages in thread From: kenneth_jia--- via @ 2024-12-12 13:31 UTC (permalink / raw) To: 'qemu-arm@nongnu.org' Cc: 'Cédric Le Goater', 'Philippe Mathieu-Daudé', 'Jamin Lin', 'Andrew Jeffery', 'Gavin Shan', 'open list:All patches CC here' [-- Attachment #1: Type: text/plain, Size: 1232 bytes --] From 24d3badbbb9dcc0d220609a7dd30f8da5002cba7 Mon Sep 17 00:00:00 2001 From: Kenneth Jia <kenneth_jia@asus.com> Date: Thu, 12 Dec 2024 20:42:04 +0800 Subject: [PATCH] hw/arm/aspeed: fix connect_serial_hds_to_uarts In the loop, we need ignore the index increase when uart == uart_chosen We should increase the index only after we allocate a serial. Signed-off-by: Kenneth Jia <kenneth_jia@asus.com> --- hw/arm/aspeed.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/arm/aspeed.c b/hw/arm/aspeed.c index 556498f2a0..d8cb2d1429 100644 --- a/hw/arm/aspeed.c +++ b/hw/arm/aspeed.c @@ -364,11 +364,11 @@ static void connect_serial_hds_to_uarts(AspeedMachineState *bmc) int uart_chosen = bmc->uart_chosen ? bmc->uart_chosen : amc->uart_default; aspeed_soc_uart_set_chr(s, uart_chosen, serial_hd(0)); - for (int i = 1, uart = sc->uarts_base; i < sc->uarts_num; i++, uart++) { + for (int i = 1, uart = sc->uarts_base; i < sc->uarts_num; uart++) { if (uart == uart_chosen) { continue; } - aspeed_soc_uart_set_chr(s, uart, serial_hd(i)); + aspeed_soc_uart_set_chr(s, uart, serial_hd(i++)); } } -- 2.34.1 [-- Attachment #2: Type: text/html, Size: 5507 bytes --] ^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH] hw/arm/aspeed: fix connect_serial_hds_to_uarts @ 2024-12-12 13:31 ` kenneth_jia--- via 0 siblings, 0 replies; 8+ messages in thread From: kenneth_jia--- via @ 2024-12-12 13:31 UTC (permalink / raw) To: 'qemu-arm@nongnu.org' Cc: 'Cédric Le Goater', 'Philippe Mathieu-Daudé', 'Jamin Lin', 'Andrew Jeffery', 'Gavin Shan', 'open list:All patches CC here' [-- Attachment #1: Type: text/plain, Size: 1232 bytes --] From 24d3badbbb9dcc0d220609a7dd30f8da5002cba7 Mon Sep 17 00:00:00 2001 From: Kenneth Jia <kenneth_jia@asus.com> Date: Thu, 12 Dec 2024 20:42:04 +0800 Subject: [PATCH] hw/arm/aspeed: fix connect_serial_hds_to_uarts In the loop, we need ignore the index increase when uart == uart_chosen We should increase the index only after we allocate a serial. Signed-off-by: Kenneth Jia <kenneth_jia@asus.com> --- hw/arm/aspeed.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/arm/aspeed.c b/hw/arm/aspeed.c index 556498f2a0..d8cb2d1429 100644 --- a/hw/arm/aspeed.c +++ b/hw/arm/aspeed.c @@ -364,11 +364,11 @@ static void connect_serial_hds_to_uarts(AspeedMachineState *bmc) int uart_chosen = bmc->uart_chosen ? bmc->uart_chosen : amc->uart_default; aspeed_soc_uart_set_chr(s, uart_chosen, serial_hd(0)); - for (int i = 1, uart = sc->uarts_base; i < sc->uarts_num; i++, uart++) { + for (int i = 1, uart = sc->uarts_base; i < sc->uarts_num; uart++) { if (uart == uart_chosen) { continue; } - aspeed_soc_uart_set_chr(s, uart, serial_hd(i)); + aspeed_soc_uart_set_chr(s, uart, serial_hd(i++)); } } -- 2.34.1 [-- Attachment #2: Type: text/html, Size: 5507 bytes --] ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] hw/arm/aspeed: fix connect_serial_hds_to_uarts 2024-12-12 13:31 ` kenneth_jia--- via (?) @ 2024-12-13 7:19 ` Cédric Le Goater 2024-12-13 9:48 ` 答复: " kenneth_jia--- via -1 siblings, 1 reply; 8+ messages in thread From: Cédric Le Goater @ 2024-12-13 7:19 UTC (permalink / raw) To: Kenneth Jia(贾纪东_华硕上海), 'qemu-arm@nongnu.org' Cc: 'Cédric Le Goater', 'Philippe Mathieu-Daudé', 'Jamin Lin', 'Andrew Jeffery', 'Gavin Shan', 'open list:All patches CC here' Hello Kenneth, On 12/12/24 14:31, Kenneth Jia(贾纪东_华硕上海) wrote: > From 24d3badbbb9dcc0d220609a7dd30f8da5002cba7 Mon Sep 17 00:00:00 2001 > > From: Kenneth Jia <kenneth_jia@asus.com> > > Date: Thu, 12 Dec 2024 20:42:04 +0800 > > Subject: [PATCH] hw/arm/aspeed: fix connect_serial_hds_to_uarts > > In the loop, we need ignore the index increase when uart == uart_chosen > > We should increase the index only after we allocate a serial. > > Signed-off-by: Kenneth Jia <kenneth_jia@asus.com> Could you please tell us a bit on your test environment ? I am interested to know how you found the issue. Also, please use "git send-email" to send patches. Anyhow, Fixes: d2b3eaefb4d7 ("aspeed: Refactor UART init for multi-SoC machines") Reviewed-by: Cédric Le Goater <clg@redhat.com> Thanks, C. > > --- > > hw/arm/aspeed.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/hw/arm/aspeed.c b/hw/arm/aspeed.c > > index 556498f2a0..d8cb2d1429 100644 > > --- a/hw/arm/aspeed.c > > +++ b/hw/arm/aspeed.c > > @@ -364,11 +364,11 @@ static void connect_serial_hds_to_uarts(AspeedMachineState *bmc) > > int uart_chosen = bmc->uart_chosen ? bmc->uart_chosen : amc->uart_default; > > aspeed_soc_uart_set_chr(s, uart_chosen, serial_hd(0)); > > - for (int i = 1, uart = sc->uarts_base; i < sc->uarts_num; i++, uart++) { > > + for (int i = 1, uart = sc->uarts_base; i < sc->uarts_num; uart++) { > > if (uart == uart_chosen) { > > continue; > > } > > - aspeed_soc_uart_set_chr(s, uart, serial_hd(i)); > > + aspeed_soc_uart_set_chr(s, uart, serial_hd(i++)); > > } > > } > > -- > > 2.34.1 > ^ permalink raw reply [flat|nested] 8+ messages in thread
* 答复: [PATCH] hw/arm/aspeed: fix connect_serial_hds_to_uarts 2024-12-13 7:19 ` Cédric Le Goater @ 2024-12-13 9:48 ` kenneth_jia--- via 2024-12-13 9:56 ` Cédric Le Goater 0 siblings, 1 reply; 8+ messages in thread From: kenneth_jia--- via @ 2024-12-13 9:48 UTC (permalink / raw) To: 'Cédric Le Goater', 'qemu-arm@nongnu.org' Cc: 'Cédric Le Goater', 'Philippe Mathieu-Daudé', 'Jamin Lin', 'Andrew Jeffery', 'Gavin Shan', 'open list:All patches CC here' We plans to use a relatively large number of UART connections, and currently test all UARTs that can be used. When we use qumu for simulation, the problem was discovered. The current code does not allocate char devices with index 5 (label serial5). And even if there are more serial devices, the last one (UAR12) is not connected to serial. Add debug information to watch: diff --git a/hw/arm/aspeed.c b/hw/arm/aspeed.c index 556498f2a0..786df450ed 100644 --- a/hw/arm/aspeed.c +++ b/hw/arm/aspeed.c @@ -369,6 +369,7 @@ static void connect_serial_hds_to_uarts(AspeedMachineState *bmc) continue; } aspeed_soc_uart_set_chr(s, uart, serial_hd(i)); + printf("uart %d char index %d name %s\n", uart-sc->uarts_base, i, serial_hd(i)?serial_hd(i)->filename:"null"); } } Test command ~/office_qemu/build/qemu-system-arm -m 1G -M ast2600-evb,fmc-model=mx66l1g45g -nographic \ -drive file=./${evb}.bin,format=raw,if=mtd,unit=0 \ -drive file=./emmc1.bin,format=raw,if=sd,unit=2 \ -net nic -net nic -net nic -net nic \ -net user,hostfwd=::2443-:443,hostfwd=::2242-:22,hostfwd=udp::2623-:623 \ -serial pty \ -serial pty \ -serial pty \ -serial pty \ -serial pty \ -serial pty \ -serial pty \ -serial pty \ -serial pty \ -serial pty \ -serial pty \ -serial pty \ -serial pty \ -serial pty Output Log: QEMU 9.2.50 monitor - type 'help' for more information char device redirected to /dev/pts/6 (label serial0) char device redirected to /dev/pts/7 (label serial1) char device redirected to /dev/pts/8 (label serial2) char device redirected to /dev/pts/9 (label serial3) char device redirected to /dev/pts/10 (label serial4) char device redirected to /dev/pts/11 (label serial5) char device redirected to /dev/pts/12 (label serial6) char device redirected to /dev/pts/13 (label serial7) char device redirected to /dev/pts/14 (label serial8) char device redirected to /dev/pts/15 (label serial9) char device redirected to /dev/pts/16 (label serial10) char device redirected to /dev/pts/17 (label serial11) char device redirected to /dev/pts/19 (label serial12) char device redirected to /dev/pts/21 (label serial13) char device redirected to /dev/pts/22 (label serial14) uart 0 char index 1 name pty:/dev/pts/7 uart 1 char index 2 name pty:/dev/pts/8 uart 2 char index 3 name pty:/dev/pts/9 uart 3 char index 4 name pty:/dev/pts/10 uart 5 char index 6 name pty:/dev/pts/12 uart 6 char index 7 name pty:/dev/pts/13 uart 7 char index 8 name pty:/dev/pts/14 uart 8 char index 9 name pty:/dev/pts/15 uart 9 char index 10 name pty:/dev/pts/16 uart 10 char index 11 name pty:/dev/pts/17 uart 11 char index 12 name pty:/dev/pts/19 (qemu) Issue: /dev/pts/17 (label serial11) is not connected to a UART, UART12 is not connect to a serial device. -----邮件原件----- 发件人: Cédric Le Goater <clg@kaod.org> 发送时间: 2024年12月13日 15:19 收件人: Kenneth Jia(贾纪东_华硕上海) <kenneth_jia@asus.com>; 'qemu-arm@nongnu.org' <qemu-arm@nongnu.org> 抄送: 'Cédric Le Goater' <clg@redhat.com>; 'Philippe Mathieu-Daudé' <philmd@linaro.o > > Subject: [PATCH] hw/arm/aspeed: fix connect_serial_hds_to_uarts > > In the loop, we need ignore the index increase when uart == > uart_chosen > > We should increase the index only after we allocate a serial. > > Signed-off-by: Kenneth Jia <kenneth_jia@asus.com> Could you please tell us a bit on your test environment ? I am interested to know how you found the issue. Also, please use "git send-email" to send patches. Anyhow, Fixes: d2b3eaefb4d7 ("aspeed: Refactor UART init for multi-SoC machines") Reviewed-by: Cédric Le Goater <clg@redhat.com> Thanks, C. > > --- > > hw/arm/aspeed.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/hw/arm/aspeed.c b/hw/arm/aspeed.c > > index 556498f2a0..d8cb2d1429 100644 > > --- a/hw/arm/aspeed.c > > +++ b/hw/arm/aspeed.c > > @@ -364,11 +364,11 @@ static void > connect_serial_hds_to_uarts(AspeedMachineState *bmc) > > int uart_chosen = bmc->uart_chosen ? bmc->uart_chosen : > amc->uart_default; > > aspeed_soc_uart_set_chr(s, uart_chosen, serial_hd(0)); > > - for (int i = 1, uart = sc->uarts_base; i < sc->uarts_num; i++, > uart++) { > > + for (int i = 1, uart = sc->uarts_base; i < sc->uarts_num; uart++) > +{ > > if (uart == uart_chosen) { > > continue; > > } > > - aspeed_soc_uart_set_chr(s, uart, serial_hd(i)); > > + aspeed_soc_uart_set_chr(s, uart, serial_hd(i++)); > > } > > } > > -- > > 2.34.1 > <p></p> ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: 答复: [PATCH] hw/arm/aspeed: fix connect_serial_hds_to_uarts 2024-12-13 9:48 ` 答复: " kenneth_jia--- via @ 2024-12-13 9:56 ` Cédric Le Goater 2024-12-13 10:11 ` kenneth_jia--- via 0 siblings, 1 reply; 8+ messages in thread From: Cédric Le Goater @ 2024-12-13 9:56 UTC (permalink / raw) To: Kenneth Jia(贾纪东_华硕上海), 'qemu-arm@nongnu.org' Cc: 'Cédric Le Goater', 'Philippe Mathieu-Daudé', 'Jamin Lin', 'Andrew Jeffery', 'Gavin Shan', 'open list:All patches CC here' On 12/13/24 10:48, Kenneth Jia(贾纪东_华硕上海) wrote: > We plans to use a relatively large number of UART connections, and currently test all UARTs that can be used. > When we use qumu for simulation, the problem was discovered. Which QEMU machine are you using ? Thanks, C. ^ permalink raw reply [flat|nested] 8+ messages in thread
* 答复: 答复: [PATCH] hw/arm/aspeed: fix connect_serial_hds_to_uarts 2024-12-13 9:56 ` Cédric Le Goater @ 2024-12-13 10:11 ` kenneth_jia--- via 0 siblings, 0 replies; 8+ messages in thread From: kenneth_jia--- via @ 2024-12-13 10:11 UTC (permalink / raw) To: 'Cédric Le Goater', 'qemu-arm@nongnu.org' Cc: 'Cédric Le Goater', 'Philippe Mathieu-Daudé', 'Jamin Lin', 'Andrew Jeffery', 'Gavin Shan', 'open list:All patches CC here' > Which QEMU machine are you using ? ast2600-evb Thanks, Kenneth -----邮件原件----- 发件人: Cédric Le Goater <clg@kaod.org> 发送时间: 2024年12月13日 17:57 收件人: Kenneth Jia(贾纪东_华硕上海) <kenneth_jia@asus.com>; 'qemu-arm@nongnu.org' <qemu-arm@nongnu.org> 抄送: 'Cédric Le Goater' <clg@redhat.com>; 'Philippe Mathieu-Daudé' <philmd@linaro.org>; 'Jamin Lin' <jamin_lin@aspeedtech.com>; 'Andrew Jeffery' <andrew@codeconstruct.com.au>; 'Gavin Shan' <gshan@redhat.com>; 'open list:All patches CC here' <qemu-devel@nongnu.org> 主题: Re: 答复: [PATCH] hw/arm/aspeed: fix connect_serial_hds_to_uarts External email : Ensure your email is secure before opening links and attachments. On 12/13/24 10:48, Kenneth Jia(贾纪东_华硕上海) wrote: > We plans to use a relatively large number of UART connections, and currently test all UARTs that can be used. > When we use qumu for simulation, the problem was discovered. Which QEMU machine are you using ? Thanks, C. <p></p> ^ permalink raw reply [flat|nested] 8+ messages in thread
* 答复: 答复: [PATCH] hw/arm/aspeed: fix connect_serial_hds_to_uarts @ 2024-12-13 10:11 ` kenneth_jia--- via 0 siblings, 0 replies; 8+ messages in thread From: kenneth_jia--- via @ 2024-12-13 10:11 UTC (permalink / raw) To: 'Cédric Le Goater', 'qemu-arm@nongnu.org' Cc: 'Cédric Le Goater', 'Philippe Mathieu-Daudé', 'Jamin Lin', 'Andrew Jeffery', 'Gavin Shan', 'open list:All patches CC here' > Which QEMU machine are you using ? ast2600-evb Thanks, Kenneth -----邮件原件----- 发件人: Cédric Le Goater <clg@kaod.org> 发送时间: 2024年12月13日 17:57 收件人: Kenneth Jia(贾纪东_华硕上海) <kenneth_jia@asus.com>; 'qemu-arm@nongnu.org' <qemu-arm@nongnu.org> 抄送: 'Cédric Le Goater' <clg@redhat.com>; 'Philippe Mathieu-Daudé' <philmd@linaro.org>; 'Jamin Lin' <jamin_lin@aspeedtech.com>; 'Andrew Jeffery' <andrew@codeconstruct.com.au>; 'Gavin Shan' <gshan@redhat.com>; 'open list:All patches CC here' <qemu-devel@nongnu.org> 主题: Re: 答复: [PATCH] hw/arm/aspeed: fix connect_serial_hds_to_uarts External email : Ensure your email is secure before opening links and attachments. On 12/13/24 10:48, Kenneth Jia(贾纪东_华硕上海) wrote: > We plans to use a relatively large number of UART connections, and currently test all UARTs that can be used. > When we use qumu for simulation, the problem was discovered. Which QEMU machine are you using ? Thanks, C. <p></p> ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] hw/arm/aspeed: fix connect_serial_hds_to_uarts 2024-12-12 13:31 ` kenneth_jia--- via (?) (?) @ 2024-12-13 10:47 ` Cédric Le Goater -1 siblings, 0 replies; 8+ messages in thread From: Cédric Le Goater @ 2024-12-13 10:47 UTC (permalink / raw) To: Kenneth Jia(贾纪东_华硕上海), 'qemu-arm@nongnu.org' Cc: 'Cédric Le Goater', 'Philippe Mathieu-Daudé', 'Jamin Lin', 'Andrew Jeffery', 'Gavin Shan', 'open list:All patches CC here' On 12/12/24 14:31, Kenneth Jia(贾纪东_华硕上海) wrote: > From 24d3badbbb9dcc0d220609a7dd30f8da5002cba7 Mon Sep 17 00:00:00 2001 > > From: Kenneth Jia <kenneth_jia@asus.com> > > Date: Thu, 12 Dec 2024 20:42:04 +0800 > > Subject: [PATCH] hw/arm/aspeed: fix connect_serial_hds_to_uarts > > In the loop, we need ignore the index increase when uart == uart_chosen > > We should increase the index only after we allocate a serial. > > Signed-off-by: Kenneth Jia <kenneth_jia@asus.com> > Applied to aspeed-next. Thanks, C. ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2024-12-13 10:48 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2024-12-12 13:31 [PATCH] hw/arm/aspeed: fix connect_serial_hds_to_uarts kenneth_jia--- via 2024-12-12 13:31 ` kenneth_jia--- via 2024-12-13 7:19 ` Cédric Le Goater 2024-12-13 9:48 ` 答复: " kenneth_jia--- via 2024-12-13 9:56 ` Cédric Le Goater 2024-12-13 10:11 ` 答复: " kenneth_jia--- via 2024-12-13 10:11 ` kenneth_jia--- via 2024-12-13 10:47 ` Cédric Le Goater
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.