All of lore.kernel.org
 help / color / mirror / Atom feed
From: Simon Horman <horms@kernel.org>
To: Rongguang Wei <clementwei90@163.com>
Cc: netdev@vger.kernel.org, willemdebruijn.kernel@gmail.com,
	jasowangio@gmail.com, andrew+netdev@lunn.ch, davem@davemloft.net,
	kuba@kernel.org, Rongguang Wei <weirongguang@kylinos.cn>
Subject: Re: [PATCH net v5 0/2] tun: fix re-attaching the socket filter
Date: Sat, 10 Oct 2026 09:00:13 +0100	[thread overview]
Message-ID: <20261010080013.GN83879@horms.kernel.org> (raw)
In-Reply-To: <20261008080420.132050-1-clementwei90@163.com>

On Thu, Oct 08, 2026 at 04:04:18PM +0800, Rongguang Wei wrote:
> From: Rongguang Wei <weirongguang@kylinos.cn>
> 
> This is v5 of the series that fixes attaching a queue to a TAP device which
> re-installs the socket filter of a persistent device.
> 
> Patch 1 keeps a copy of the program in the kernel when it is configured,
> instead of reading tun->fprog from user space again on every later attach,
> and fixes the inverted error check in tun_attach(). The two changes are
> one patch on purpose: with only the kernel copy every re-attach returns 0
> without publishing the queue, and with only the check fixed a re-attach
> that used to succeed without a filter starts to fail. IFF_NOFILTER keeps
> working and is no longer needed as a workaround.
> 
> Patch 2 adds selftests: it re-attaches a queue after the buffer the program
> was copied from was made unreadable, attaches a second queue from a fresh
> socket to check that the filter is rebuilt from the kernel copy, and checks
> that a rejected TUNATTACHFILTER keeps the saved program, plus the ways to
> attach a queue without a filter.
> 
> Thanks to Willem de Bruijn for the reviews, and to the CI reviews for
> the comments they raised.
> 
> Rongguang Wei (2):
>   tun: keep a kernel copy of the socket filter program
>   selftests: net: add TAP socket filter attach tests

...

Hi Rongguang,

Unfortunately this patch-set did not apply cleanly to net,
and thus our CI did not run.

Please rebase and repost.

--
pw-bot: changes-requested

      parent reply	other threads:[~2026-10-10  8:00 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-08  8:04 [PATCH net v5 0/2] tun: fix re-attaching the socket filter Rongguang Wei
2026-10-08  8:04 ` [PATCH net v5 1/2] tun: keep a kernel copy of the socket filter program Rongguang Wei
2026-10-08  8:04 ` [PATCH net v5 2/2] selftests: net: add TAP socket filter attach tests Rongguang Wei
2026-10-10  8:00 ` Simon Horman [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20261010080013.GN83879@horms.kernel.org \
    --to=horms@kernel.org \
    --cc=andrew+netdev@lunn.ch \
    --cc=clementwei90@163.com \
    --cc=davem@davemloft.net \
    --cc=jasowangio@gmail.com \
    --cc=kuba@kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=weirongguang@kylinos.cn \
    --cc=willemdebruijn.kernel@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.