Re: [tip:perfcounters/core] x86: Add NMI types for kmap_atomic
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [tip:perfcounters/core] x86: Add NMI types for kmap_atomic
- From: Peter Zijlstra <peterz@xxxxxxxxxxxxx>
- Date: Mon, 15 Jun 2009 20:30:08 +0200
- Cc: Hugh Dickins <hugh.dickins@xxxxxxxxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx, mingo@xxxxxxxxxx, hpa@xxxxxxxxx, paulus@xxxxxxxxx, acme@xxxxxxxxxx, efault@xxxxxx, npiggin@xxxxxxx, tglx@xxxxxxxxxxxxx, linux-tip-commits@xxxxxxxxxxxxxxx, Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
- In-reply-to: <20090615182549.GD11248@xxxxxxx>
- References: <tip-3ff0141aa3a03ca3388b40b36167d0a37919f3fd@xxxxxxxxxxxxxx> <1245077165.6800.497.camel@laptop> <Pine.LNX.4.64.0906151623570.5225@xxxxxxxxxxxxx> <1245080486.6800.561.camel@laptop> <1245089065.13761.19316.camel@twins> <20090615181555.GA11248@xxxxxxx> <1245089943.13761.19334.camel@twins> <20090615182549.GD11248@xxxxxxx>
On Mon, 2009-06-15 at 20:25 +0200, Ingo Molnar wrote:
> * Peter Zijlstra <peterz@xxxxxxxxxxxxx> wrote:
> but ... look at the APIs i propose above. We dont need _any_
> 'types'.
>
> That type enumeration is basically an open-coded allocator. If we do
> a _real_ allocator (a balanced stack of atomic kmaps) we dont need
> any of those indices, and all the potential for mismatch goes away
> as well - a stack nests trivially with IRQ and NMI and arbitrary
> other contexts.
You want types because:
- they encode the intent, and can be verified
- they help keep track of the max nesting depth
In the proposed implementation all type code basically falls away no !
CONFIG_DEBUG_VM, but is kept around for robustness.
--
To unsubscribe from this list: send the line "unsubscribe linux-tip-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
[Index of Archives]
[Linux Stable Commits]
[Linux Stable Kernel]
[Linux Kernel]
[Linux USB Devel]
[Linux Video &Media]
[Linux Audio Users]
[Yosemite News]
[Linux SCSI]