Re: [PATCH v2 1/1] samples/bpf: Fix spelling typo in samples/bpf

Alex Williamson
2025-10-14T20:00:41Z

On Tue, 14 Oct 2025 14:08:49 +0800
Chu Guangqing <chuguangqing@inspur.com> wrote:

> do_hbm_test.sh:
> The comment incorrectly used "upcomming" instead of "upcoming".
>
> hbm.c
> The comment incorrectly used "Managment" instead of "Management".
> The comment incorrectly used "Currrently" instead of "Currently".
>
> tcp_cong_kern.c
> The comment incorrectly used "deteremined" instead of "determined".
>
> tracex1.bpf.c
> The comment incorrectly used "loobpack" instead of "loopback".
>
> mtty.c
> The comment incorrectly used "atleast" instead of "at least".
>
> Signed-off-by: Chu Guangqing <chuguangqing@inspur.com>
> ---
> samples/bpf/do_hbm_test.sh | 2 +-
> samples/bpf/hbm.c | 4 ++--
> samples/bpf/tcp_cong_kern.c | 2 +-
> samples/bpf/tracex1.bpf.c | 2 +-
> samples/vfio-mdev/mtty.c | 2 +-
> 5 files changed, 6 insertions(+), 6 deletions(-)
>
...
> diff --git a/samples/vfio-mdev/mtty.c b/samples/vfio-mdev/mtty.c
> index 59eefe2fed10..6cb3e5974990 100644
> --- a/samples/vfio-mdev/mtty.c
> +++ b/samples/vfio-mdev/mtty.c
> @@ -624,7 +624,7 @@ static void handle_bar_read(unsigned int index, struct mdev_state *mdev_state,
> u8 lsr = 0;
>
> mutex_lock(&mdev_state->rxtx_lock);
> - /* atleast one char in FIFO */
> + /* at least one char in FIFO */
> if (mdev_state->s[index].rxtx.head !=
> mdev_state->s[index].rxtx.tail)
> lsr |= UART_LSR_DR;

I'd suggest this go through bpf since it touches more there. For mtty,

Acked-by: Alex Williamson <alex@shazbot.org>