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
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.
>>
>>
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
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
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
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
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
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/
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
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
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
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
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
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
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.
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
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
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
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
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
20 matches
Mail list logo