netdev - Re: [PATCH] vsock/test: Remove redundant semicolons

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAGxU2F4f+Ks6hUNuqvfAJ_kLzjf4sy5CfLkzPJ=Whn7n67-Szw@mail.gmail.com>
Date: Wed, 27 Aug 2025 09:45:30 +0200
From: Stefano Garzarella <sgarzare@...hat.com>
To: Liao Yuanhong <liaoyuanhong@...o.com>
Cc: Joe Damato <joe@...a.to>, Paolo Abeni <pabeni@...hat.com>, 
	Konstantin Shkolnyy <kshk@...ux.ibm.com>, 
	"open list:VM SOCKETS (AF_VSOCK)" <virtualization@...ts.linux.dev>, 
	"open list:VM SOCKETS (AF_VSOCK)" <netdev@...r.kernel.org>, open list <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] vsock/test: Remove redundant semicolons

On Wed, 13 Aug 2025 at 08:57, Liao Yuanhong <liaoyuanhong@...o.com> wrote:
>
> On 8/13/2025 1:45 AM, Joe Damato wrote:
>
> > [You don't often get email from joe@...a.to. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ]
> >
> > On Tue, Aug 12, 2025 at 12:01:15PM +0800, Liao Yuanhong wrote:
> >> Remove unnecessary semicolons.
> >>
> >> Fixes: 86814d8ffd55f ("vsock/test: verify socket options after setting them")
> >> Signed-off-by: Liao Yuanhong <liaoyuanhong@...o.com>
> >> ---
> >>   tools/testing/vsock/util.c | 1 -
> >>   1 file changed, 1 deletion(-)
> >>
> >> diff --git a/tools/testing/vsock/util.c b/tools/testing/vsock/util.c
> >> index 7b861a8e997a..d843643ced6b 100644
> >> --- a/tools/testing/vsock/util.c
> >> +++ b/tools/testing/vsock/util.c
> >> @@ -756,7 +756,6 @@ void setsockopt_ull_check(int fd, int level, int optname,
> >>   fail:
> >>        fprintf(stderr, "%s  val %llu\n", errmsg, val);
> >>        exit(EXIT_FAILURE);
> >> -;
> >>   }
> > This isn't a fixes since it doesn't fix a bug; it's cleanup so I'd probably
> > target net-next and drop the fixes tag.
>
> Do you need me to resend the v2 version without the fixes tag?

Yes, please.

Also target net-next:
https://docs.kernel.org/process/maintainer-netdev.html#indicating-target-tree

Thanks,
Stefano

>
>
> Thanks,
>
> Liao
>