linux-kernel - Re: [PATCH 00/35] cpumask: Replace cpumask

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]

Message-ID: <20081023120322.GC25132@elte.hu>
Date:	Thu, 23 Oct 2008 14:03:22 +0200
From:	Ingo Molnar <mingo@...e.hu>
To:	Mike Travis <travis@....com>
Cc:	Rusty Russell <rusty@...tcorp.com.au>,
	Andrew Morton <akpm@...ux-foundation.org>,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 00/35] cpumask: Replace cpumask_t with struct cpumask


* Mike Travis <travis@....com> wrote:

> [Resubmit: cleanup and rebase on latest tip/master.]
> 
> Redesign cpumask API to explicitly declare struct cpumask pointers to 
> the cpumask_* operators and add functions to make it easier to support 
> struct cpumask pointers on the stack.
> 
> This patchset supplies the infrastructure going forward to implement 
> this new struct cpumask API.  Many more patches (currently 58) have 
> been written and tested to verify the functionality of this API.  
> These will be submitted as soon as they are thoroughly tested.
> 
> Compiled and tested on x86_64.
> 
> Based on tip/master @ v2.6.27-6973-ga90cd11

okay, i've picked up these patches into tip/cpus4096-v2 and started 
testing them.

I fixed the From: line oddities - please holler if they are wrong 
anywhere, we can still rebase this.

Note, i've merged this to _after_ the huge arch/x86/include/asm/ headers 
move which we sent a pull request for earlier today - this will simplify 
logistics.

( I also fixed up a handful of obvious style problems in various places
  - please be more careful about comment structure, whitespaces, etc. -
  they just distract from general review and hurt the merits of your
  patches. )

I also added "Impact:" lines to every commit - a one-line summary of the 
expected outcome of the change. (Please double-check those impact lines 
- if you see anything odd it means that i missed some detail in the 
commit - that will need to be fixed if it happens.)

I've just completed the first basic step of testing: i did 68 kernel 
builds to test bisectability: all 34 commit point builds fine on both 
64-bit and 32-bit as well. (This took some time as almost every commit 
touches cpumask.h, forcing a full kernel rebuild.)

	Ingo
--
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/