Remove unsed KQUEUE from usr.bin/make?
Max Okumoto
okumoto at ucsd.edu
Thu Jul 7 11:01:33 PDT 2005
Matthew Dillon wrote:
:Joerg Sonnenberger wrote:
:> On Thu, Jul 07, 2005 at 12:56:48AM -0700, Max Okumoto wrote:
:>
:>>Does anyone object if I remove the KQUEUE code from make. It
:>>is not used, and it does not improve performance. Or atleast
:>>the comments in the code indicate that it does not.
:>
:>
:> It might not improve the performance in the current state.
:> But it is also only partially used for now. You could IMO
:> dramatically simplify the logical since it provides a common
:> place for both IO and child related handling. I think I'd prefer
:> to make (a) the default, (b) remove the non-kqueue code and
:> (c) fully exploit it.
:>
:> Joerg
:
:That makes sense too. I will send mail to phk and ask him what his
:results were.
:
:
: Max
I don't think its going to matter either way. For one thing, this
is 'make', which isn't really a critical utility. Secondly, unless
make is managing a hundred children all at once I really doubt the
difference in performance would be noticeable. But its up to you.
-Matt
Matthew Dillon
<dillon at xxxxxxxxxxxxx>
Yea, thats what I expect. But I want to do the responsable thing. :-)
Most of the time make is just waiting for the children. But its more a
question of which one will result in cleaner code. Ether we go KQUEUE
all the way, or we remove it. Having it ifdef makes the code harder to
maintain.
Max
More information about the Users
mailing list