lkml.org 
[lkml]   [2008]   [May]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: -mm merge plans for 2.6.26
Andrew Morton wrote:
> On Fri, 16 May 2008 17:03:10 -0700 Randy Dunlap <randy.dunlap@oracle.com> wrote:
>
>> On Sun, 20 Apr 2008 07:20:53 -0700 Andrew Morton wrote:
>>
>>> These can be found at
>>> ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.25/2.6.25-mm1
>>>
>>>
>>>
>>> #
>>> # documentation
>>> #
>>> documentation-build-source-files-in-documentation-sub-dir.patch
>>> documentation-build-source-files-in-documentation-sub-dir-update.patch
>>> documentation-build-source-files-in-documentation-sub-dir-disable.patch
>>>
>>> Merge
>> Hi,
>> What happened to/with these?
>
> I was going to ask you that. Last I heard,
>
> a) stuff broke and
>
> b) you had an updated version somewhere.

Oh yeah. I think that this was fixed by
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=b11caa7c7063ea92a0a58115d3fc6d038ed89510
but I'll check/verify/confirm that.


> On Thu, 10 Apr 2008 17:56:30 -0700 Randy Dunlap <randy.dunlap@oracle.com> wrote:
>
>> On Wed, 9 Apr 2008 14:54:09 -0700 Andrew Morton wrote:
>>
>>> make allmodconfig
>>> gcc -I$(/bin/pwd)/include Documentation/accounting/getdelays.c -o getdelays
>>>
>>> In file included from /usr/src/devel/include/linux/netlink.h:5,
>>> from /usr/src/devel/include/linux/genetlink.h:4,
>>> from Documentation/accounting/getdelays.c:26:
>>> /usr/src/devel/include/linux/types.h:203: error: expected specifier-qualifier-list before '__kernel_daddr_t'
>>
>> Those __kernel_* types shouldn't be used outside of the #ifdef __KERNEL__
>> block, should they?
>>
>> Patch below fixes kernel side for me. Don't have any idea what it
>> may do to userspace users of the header file.
>>
>> ---
>> include/linux/types.h | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> --- mmotm-2008-0410-0157.orig/include/linux/types.h
>> +++ mmotm-2008-0410-0157/include/linux/types.h
>> @@ -200,8 +200,8 @@ typedef u32 resource_size_t;
>> #endif /* __KERNEL__ */
>>
>> struct ustat {
>> - __kernel_daddr_t f_tfree;
>> - __kernel_ino_t f_tinode;
>> + daddr_t f_tfree;
>> + ino_t f_tinode;
>> char f_fname[6];
>> char f_fpack[6];
>> };
>
> I didn't apply that. I guess the "Don't have any idea" bit was scary.


--
~Randy


\
 
 \ /
  Last update: 2008-05-17 04:21    [from the cache]
©2003-2011 Jasper Spaans. Advertise on this site