linux-kernel - Re: unnecessary timer interrupt of slab.c and bdi tasks when the system is in sleep state

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-next>] [day] [month] [year] [list]

Date:	Thu, 30 Sep 2010 09:13:02 -0500 (CDT)
From:	Christoph Lameter <cl@...ux.com>
To:	"Wu, Xia" <xia.wu@...el.com>
cc:	"akpm@...ux-foundation.org" <akpm@...ux-foundation.org>,
	"viro@...iv.linux.org.uk" <viro@...iv.linux.org.uk>,
	"mingo@...e.hu" <mingo@...e.hu>,
	"peterz@...radead.org" <peterz@...radead.org>,
	"tglx@...utronix.de" <tglx@...utronix.de>,
	"Van De Ven, Arjan" <arjan.van.de.ven@...el.com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"Zhu, Daniel" <daniel.zhu@...el.com>,
	"Wang, Yong Y" <yong.y.wang@...el.com>
Subject: Re: unnecessary timer interrupt of slab.c and bdi tasks when the
 system is in sleep state

On Thu, 30 Sep 2010, Wu, Xia wrote:

> I found some unnecessary timer interrupts when the system enter sleep state.
> (1)     /mm/slab.c
> cache_reap() clean up on allocated memory every 2s. If the system is in sleep state, the system is waked-up when this timer expires. In fact,
> there isn't more slabs to been cleaned up in sleep state.

Right. We could switch off the timer when idle without much of an issue.
The expiration of the caches wont occur and so we will have stale objects
on the queues when we exit sleep state. You could flush the queues before
switching off the timers?

(The alternate allocator SLUB does not use timers)

> I think these timers should not wake-up the system when the system is in
> sleep state. In these cases, these timers only waste the CPU resource
> and consume more power.

Agreed. In addition I would like these timers to also switch off if the
code running on this hardward thread does not engage in any memory
allocation (slab) or if there is no dirty writeback on a cpu. These timers
can cause unexpected latencies and introduce undesirable variability into
processes needing either full processor speed or low latency responses to
events.
--
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/