Re: [PATCH bpf-next v2 2/5] bpf: Add file descriptor interface for program streams

["Alexei Starovoitov" <alexei.starovoitov@gmail.com>]

On Thu, Sep 24, 2026 at 06:26 PM Kumar Kartikeya Dwivedi <memxor@gmail.com> wrote:
> +static void bpf_stream_publish(struct bpf_stream *stream, int len)
> +{
> +	/* Pairs with atomic_read_acquire() in bpf_stream_readable_bytes(). */
> +	(void)atomic_add_return_release(len, &stream->readable);
> +	bpf_stream_queue_notify(stream);
> +}

This queues irq_work on every bpf_stream_vprintk() and every stage
commit, whether anyone opened the stream or not.
That's an irq_work interrupt per print for every existing user.
ringbuf queues it only when the consumer caught up.

The result of atomic_add_return_release() is thrown away.
can we queue the work only when it returns len, i.e. when the stream
was empty ?

pw-bot: cr