lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Tue, 05 Aug 2008 20:53:15 -0700
From:	Max Krasnyansky <maxk@...lcomm.com>
To:	Paul Jackson <pj@....com>
CC:	mingo@...e.hu, linux-kernel@...r.kernel.org, menage@...gle.com,
	a.p.zijlstra@...llo.nl, vegard.nossum@...il.com,
	lizf@...fujitsu.com
Subject: Re: [PATCH] cpuset: Rework sched domains and CPU hotplug handling
 (2.6.27-rc1)

Paul Jackson wrote:
> Max wrote:
>> But it does seem like an overkill to schedule a workqueue ...
> 
> "overkill" -- by what metric?
> 
> If something is overkill, it means something is excessive.
> Excess (and deficiency) occur along some scale, some metric.
> 
> If not by CPU cycles, then by what metric is it overkill?
> 

Paul, I think you're focusing on the wrong part of my reply ;-).
You are, of course, correct about the overkill metric. I was saying
that saving cycles in that path was not my goal when I wrote
the patch. I just added necessary functions to make existing paths
work they way they currently do and only changed one path because
it had to change due to lock nesting issues.
Along the way simply pointed it out that scheduling work unnecessarily
does seem less efficient. That was not the important part though :)

Max

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ