You are right exactly.
Please find attached revised patch.

Thanks,
Joonwoo

On Wed, Sep 29, 2010 at 4:04 PM, Eddie Kohler <[email protected]> wrote:
> I believe 4e73b8b9157a6c62c2d5e855e0c13ffbf353668e means we don't need the
> rcupdate.h portion of this patch.  Confirm & resubmit?
>
> E
>
>
> On 09/25/2010 10:29 PM, Joonwoo Park wrote:
>>
>> eliminate systax error below:
>> use of enum ‘kobj_ns_type’ without previous declaration
>>
>> Signed-off-by: Joonwoo Park<[email protected]>
>> ---
>>  linuxmodule/fixincludes.pl |    6 ++++++
>>  1 files changed, 6 insertions(+), 0 deletions(-)
>>
>> diff --git a/linuxmodule/fixincludes.pl b/linuxmodule/fixincludes.pl
>> index a4c9aa7..5553d9a 100755
>> --- a/linuxmodule/fixincludes.pl
>> +++ b/linuxmodule/fixincludes.pl
>> @@ -205,6 +205,12 @@ sub one_includeroot ($$) {
>>                # array dimension.
>>
>>  s&sizeof\s*\(\s*struct\s*\{\s*\w+\s*:\s*-\s*\!\s*\!\s*\(e\)\s*;\s*\}\s*\)&(sizeof(int[-!!(e)])*(size_t)0)&g;
>>            }
>> +           if ($d eq "rcupdate.h") {
>> +               s{(^static inline notrace)(.+)}{"$1$2;\nstatic
>> inline$2"}emg;
>> +           }
>> +           if ($d eq "kobject.h") {
>> +               s{(^\#include \<linux\/sysfs.h\>(.*\n)*)(^enum
>> kobj_ns_type\s\{\n([^\}].*\n)*\}\;)((.*\n)*)}{"$3\n$1\/*\n$3\n*\/$5"}emg;
>> +           }
>>
>>            # unquote.
>>            $_ = sunprotect($_);
>
From 1b4e567aff6b19a806966b2ed3ceac4e92ad92e0 Mon Sep 17 00:00:00 2001
From: Joonwoo Park <[email protected]>
Date: Sun, 3 Oct 2010 18:16:17 -0700
Subject: [PATCH] =?UTF-8?q?eliminate=20systax=20error=20below:
=20use=20of=20enum=20=E2=80=98kobj=5Fns=5Ftype=E2=80=99=20without=20previous=20declaration?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Joonwoo Park <[email protected]>
---
 linuxmodule/fixincludes.pl |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/linuxmodule/fixincludes.pl b/linuxmodule/fixincludes.pl
index 4f48e05..3af79b5 100755
--- a/linuxmodule/fixincludes.pl
+++ b/linuxmodule/fixincludes.pl
@@ -208,6 +208,9 @@ sub one_includeroot ($$) {
 	    if ($d eq "sched.h") {
 		s<^(extern char ___assert_task_state)((?:.*?\n)*?.*?\;.*)$><\#ifndef __cplusplus\n$1$2\n\#endif>mg;
 	    }
+	    if ($d eq "kobject.h") {
+		s{(^\#include \<linux\/sysfs.h\>(.*\n)*)(^enum kobj_ns_type\s\{\n([^\}].*\n)*\}\;)((.*\n)*)}{"$3\n$1\/*\n$3\n*\/$5"}emg;
+	    }
 
 	    # unquote.
 	    $_ = sunprotect($_);
-- 
1.7.0.4

_______________________________________________
click mailing list
[email protected]
https://amsterdam.lcs.mit.edu/mailman/listinfo/click

Reply via email to