Re: [PATCH 00/13] UAPI header file split

2012-08-02 Thread Paul E. McKenney
On Fri, Jul 20, 2012 at 10:56:37PM +0100, David Howells wrote: > > Here's the second installment of patches from step 1 of my plan below to clean > up the kernel header files and sort out the inclusion recursion problems. > > Note that these patches will need regenerating if the header files they

Re: [PATCH 00/13] UAPI header file split

2012-07-27 Thread Michael Kerrisk
On Thu, Jul 26, 2012 at 12:46 PM, David Howells wrote: > Michael Kerrisk wrote: > >> >> >> 3. HEADER COMMENTS NOT RETAINED IN KAPI FILES >> > >> > How about the attached changes? This is a delta to the disintegrate >> > markers >> > diff I sent earlier. >> >> That looks about right to me. >> >>

Re: [PATCH 00/13] UAPI header file split

2012-07-26 Thread David Howells
Michael Kerrisk wrote: > I haven't looked over the changes yet, but what do my scripts now say? > (If all's well, they generate no output beyond the list of files.) Okay, the comparator script gives me: warthog>sh /tmp/mtk-cmp.sh include/linux/irqnr.h include/uapi/linu

Re: [PATCH 00/13] UAPI header file split

2012-07-26 Thread Michael Kerrisk
On Thu, Jul 26, 2012 at 4:32 PM, David Howells wrote: > Michael Kerrisk wrote: > >> I've not checked whether any of the below are fixed in the adjustments >> that you made in the last 20 hours (though it looks like at least some >> of them are not), but a little scripting to check the content of

Re: [PATCH 00/13] UAPI header file split

2012-07-26 Thread David Howells
Michael Kerrisk wrote: > I've not checked whether any of the below are fixed in the adjustments > that you made in the last 20 hours (though it looks like at least some > of them are not), but a little scripting to check the content of the > split files showed that while most of them were okay, i

Re: [PATCH 00/13] UAPI header file split

2012-07-26 Thread Michael Kerrisk
On Thu, Jul 26, 2012 at 12:46 PM, David Howells wrote: > > Oh, and thanks for the write up! You're welcome. You may want to respond to comments that appear there, such as this one https://lwn.net/Articles/508203/ (You can sign up to receive comments on a specific article by mail) -- Michael Ke

Re: [PATCH 00/13] UAPI header file split

2012-07-26 Thread Michael Kerrisk
David, I've not checked whether any of the below are fixed in the adjustments that you made in the last 20 hours (though it looks like at least some of them are not), but a little scripting to check the content of the split files showed that while most of them were okay, in the cases below, some c

Re: [PATCH 00/13] UAPI header file split

2012-07-26 Thread David Howells
Oh, and thanks for the write up! David -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [PATCH 00/13] UAPI header file split

2012-07-26 Thread David Howells
Michael Kerrisk wrote: > >> >> 3. HEADER COMMENTS NOT RETAINED IN KAPI FILES > > > > How about the attached changes? This is a delta to the disintegrate markers > > diff I sent earlier. > > That looks about right to me. > > Acked-by: Michael Kerrisk Excellent, thanks. The question is where

Re: [PATCH 00/13] UAPI header file split

2012-07-25 Thread David Howells
David Howells wrote: > > >> 3. HEADER COMMENTS NOT RETAINED IN KAPI FILES > > How about the attached changes? This is a delta to the disintegrate markers > diff I sent earlier. See the duplicated-important-banners tag in my GIT tree. David -- To unsubscribe from this list: send the line "unsu

Re: [PATCH 00/13] UAPI header file split

2012-07-25 Thread David Howells
David Howells wrote: > Okay... Dealt with that in the script. The following command: > > git diff uapi-post-split-20120724 > > shows the attached. See tag removed-left-over-markers in my GIT tree. David -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the

Re: [PATCH 00/13] UAPI header file split

2012-07-25 Thread David Howells
Michael Kerrisk wrote: > >> 4. DISINTEGRATE MARKERS LEFT OVER (?) > >> > >> Some of the DISINTEGRATE markers that you create during the scripting > >> process are left in the final uapi files. Was this intentional? > > > > Ummm... no, there shouldn't be any. > > > > Certainly the marker has worke

Re: [PATCH 00/13] UAPI header file split

2012-07-25 Thread David Howells
Michael Kerrisk wrote: > >> 2. EMPTY UAPI HEADERS > >> > >> Some of the resulting uapi header files are empty: > >> ... > >> I imagine this should be reasonably easy to fix. > > > > Fix how? The Kbuild files say these headers must exist in UAPI space, but > > the __KERNEL__ guards therein don't

Re: [PATCH 00/13] UAPI header file split

2012-07-25 Thread Michael Kerrisk
On Wed, Jul 25, 2012 at 12:23 PM, David Howells wrote: > Michael Kerrisk wrote: > >> A few other points that I noticed now... >> >> 1. GIT HISTORY COULD BE RETAINED IN SOME CASES >> ... >> But, as currently scripted the "new" uapi header file does not carry >> over the git history of the old "kap

Re: [PATCH 00/13] UAPI header file split

2012-07-25 Thread David Howells
Michael Kerrisk wrote: > A few other points that I noticed now... > > 1. GIT HISTORY COULD BE RETAINED IN SOME CASES > ... > But, as currently scripted the "new" uapi header file does not carry > over the git history of the old "kapi" header, even though it is an > exact duplicate of that file.

Re: [PATCH 00/13] UAPI header file split

2012-07-25 Thread Michael Kerrisk
On Tue, Jul 24, 2012 at 3:19 PM, David Howells wrote: > Michael Kerrisk wrote: > >> In the uapi-split branch, there are now 44 empty Kbuild files. Was >> that intended? Or, should these files rather be removed by your >> patches? > > To be removed by a later patch, I think. Getting rid of some o

Re: [PATCH 00/13] UAPI header file split

2012-07-24 Thread David Howells
Michael Kerrisk wrote: > In the uapi-split branch, there are now 44 empty Kbuild files. Was > that intended? Or, should these files rather be removed by your > patches? To be removed by a later patch, I think. Getting rid of some of them isn't trivial - ones in arch/x/include/asm/Kbuild for exa

Re: [PATCH 00/13] UAPI header file split

2012-07-24 Thread Michael Kerrisk
On Fri, Jul 20, 2012 at 11:56 PM, David Howells wrote: [...] > === > IMPLEMENTING STEP 1 > === > > The patches actually posted here are the manual preparation for the UAPI split > in step (1) above. I haven't posted the patches that do the actual splitting > by e

Re: [PATCH 00/13] UAPI header file split

2012-07-23 Thread Arnd Bergmann
On Friday 20 July 2012, David Howells wrote: > (a) It reduces the size of the kernel-only headers and obviates the need > for __KERNEL__ conditionals in the remnant kernel-only headers. > > (b) In what we have today, there are complex interdependencies between > headers

[PATCH 00/13] UAPI header file split

2012-07-20 Thread David Howells
Here's the second installment of patches from step 1 of my plan below to clean up the kernel header files and sort out the inclusion recursion problems. Note that these patches will need regenerating if the header files they alter change before they're applied. However, the disintegration is scr