All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] fix compile error while opening the macro DOUBLE_CHECK
@ 2015-11-17  3:04 zhangaihua1
  2015-11-17  3:26 ` kbuild test robot
  2015-11-17  4:03 ` Greg KH
  0 siblings, 2 replies; 7+ messages in thread
From: zhangaihua1 @ 2015-11-17  3:04 UTC (permalink / raw)
  To: tytso, linux-ext4, stable; +Cc: Aihua Zhang

From: Aihua Zhang <zhangaihua1@huawei.com>

the error is:
    fs/ext4/mballoc.c:475:43: error: 'struct ext4_group_info' has
no member named 'bb_bitmap'
    the definition of macro should before 'struct ext4_group_info'

Signed-off-by: Aihua Zhang <zhangaihua1@huawei.com>
---
 fs/ext4/ext4.h    |   12 ++++++++++++
 fs/ext4/mballoc.h |   12 ------------
 2 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/fs/ext4/ext4.h b/fs/ext4/ext4.h
index 750063f..e743955 100644
--- a/fs/ext4/ext4.h
+++ b/fs/ext4/ext4.h
@@ -41,6 +41,18 @@
  */
 
 /*
+ * with AGGRESSIVE_CHECK allocator runs consistency checks over
+ * structures. these checks slow things down a lot
+ * /
+#define AGGRESSIVE_CHECK__
+
+/*
+ * with DOUBLE_CHECK defined mballoc creates persistent in-core
+ * bitmaps, maintains and uses them to check for double allocations
+ * /
+#define DOUBLE_CHECK__
+
+/*
  * Define EXT4FS_DEBUG to produce debug messages
  */
 #undef EXT4FS_DEBUG
diff --git a/fs/ext4/mballoc.h b/fs/ext4/mballoc.h
index d634e18..3ef1df6 100644
--- a/fs/ext4/mballoc.h
+++ b/fs/ext4/mballoc.h
@@ -23,18 +23,6 @@
 #include "ext4.h"
 
 /*
- * with AGGRESSIVE_CHECK allocator runs consistency checks over
- * structures. these checks slow things down a lot
- */
-#define AGGRESSIVE_CHECK__
-
-/*
- * with DOUBLE_CHECK defined mballoc creates persistent in-core
- * bitmaps, maintains and uses them to check for double allocations
- */
-#define DOUBLE_CHECK__
-
-/*
  */
 #ifdef CONFIG_EXT4_DEBUG
 extern ushort ext4_mballoc_debug;
-- 
1.7.1

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH] fix compile error while opening the macro DOUBLE_CHECK
  2015-11-17  3:04 [PATCH] fix compile error while opening the macro DOUBLE_CHECK zhangaihua1
@ 2015-11-17  3:26 ` kbuild test robot
  2015-11-17  4:03 ` Greg KH
  1 sibling, 0 replies; 7+ messages in thread
From: kbuild test robot @ 2015-11-17  3:26 UTC (permalink / raw)
  To: zhangaihua1; +Cc: kbuild-all, tytso, linux-ext4, stable, Aihua Zhang

[-- Attachment #1: Type: text/plain, Size: 1736 bytes --]

Hi Aihua,

[auto build test WARNING on: v4.4-rc1]
[also build test WARNING on: next-20151117]

url:    https://github.com/0day-ci/linux/commits/zhangaihua1-huawei-com/fix-compile-error-while-opening-the-macro-DOUBLE_CHECK/20151117-110904
config: parisc-c3000_defconfig (attached as .config)
reproduce:
        wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=parisc 

All warnings (new ones prefixed by >>):

   In file included from fs/ext4/ext4_jbd2.h:20:0,
                    from fs/ext4/extents.c:43:
>> fs/ext4/ext4.h:49:1: warning: "/*" within comment [-Wcomment]
    /*
    ^
   fs/ext4/ext4.h:55:1: warning: "/*" within comment [-Wcomment]
    /*
    ^

vim +49 fs/ext4/ext4.h

    33	#include <crypto/hash.h>
    34	#include <linux/falloc.h>
    35	#ifdef __KERNEL__
    36	#include <linux/compat.h>
    37	#endif
    38	
    39	/*
    40	 * The fourth extended filesystem constants/structures
    41	 */
    42	
    43	/*
    44	 * with AGGRESSIVE_CHECK allocator runs consistency checks over
    45	 * structures. these checks slow things down a lot
    46	 * /
    47	#define AGGRESSIVE_CHECK__
    48	
  > 49	/*
    50	 * with DOUBLE_CHECK defined mballoc creates persistent in-core
    51	 * bitmaps, maintains and uses them to check for double allocations
    52	 * /
    53	#define DOUBLE_CHECK__
    54	
    55	/*
    56	 * Define EXT4FS_DEBUG to produce debug messages
    57	 */

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 13631 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] fix compile error while opening the macro DOUBLE_CHECK
  2015-11-17  3:04 [PATCH] fix compile error while opening the macro DOUBLE_CHECK zhangaihua1
  2015-11-17  3:26 ` kbuild test robot
@ 2015-11-17  4:03 ` Greg KH
  2015-11-17  6:31   ` Zhang AiHua
  1 sibling, 1 reply; 7+ messages in thread
From: Greg KH @ 2015-11-17  4:03 UTC (permalink / raw)
  To: zhangaihua1; +Cc: tytso, linux-ext4, stable

On Tue, Nov 17, 2015 at 11:04:03AM +0800, zhangaihua1@huawei.com wrote:
> From: Aihua Zhang <zhangaihua1@huawei.com>
> 
> the error is:
>     fs/ext4/mballoc.c:475:43: error: 'struct ext4_group_info' has
> no member named 'bb_bitmap'
>     the definition of macro should before 'struct ext4_group_info'
> 
> Signed-off-by: Aihua Zhang <zhangaihua1@huawei.com>
> ---
>  fs/ext4/ext4.h    |   12 ++++++++++++
>  fs/ext4/mballoc.h |   12 ------------
>  2 files changed, 12 insertions(+), 12 deletions(-)

<formletter>

This is not the correct way to submit patches for inclusion in the
stable kernel tree.  Please read Documentation/stable_kernel_rules.txt
for how to do this properly.

</formletter>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] fix compile error while opening the macro DOUBLE_CHECK
  2015-11-17  4:03 ` Greg KH
@ 2015-11-17  6:31   ` Zhang AiHua
  0 siblings, 0 replies; 7+ messages in thread
From: Zhang AiHua @ 2015-11-17  6:31 UTC (permalink / raw)
  To: Greg KH; +Cc: tytso, linux-ext4, stable


Hi,

     I want to submit the patch to mainline, I feel so ashamed for disturbing you,
you can ignore it....  I have written wrong email.
     Later, when I need, I will read Documentation/stable_kernel_rules.txt first,
thank you anyway.

Regards, AiHua.


在 2015/11/17 12:03, Greg KH 写道:
> On Tue, Nov 17, 2015 at 11:04:03AM +0800, zhangaihua1@huawei.com wrote:
>> From: Aihua Zhang <zhangaihua1@huawei.com>
>>
>> the error is:
>>      fs/ext4/mballoc.c:475:43: error: 'struct ext4_group_info' has
>> no member named 'bb_bitmap'
>>      the definition of macro should before 'struct ext4_group_info'
>>
>> Signed-off-by: Aihua Zhang <zhangaihua1@huawei.com>
>> ---
>>   fs/ext4/ext4.h    |   12 ++++++++++++
>>   fs/ext4/mballoc.h |   12 ------------
>>   2 files changed, 12 insertions(+), 12 deletions(-)
>
> <formletter>
>
> This is not the correct way to submit patches for inclusion in the
> stable kernel tree.  Please read Documentation/stable_kernel_rules.txt
> for how to do this properly.
>
> </formletter>
>
> .
>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] fix compile error while opening the macro DOUBLE_CHECK
  2015-12-01  8:58 zhangaihua1
@ 2016-03-13 21:19 ` Theodore Ts'o
  0 siblings, 0 replies; 7+ messages in thread
From: Theodore Ts'o @ 2016-03-13 21:19 UTC (permalink / raw)
  To: zhangaihua1; +Cc: linux-ext4

On Tue, Dec 01, 2015 at 04:58:29PM +0800, zhangaihua1@huawei.com wrote:
> From: Aihua Zhang <zhangaihua1@huawei.com>
> 
> the error is:
>     fs/ext4/mballoc.c:475:43: error: 'struct ext4_group_info' has
> no member named 'bb_bitmap'.
>     so, the definition of macro DOUBLE_CHECK should before
> 'struct ext4_group_info', I fixed it, and I moved the macro
> AGGRESSIVE_CHECK together, because I think they shoule be together.
> 
> Signed-off-by: Aihua Zhang <zhangaihua1@huawei.com>

Applied, thanks.  Apologies for the delay.

							- Ted

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH] fix compile error while opening the macro DOUBLE_CHECK
@ 2015-12-01  8:58 zhangaihua1
  2016-03-13 21:19 ` Theodore Ts'o
  0 siblings, 1 reply; 7+ messages in thread
From: zhangaihua1 @ 2015-12-01  8:58 UTC (permalink / raw)
  To: tytso, linux-ext4; +Cc: Aihua Zhang

From: Aihua Zhang <zhangaihua1@huawei.com>

the error is:
    fs/ext4/mballoc.c:475:43: error: 'struct ext4_group_info' has
no member named 'bb_bitmap'.
    so, the definition of macro DOUBLE_CHECK should before
'struct ext4_group_info', I fixed it, and I moved the macro
AGGRESSIVE_CHECK together, because I think they shoule be together.

Signed-off-by: Aihua Zhang <zhangaihua1@huawei.com>
---
 fs/ext4/ext4.h    |   12 ++++++++++++
 fs/ext4/mballoc.h |   12 ------------
 2 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/fs/ext4/ext4.h b/fs/ext4/ext4.h
index 750063f..04a3732 100644
--- a/fs/ext4/ext4.h
+++ b/fs/ext4/ext4.h
@@ -41,6 +41,18 @@
  */
 
 /*
+ * with AGGRESSIVE_CHECK allocator runs consistency checks over
+ * structures. these checks slow things down a lot
+ */
+#define AGGRESSIVE_CHECK__
+
+/*
+ * with DOUBLE_CHECK defined mballoc creates persistent in-core
+ * bitmaps, maintains and uses them to check for double allocations
+ */
+#define DOUBLE_CHECK__
+
+/*
  * Define EXT4FS_DEBUG to produce debug messages
  */
 #undef EXT4FS_DEBUG
diff --git a/fs/ext4/mballoc.h b/fs/ext4/mballoc.h
index d634e18..3ef1df6 100644
--- a/fs/ext4/mballoc.h
+++ b/fs/ext4/mballoc.h
@@ -23,18 +23,6 @@
 #include "ext4.h"
 
 /*
- * with AGGRESSIVE_CHECK allocator runs consistency checks over
- * structures. these checks slow things down a lot
- */
-#define AGGRESSIVE_CHECK__
-
-/*
- * with DOUBLE_CHECK defined mballoc creates persistent in-core
- * bitmaps, maintains and uses them to check for double allocations
- */
-#define DOUBLE_CHECK__

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH] fix compile error while opening the macro DOUBLE_CHECK
@ 2015-11-17  3:52 zhangaihua1
  0 siblings, 0 replies; 7+ messages in thread
From: zhangaihua1 @ 2015-11-17  3:52 UTC (permalink / raw)
  To: tytso, linux-ext4, stable; +Cc: Aihua Zhang

From: Aihua Zhang <zhangaihua1@huawei.com>

the error is:
    fs/ext4/mballoc.c:475:43: error: 'struct ext4_group_info' has
no member named 'bb_bitmap'
    the definition of macro should before 'struct ext4_group_info'

Signed-off-by: Aihua Zhang <zhangaihua1@huawei.com>
---
 fs/ext4/ext4.h    |   12 ++++++++++++
 fs/ext4/mballoc.h |   12 ------------
 2 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/fs/ext4/ext4.h b/fs/ext4/ext4.h
index 750063f..04a3732 100644
--- a/fs/ext4/ext4.h
+++ b/fs/ext4/ext4.h
@@ -41,6 +41,18 @@
  */
 
 /*
+ * with AGGRESSIVE_CHECK allocator runs consistency checks over
+ * structures. these checks slow things down a lot
+ */
+#define AGGRESSIVE_CHECK__
+
+/*
+ * with DOUBLE_CHECK defined mballoc creates persistent in-core
+ * bitmaps, maintains and uses them to check for double allocations
+ */
+#define DOUBLE_CHECK__
+
+/*
  * Define EXT4FS_DEBUG to produce debug messages
  */
 #undef EXT4FS_DEBUG
diff --git a/fs/ext4/mballoc.h b/fs/ext4/mballoc.h
index d634e18..3ef1df6 100644
--- a/fs/ext4/mballoc.h
+++ b/fs/ext4/mballoc.h
@@ -23,18 +23,6 @@
 #include "ext4.h"
 
 /*
- * with AGGRESSIVE_CHECK allocator runs consistency checks over
- * structures. these checks slow things down a lot
- */
-#define AGGRESSIVE_CHECK__
-
-/*
- * with DOUBLE_CHECK defined mballoc creates persistent in-core
- * bitmaps, maintains and uses them to check for double allocations
- */
-#define DOUBLE_CHECK__
-
-/*
  */
 #ifdef CONFIG_EXT4_DEBUG
 extern ushort ext4_mballoc_debug;
-- 
1.7.1

^ permalink raw reply related	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2016-03-13 21:19 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-11-17  3:04 [PATCH] fix compile error while opening the macro DOUBLE_CHECK zhangaihua1
2015-11-17  3:26 ` kbuild test robot
2015-11-17  4:03 ` Greg KH
2015-11-17  6:31   ` Zhang AiHua
2015-11-17  3:52 zhangaihua1
2015-12-01  8:58 zhangaihua1
2016-03-13 21:19 ` Theodore Ts'o

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.