Konstantin Belousov <k...@freebsd.org> writes:
> commit a52b30ff98cdab82af140285fa7fcdf1036fef27
> Author: Konstantin Belousov <k...@freebsd.org>
> AuthorDate: 2024-09-20 18:48:19 +0000
> Commit: Konstantin Belousov <k...@freebsd.org>
> CommitDate: 2024-09-20 21:08:51 +0000
>
> sys_pipe: consistently use cr_ruidinfo for accounting of pipebuf
>
> Tested by: yasu
> Sponsored by: The FreeBSD Foundation
> MFC after: 1 week
> ---
> sys/kern/sys_pipe.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/sys/kern/sys_pipe.c b/sys/kern/sys_pipe.c
> index 2b81d08f7077..6a5d150423bc 100644
> --- a/sys/kern/sys_pipe.c
> +++ b/sys/kern/sys_pipe.c
> @@ -1677,7 +1677,7 @@ pipe_free_kmem(struct pipe *cpipe)
>
> if (cpipe->pipe_buffer.buffer != NULL) {
> atomic_subtract_long(&amountpipekva, cpipe->pipe_buffer.size);
> - chgpipecnt(cpipe->pipe_pair->pp_owner->cr_uidinfo,
> + chgpipecnt(cpipe->pipe_pair->pp_owner->cr_ruidinfo,
> -cpipe->pipe_buffer.size, 0);
> vm_map_remove(pipe_map,
> (vm_offset_t)cpipe->pipe_buffer.buffer,
This appears to be the opposite of the patch which you posted on
-current and which yasu@ tested (minus the bit that you committed as
af96ccc6a508). That patch replaced cr_ruidinfo with cr_uidinfo in three
places; this patch replaces cr_uidinfo with cr_ruidinfo somewhere else.
DES
--
Dag-Erling Smørgrav - d...@freebsd.org
