Re: git: a52b30ff98cd - main - sys_pipe: consistently use cr_ruidinfo for accounting of pipebuf
- Reply: Konstantin Belousov : "Re: git: a52b30ff98cd - main - sys_pipe: consistently use cr_ruidinfo for accounting of pipebuf"
- In reply to: Konstantin Belousov : "git: a52b30ff98cd - main - sys_pipe: consistently use cr_ruidinfo for accounting of pipebuf"
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Sat, 21 Sep 2024 11:13:48 UTC
Konstantin Belousov <kib@FreeBSD.org> writes: > commit a52b30ff98cdab82af140285fa7fcdf1036fef27 > Author: Konstantin Belousov <kib@FreeBSD.org> > AuthorDate: 2024-09-20 18:48:19 +0000 > Commit: Konstantin Belousov <kib@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 - des@FreeBSD.org