Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 1 Jul 2020 11:35:21 +0200
From:      Hans Petter Selasky <hps@selasky.org>
To:        Konstantin Belousov <kostikbel@gmail.com>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r362829 - head/sys/compat/linuxkpi/common/src
Message-ID:  <6478a8b2-9bfc-a671-4087-c2de92fea239@selasky.org>
In-Reply-To: <20200701092123.GI32126@kib.kiev.ua>
References:  <202007010823.0618Nvcu028770@repo.freebsd.org> <20200701092123.GI32126@kib.kiev.ua>

next in thread | previous in thread | raw e-mail | index | archive | help
On 2020-07-01 11:21, Konstantin Belousov wrote:
> It should be expressed as pget(pid, 0); instead of duplicating.

Hi,

Currently the LinuxKPI style is to use tdfind() and pfind(). If you look 
at linux_current.c you see multiple uses of the exact same syntax.

Quickly looking at the pget() implementation, I see it doesn't expand to 
exactly tdfind() and pfind(). pget() uses pfind_tid() which looks 
overkill compared to tdfind(). tdfind() uses a hash-table lookup, while 
pfind_tid() doesn't .... I'm confused.

--HPS





Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?6478a8b2-9bfc-a671-4087-c2de92fea239>