From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.remote-ip=74.125.82.171 ARC-Seal: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; cv=none; t=1792686752; b=HPIOcrSXick7v9oEhn5dqi0/CBMdpCGpElIOTYUnLxTUTiYbs6xY9XeoDOYeSrfueOnf kflr3K+1rGY0Fr2pWvxu85nq16Ickeg2gon8ZyDh12J8bqo4yaWVQUiBSm90pXcLFKVKr 9dOVnH/2C5A3Ol/ObMfMnyyy4fWBPMz39+CNU/27PRc79QOb/cDaDi1SXB+ACLP/bOb5Y X65u2peIwgiRQcr4rsoox5bZSVBrNQ6TOjETXHWRWGbJIoF2LvOhUbTj0czyS7jkJPS6P ye2P00zmQZNlAKnf/AnUKnS3pjYTLd4kb0bUu6+4++VLWQ0lfD39mCf41gKOLWXMbhA== ARC-Message-Signature: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; c=relaxed/relaxed; t=1792686752; h=DMARC-Filter:Received:DKIM-Signature:X-Google-DKIM-Signature: X-Forwarded-Encrypted:X-Gm-Message-State:X-Gm-Gg:X-Received:Received: From:To:Cc:Subject:Date:Message-ID:X-Mailer:In-Reply-To:References: MIME-Version:Content-Transfer-Encoding; bh=dyDoCHAK3x3k2IkUzfNota+vU5adkAbdwanSXQJIy3c=; b=cyci/fR9zTnAZI0D/1kB5ioxkzv2NV8qReWF4RPbW11ktsz4OrCBflSGIXK3xd2P4Rvs pNcFexbEK6BfsbzLzaGgDoPd3dAC1Gd4H0Llo054czyKwhUKSs9eV33Mn+E8QrngDAKTU qYH88nL0Gka/+Dsv6WEwxtKDsDuDUycEaI2pKnl5O6nP7zd+9EBoJRXkqrzs7t5ImuXwV A103jUqA7Ps7SS8dfp2nHwUb26aQwn4xtfp73aGTd4Uh/WtZmE/e1+qGzsmp/4G1hb1Hy iJxfF4KHOorF5KU9BJZuLP7mpM234fj0iaSEM9vt3rzcNI5LGasrhHeKF2Yz21dG6SQ== ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256 header.s=20251104 header.b=nlgT2q1R; arc=none smtp.remote-ip=74.125.82.171 Received: from mail-dy1-f171.google.com (mail-dy1-f171.google.com [74.125.82.171]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 343AC606A31B for ; Thu, 22 Oct 2026 16:32:32 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.2 smtp.subspace.kernel.org 343AC606A31B 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, unprotected) header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256 header.s=20251104 header.b=nlgT2q1R Received: by mail-dy1-f171.google.com with SMTP id 5a478bee46e88-33fb4680717so4965519eec.1 for ; Thu, 22 Oct 2026 09:32:32 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1792686751; x=1793291551; 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=dyDoCHAK3x3k2IkUzfNota+vU5adkAbdwanSXQJIy3c=; b=nlgT2q1RTU0EH+YxL4z+hsL4T+7TlQvAkGhXm/A3gBiBq1OjpdykFD0p79P+2QHVXp rrRmUnE13x6JIjeqAVMN86RahXl7yQZrxVVvmdH/h3z701xAU3dJkZMO16qz1XzHWXwb yvWxB5Dz/qX8uF52sjXSeZYPJDeDbcgeZU1+MUNol869hzum1Olo40CBOrZyYUs873o1 d2JfUi0Atheb1KdNQAd4ROzSnSe9hAl6UfeSPcuJai48Apf3no8HkCxQgOWlYtlUSBsd BQlVpeu2F6rRyR4+kHtPpl0iu72eMIo1cY+IV1lD/TqySeNzClg2LJWDXn8biN9HVq8N Px3g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1792686751; x=1793291551; 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=dyDoCHAK3x3k2IkUzfNota+vU5adkAbdwanSXQJIy3c=; b=truP/IvRlRjylJdcSA/hE1JQ79MuZFSi0m9y8SFZnQUDk7YDBRFFk7YeQqLCCRpBpk j1RjKTtaavo7R3HwMVB/VV5GfRltDTgs48/S5QDCwflXIsaNbt7NbRsztg3b2F4L/QET cTlzckCzbqZEmhyHWp2KkDq6XS0zHwnfoAAU9k68Eflxel6+Qkz0v7a5NTeb1Xvvc4Hy /A57GBvI4Tm6uJJKwSRCFfxhgmxTbhwSdG8OYJhf51eS1tkaJBiXKjNqtbEWGux96PRb m571dsR4+vr/ph/EEKgUNlpYFwf/FM9jhL4RarNLa/hSp6YJnhZNqzc7p9+EwcHjOPlF hd0Q== X-Forwarded-Encrypted: i=1; AKwUvBzV7TgcQ1sDvx2PdGieboEf/Zpfuq/AdN9dQUPgTFJlLSGx47Ld31TxAzaTHNXc0Yxq/YDJ6AM=@vger.kernel.org X-Gm-Message-State: AFq9FYJk0vYnDhEu6RJmU66RrYJRN2coE+W1JBpTwpUb7jJG9Yv4/WlL BSp6b1ze8piRQbTimNqQLCg8Yj8lJSIXkKiOcV3tGWRXaXBIGpR8IWQF X-Gm-Gg: AYBFou1rdpba+4+DQlKCW+JjHrMDUWrPJuv4bJuQp66uo6SK/HlbYKHVkdzl3uv0ckd j6uUeyykRwd7V9i2LMAJwxr5gwuxZUrDizqNtR7t8JZTht0MTN4d++1zrhcdBmbOAQlApT7xxMG YJjgYDkYm5ezAMvGlXJeJOpi49xwLd9RiTGa61lb+bvKw6lQUcMo2dc5hMbEdhbpLETaR03PH/I dmxF+cGj/ZZ9DlDldGpd42szn9VClclgSxW8m387256c/2zyvtCcRjVXykF0ri+ZCGk1LGU1w/l noZxJEoNzJROEMB1aXaAuIUY2n8wkuzdMB5rbk7W+R2YWVIrJoYRh70XAmdpfSuaave0xA7Vp85 i6N6guLd6dJ/H001x69hEoaleaDY4rI5yaRzucib/PmrHivkcb/BKSJLl2HFjkUJTlsJ7RszTUU CAiTID6WAgXcw2G6FNp3WJTXJTur/2QlvU9NlfFNVdZefwGbLyz1VG/M8xZc+jHaXR22Qzqumoi uDnIcidi6Ox5Wd9zGVZWO3U3X28LHwgivgiBzICw7BPQ2JOhR/AHyNQp8PNHLESA/j7U2GEBopR mzugXVS+cXdEpYwr X-Received: by 2002:a05:7301:18a1:b0:359:9302:a526 with SMTP id 5a478bee46e88-3674c93ba42mr7166066eec.7.1792686751237; Thu, 22 Oct 2026 09:32:31 -0700 (PDT) Received: from DESKTOP-NM9EKIA ([125.134.240.130]) by smtp.gmail.com with ESMTPSA id 5a478bee46e88-36741e71eaesm37214243eec.26.2026.10.22.09.32.27 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 22 Oct 2026 09:32:30 -0700 (PDT) From: Joonhoe Kim <26rote@gmail.com> To: netdev-bot+sashiko@kernel.org Cc: mani@kernel.org, davem@davemloft.net, edumazet@kernel.org, kuba@kernel.org, pabeni@redhat.com, horms@kernel.org, christopher.lew@oss.qualcomm.com, deepak.singh@oss.qualcomm.com, pranav.phansalkar@oss.qualcomm.com, andersson@kernel.org, linux-arm-msm@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, regressions@lists.linux.dev, stable@vger.kernel.org Subject: Re: [PATCH net] net: qrtr: ns: retry service announcements on -EAGAIN Date: Fri, 23 Oct 2026 01:32:23 +0900 Message-ID: <20261022163226.1091-1-26rote@gmail.com> X-Mailer: git-send-email 2.55.0.windows.5 In-Reply-To: <179250109438.3135.5278679700614096504@kernel.org> References: <20261016124600.31-1-26rote@gmail.com> <179250109438.3135.5278679700614096504@kernel.org> X-Mailing-List: stable@vger.kernel.org List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list MIME-Version: 1.0 Content-Transfer-Encoding: 8bit On Tue, 20 Oct 2026 12:58:14 +0000, Sashiko AI review wrote: > - [High] The fix only covers one of the two paths that send NEW_SERVER. Agreed. A service registered after the remote's HELLO has been handled but before our HELLO goes out is broadcast from ctrl_cmd_new_server(), and qrtr_bcast_enqueue() drops it without telling the caller. If the local node had no services when the HELLO was handled, v1 has nothing to retry. > Could this be handled in the core instead of by polling in one caller? > For example, packets could be queued in qrtr_node_enqueue(), or the > announcements replayed once hello_sent becomes true. v2 does the second, as you suggested: the core records that the remote has sent a HELLO, and when our HELLO has been sent to such a node, it passes a HELLO from that node to the local name service again. The name service then announces every service it has at that point, which also covers the dropped NEW_SERVER broadcasts. > The same window opens again after qrtr_endpoint_hello() clears hello_sent > on resume, because the nid stays assigned. The flag is not cleared on resume, so the replay also happens after the HELLO that qrtr_endpoint_hello() schedules. > DEL_SERVER broadcasts from > server_del() are dropped the same way. v2 does not replay these. A service that is removed before our HELLO goes out has not been announced to the remote either, and on resume the remote has been reset, so it has no entry to remove. > - [Medium] The new retry loop in announce_servers() (ns.c:232-236) can > stall the whole name service for about 10 s per remote HELLO. Agreed, this is the same problem Mani pointed out. Nothing sleeps in the name service worker in v2. > A HELLO that fails with -ETIMEDOUT or -ENOMEM leaves hello_sent false > for good. That is a separate issue in the core, and v2 does not change it. With hello_sent false nothing but HELLO reaches the node, so the replay would not help there. > - [Medium] The NS side gives up after a fixed 10 s, but the core keeps > retrying our HELLO with no limit. Agreed. Any limit in the name service can expire before the core gets our HELLO out. v2 has no limit: the replay follows whenever our HELLO is sent. > - [Medium] A retry can re-advertise a service that was closed during the > retry window, and the remote then keeps the stale entry. v2 has no retry window, but the replay is one more HELLO for the name service to handle, and the ordering race you describe applies to it too: a port closed after the replay is queued but before the name service handles it is still announced. v2 does not fix that race. v2: https://lore.kernel.org/all/20261022163116.1042-1-26rote@gmail.com/ pw-bot: cr Thanks, Joonhoe Kim