From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj1-f42.google.com (mail-pj1-f42.google.com [209.85.216.42]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 20CFA384233 for ; Thu, 13 Aug 2026 17:31:55 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.42 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786642317; cv=none; b=INsolHWKzksNegBqNlfPimehDNjxBf0DjaGkSQim1DgMJ7fz5fD1+I1NkFXdnogi6/1E6gYxpND7ooc7FbVfkvzqBz+3fln3KsghwBVptgd84KazibpkQuHqzbgZNUEz0X2Mmr+agNpqBjzNyrlp8wfphPAij2UpCFUg+UPFAW0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786642317; c=relaxed/simple; bh=OK+3PSY7/vVg8tPTstwFNKlug+QDc4pfGSyYeQEKZn8=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=Ed9dcW7M/0LNiP0ZKrTx0R1oPy7t/OyxFhVARXRnnI56Fbb3GudnDcrLriyVb9BAuDejaElON0KyrKZ6ElOhq2FR+djjbLTzcS0B4XBEcM+DP1+EYuVYl/Lv5C+R299YDEsO2cIipigm2dO7UxA2t+ywrQEHVQFMZzrHktrJo+4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=iWHSLbQZ; arc=none smtp.client-ip=209.85.216.42 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="iWHSLbQZ" Received: by mail-pj1-f42.google.com with SMTP id 98e67ed59e1d1-38dc4553f62so155724a91.0 for ; Thu, 13 Aug 2026 10:31:55 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1786642315; x=1787247115; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=hHhRM52KdKrRi6gpE3cIbQsE04qBYaQW/Y0DpiPJx5g=; b=iWHSLbQZnFFTq5IIqwBV2/o32D25e71oEmXPTin7AOAIgcc9J/pezSWTPZQx3MIIf9 CjWlBijwkGyxfxInWNdloYIpKK2jH2QdYkwwgfgmNSSygmDKtSo0wcxQmFW9faT2MujV lU56m/VH2EPkDW0J1MrG9qncgPQp9gpKZFzPI4zMkClnGRJvlx0yMB/IRlvgUqhotnlj LwKihQhey3q+8NmVE9kRX+J/2rqhXpVdJvoqG6KyfIQ6Bdm94wln5IqiIw/TcKpjusNT MXc1fxR0+W2P9+O8/Rgg9MTzJsUX+gByliRRvo9fevugwnwDCC93YZmcMnbJRakNgOTQ i9ZQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1786642315; x=1787247115; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to:content-type; bh=hHhRM52KdKrRi6gpE3cIbQsE04qBYaQW/Y0DpiPJx5g=; b=r5jOjsqWccGPSL6D6/iD9T2XxcNrAtSRMwpzOi05Lcwxx4JssJdBVH42XisZiC9rbi nzXtz35xU4IU67qPRlPO+PZNBFKQlpD+HfymIuf+ncxHyANZR2YqUWJeDc0jHuWo0bPw olYhYE0syqu5dWerx4FujS62uctJvngq/VDuZsGdextIADyH5itnkf/Ys058Q6jL6sYD wMIYtstRNFkfsWVAarC09EH8SPdHeSxpb0HqxnbBHvM+kAVJYoWdm64lFh5A5j5Rnh+E gpP4+4J2KuGuUPS0zB5slAtcXIHr/GI988TAUzgYSdKq07LaiGbZjKQ2aiu63DHGlTqZ SjMA== X-Forwarded-Encrypted: i=1; AHgh+Rq/4ccrV4wQauxMV3hVWcZLq6mNfAu48Milmnhfhyk25QQvGms0X7QevfqbiKlGsithhd4Z3HzHAjsdCCI=@vger.kernel.org X-Gm-Message-State: AOJu0Ywo2w5V3xmoefz3HHcxmfOeYSlCHETVhM6DEa8d2nkKCfKW3cvv 0R8AZkcoiG4xNM3j724i7oyF8pR6uVRHouLjWceGYq1UFeLvCju2m8d3 X-Gm-Gg: AR+sD10VDy37cbiR6UUcUnt1acvU1ttm+P5VqKANuGOOB9QccmcAm6vVi+UaSmeSxEM pDsYlN+2RK+z6izrpQKh7AmOTefnPEeHwmWqJzBmSFwSq7FCwNkD922yzAQqIqasG+vqYEx5bUZ HrHT5UbNSVfdcwvbNQh768rpYnbhGvJnwutjWO0QymTPJETWA71yOWreNiCF4RLD7PyJ9xRrCiz 91XC8iWmyh9X1Sj8UT1EM8HVGdSfd3Tzwgfu4mBozdF9xcdl8RvdIlsDF/SGM5oGBIwiRaegao2 VAPc0Cl8+GsDOJJ9lYVeuI9IPovKwmvjDWeOqP6OFI3wf+z1fShNznA7cZAx0yWp2GF3TW4sW8B Iu4ZTYYM17wVsZw6DRC+SSQ9NCffcarM/hGzcN/szziNdnoIyRFdF7RXVTbrPF3yChSaijOWVy5 0H1b/oCkQJO/sHyT/TmE0ceDI4c7MuRPlMjgp1P+5fsmnfEtOXdnzbD9/aLbroqA== X-Received: by 2002:a17:90a:d40b:b0:393:19a3:4f1 with SMTP id 98e67ed59e1d1-3931e058763mr8418570a91.6.1786642315261; Thu, 13 Aug 2026 10:31:55 -0700 (PDT) Received: from phi.nguyendp ([116.86.160.247]) by smtp.gmail.com with ESMTPSA id 5a478bee46e88-31ebc75d8casm10961169eec.3.2026.08.13.10.31.52 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 13 Aug 2026 10:31:54 -0700 (PDT) From: Nguyen Dinh Phi To: Stefano Garzarella , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Simon Horman , Dmitry Torokhov , Andy King , George Zhang Cc: Nguyen Dinh Phi , Wupeng Ma , virtualization@lists.linux.dev, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH v6 3/3] vsock: use sock_error() to consume sk_err after a failed connect Date: Fri, 14 Aug 2026 01:30:20 +0800 Message-ID: <20260813173024.2362935-4-phind.uet@gmail.com> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260813173024.2362935-1-phind.uet@gmail.com> References: <20260813173024.2362935-1-phind.uet@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit vsock_connect() returns sk_err to userspace but does not clear it: if (sk->sk_err) { err = -sk->sk_err; For a blocking connect() the error has already been delivered as connect()'s return value, so leaving it set causes subsequent operations like poll()/epoll() to keep reporting POLLERR even though the connect failure was already delivered. The error should be consumed once it has been returned to userspace. Switch to sock_error(), which reads and clears sk_err atomically, matching the behavior of other protocol implementations such as __inet_stream_connect(). Fixes: d021c344051a ("VSOCK: Introduce VM Sockets") Tested-by: Wupeng Ma Reviewed-by: Stefano Garzarella Signed-off-by: Nguyen Dinh Phi --- net/vmw_vsock/af_vsock.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/net/vmw_vsock/af_vsock.c b/net/vmw_vsock/af_vsock.c index 62e22c4b13c0..e89cb84b8d73 100644 --- a/net/vmw_vsock/af_vsock.c +++ b/net/vmw_vsock/af_vsock.c @@ -1842,12 +1842,10 @@ static int vsock_connect(struct socket *sock, struct sockaddr_unsized *addr, prepare_to_wait(sk_sleep(sk), &wait, TASK_INTERRUPTIBLE); } - if (sk->sk_err) { - err = -sk->sk_err; + err = sock_error(sk); + if (err) { sk->sk_state = TCP_CLOSE; sock->state = SS_UNCONNECTED; - } else { - err = 0; } out_wait: -- 2.53.0