linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] memblock tests: compiling error
@ 2022-07-26  3:05 Liu Xinpeng
  2022-07-27  1:01 ` Souptick Joarder
  2022-07-27  1:53 ` mawupeng
  0 siblings, 2 replies; 8+ messages in thread
From: Liu Xinpeng @ 2022-07-26  3:05 UTC (permalink / raw)
  To: rppt, linux-mm, david; +Cc: linux-kernel, Liu Xinpeng

memblock.o: In function `memblock_find_in_range.constprop.9':
memblock.c:(.text+0x4651): undefined reference to `pr_warn_ratelimited'
memblock.o: In function `memblock_mark_mirror':
memblock.c:(.text+0x7171): undefined reference to `mirrored_kernelcore'

Fixs: 902c2d91582 ("memblock: Disable mirror feature if kernelcore is not
specified")
Fixs: 14d9a675fd0 ("mm: Ratelimited mirrored memory related warning")

Signed-off-by: Liu Xinpeng <liuxp11@chinatelecom.cn>
---
 tools/testing/memblock/internal.h | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/tools/testing/memblock/internal.h b/tools/testing/memblock/internal.h
index c2a492c..cf67e6e 100644
--- a/tools/testing/memblock/internal.h
+++ b/tools/testing/memblock/internal.h
@@ -9,6 +9,9 @@
 static int memblock_debug = 1;
 #endif
 
+#define pr_warn_ratelimited(fmt, ...)    printf(fmt, ##__VA_ARGS__);
+
+bool mirrored_kernelcore = false;
 struct page {};
 
 void memblock_free_pages(struct page *page, unsigned long pfn,
-- 
1.8.3.1



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

* Re: [PATCH] memblock tests: compiling error
  2022-07-26  3:05 [PATCH] memblock tests: compiling error Liu Xinpeng
@ 2022-07-27  1:01 ` Souptick Joarder
  2022-07-27  1:03   ` Randy Dunlap
  2022-07-27  1:53   ` mawupeng
  2022-07-27  1:53 ` mawupeng
  1 sibling, 2 replies; 8+ messages in thread
From: Souptick Joarder @ 2022-07-27  1:01 UTC (permalink / raw)
  To: Liu Xinpeng; +Cc: Mike Rapoport, Linux-MM, David Hildenbrand, linux-kernel

On Tue, Jul 26, 2022 at 8:35 AM Liu Xinpeng <liuxp11@chinatelecom.cn> wrote:
>
> memblock.o: In function `memblock_find_in_range.constprop.9':
> memblock.c:(.text+0x4651): undefined reference to `pr_warn_ratelimited'
> memblock.o: In function `memblock_mark_mirror':
> memblock.c:(.text+0x7171): undefined reference to `mirrored_kernelcore'
>

what is the steps to hit this compile error ?

> Fixs: 902c2d91582 ("memblock: Disable mirror feature if kernelcore is not
> specified")
> Fixs: 14d9a675fd0 ("mm: Ratelimited mirrored memory related warning")
>
> Signed-off-by: Liu Xinpeng <liuxp11@chinatelecom.cn>
> ---
>  tools/testing/memblock/internal.h | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/tools/testing/memblock/internal.h b/tools/testing/memblock/internal.h
> index c2a492c..cf67e6e 100644
> --- a/tools/testing/memblock/internal.h
> +++ b/tools/testing/memblock/internal.h
> @@ -9,6 +9,9 @@
>  static int memblock_debug = 1;
>  #endif
>
> +#define pr_warn_ratelimited(fmt, ...)    printf(fmt, ##__VA_ARGS__);
> +
> +bool mirrored_kernelcore = false;
>  struct page {};
>
>  void memblock_free_pages(struct page *page, unsigned long pfn,
> --
> 1.8.3.1
>
>


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

* Re: [PATCH] memblock tests: compiling error
  2022-07-27  1:01 ` Souptick Joarder
@ 2022-07-27  1:03   ` Randy Dunlap
  2022-07-27  1:53   ` mawupeng
  1 sibling, 0 replies; 8+ messages in thread
From: Randy Dunlap @ 2022-07-27  1:03 UTC (permalink / raw)
  To: Souptick Joarder, Liu Xinpeng
  Cc: Mike Rapoport, Linux-MM, David Hildenbrand, linux-kernel



On 7/26/22 18:01, Souptick Joarder wrote:
> On Tue, Jul 26, 2022 at 8:35 AM Liu Xinpeng <liuxp11@chinatelecom.cn> wrote:
>>
>> memblock.o: In function `memblock_find_in_range.constprop.9':
>> memblock.c:(.text+0x4651): undefined reference to `pr_warn_ratelimited'
>> memblock.o: In function `memblock_mark_mirror':
>> memblock.c:(.text+0x7171): undefined reference to `mirrored_kernelcore'
>>
> 
> what is the steps to hit this compile error ?

or what kernel .config file?

> 
>> Fixs: 902c2d91582 ("memblock: Disable mirror feature if kernelcore is not
>> specified")
>> Fixs: 14d9a675fd0 ("mm: Ratelimited mirrored memory related warning")
>>
>> Signed-off-by: Liu Xinpeng <liuxp11@chinatelecom.cn>
>> ---
>>  tools/testing/memblock/internal.h | 3 +++
>>  1 file changed, 3 insertions(+)
>>
>> diff --git a/tools/testing/memblock/internal.h b/tools/testing/memblock/internal.h
>> index c2a492c..cf67e6e 100644
>> --- a/tools/testing/memblock/internal.h
>> +++ b/tools/testing/memblock/internal.h
>> @@ -9,6 +9,9 @@
>>  static int memblock_debug = 1;
>>  #endif
>>
>> +#define pr_warn_ratelimited(fmt, ...)    printf(fmt, ##__VA_ARGS__);
>> +
>> +bool mirrored_kernelcore = false;
>>  struct page {};
>>
>>  void memblock_free_pages(struct page *page, unsigned long pfn,
>> --
>> 1.8.3.1
>>
>>

-- 
~Randy


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

* Re: [PATCH] memblock tests: compiling error
  2022-07-27  1:01 ` Souptick Joarder
  2022-07-27  1:03   ` Randy Dunlap
@ 2022-07-27  1:53   ` mawupeng
  1 sibling, 0 replies; 8+ messages in thread
From: mawupeng @ 2022-07-27  1:53 UTC (permalink / raw)
  To: jrdr.linux, liuxp11; +Cc: mawupeng1, rppt, linux-mm, david, linux-kernel



On 2022/7/27 9:01, Souptick Joarder wrote:
> On Tue, Jul 26, 2022 at 8:35 AM Liu Xinpeng <liuxp11@chinatelecom.cn> wrote:
>>
>> memblock.o: In function `memblock_find_in_range.constprop.9':
>> memblock.c:(.text+0x4651): undefined reference to `pr_warn_ratelimited'
>> memblock.o: In function `memblock_mark_mirror':
>> memblock.c:(.text+0x7171): undefined reference to `mirrored_kernelcore'
>>
> 
> what is the steps to hit this compile error ?

Do make -C tools/testing/memblock can hit this compile error.

> 
>> Fixs: 902c2d91582 ("memblock: Disable mirror feature if kernelcore is not
>> specified")

>> Fixs: 14d9a675fd0 ("mm: Ratelimited mirrored memory related warning")
>>
>> Signed-off-by: Liu Xinpeng <liuxp11@chinatelecom.cn>
>> ---
>>  tools/testing/memblock/internal.h | 3 +++
>>  1 file changed, 3 insertions(+)
>>
>> diff --git a/tools/testing/memblock/internal.h b/tools/testing/memblock/internal.h
>> index c2a492c..cf67e6e 100644
>> --- a/tools/testing/memblock/internal.h
>> +++ b/tools/testing/memblock/internal.h
>> @@ -9,6 +9,9 @@
>>  static int memblock_debug = 1;
>>  #endif
>>
>> +#define pr_warn_ratelimited(fmt, ...)    printf(fmt, ##__VA_ARGS__);
>> +
>> +bool mirrored_kernelcore = false;
>>  struct page {};
>>
>>  void memblock_free_pages(struct page *page, unsigned long pfn,
>> --
>> 1.8.3.1
>>
>>


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

* Re: [PATCH] memblock tests: compiling error
  2022-07-26  3:05 [PATCH] memblock tests: compiling error Liu Xinpeng
  2022-07-27  1:01 ` Souptick Joarder
@ 2022-07-27  1:53 ` mawupeng
  2022-07-27  2:06   ` liuxp11
  1 sibling, 1 reply; 8+ messages in thread
From: mawupeng @ 2022-07-27  1:53 UTC (permalink / raw)
  To: liuxp11, rppt, linux-mm, david; +Cc: mawupeng1, linux-kernel



On 2022/7/26 11:05, Liu Xinpeng wrote:
> memblock.o: In function `memblock_find_in_range.constprop.9':
> memblock.c:(.text+0x4651): undefined reference to `pr_warn_ratelimited'
> memblock.o: In function `memblock_mark_mirror':
> memblock.c:(.text+0x7171): undefined reference to `mirrored_kernelcore'
> 
> Fixs: 902c2d91582 ("memblock: Disable mirror feature if kernelcore is not
> specified")

Fixs should not have word warp.

Tested-by: Ma Wupeng <mawupeng1@huawei.com>

> Fixs: 14d9a675fd0 ("mm: Ratelimited mirrored memory related warning")
> 
> Signed-off-by: Liu Xinpeng <liuxp11@chinatelecom.cn>
> ---
>  tools/testing/memblock/internal.h | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/tools/testing/memblock/internal.h b/tools/testing/memblock/internal.h
> index c2a492c..cf67e6e 100644
> --- a/tools/testing/memblock/internal.h
> +++ b/tools/testing/memblock/internal.h
> @@ -9,6 +9,9 @@
>  static int memblock_debug = 1;
>  #endif
>  
> +#define pr_warn_ratelimited(fmt, ...)    printf(fmt, ##__VA_ARGS__);
> +
> +bool mirrored_kernelcore = false;
>  struct page {};
>  
>  void memblock_free_pages(struct page *page, unsigned long pfn,


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

* Re: Re: [PATCH] memblock tests: compiling error
  2022-07-27  1:53 ` mawupeng
@ 2022-07-27  2:06   ` liuxp11
  2022-07-27  2:20     ` Kefeng Wang
  0 siblings, 1 reply; 8+ messages in thread
From: liuxp11 @ 2022-07-27  2:06 UTC (permalink / raw)
  To: mawupeng, rppt, linux-mm, david; +Cc: linux-kernel

>On 2022/7/26 11:05, Liu Xinpeng wrote:

>> memblock.o: In function `memblock_find_in_range.constprop.9':

>> memblock.c:(.text+0x4651): undefined reference to `pr_warn_ratelimited'

>> memblock.o: In function `memblock_mark_mirror':

>> memblock.c:(.text+0x7171): undefined reference to `mirrored_kernelcore'

>> Fixs: 902c2d91582 ("memblock: Disable mirror feature if kernelcore is not
>> specified")

>Fixs should not have word warp.


>Tested-by: Ma Wupeng <mawupeng1@huawei.com>

I will send patch v2, such as,

Fixs: 902c2d91582 ("memblock: Disable mirror feature if kernelcore is not
specified") and 14d9a675fd0 ("mm: Ratelimited mirrored memory related warning")

> +#define pr_warn_ratelimited(fmt, ...)    printf(fmt, ##__VA_ARGS__);

And removed the comma.


thanks your test,
Liu Xinpeng


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

* Re: [PATCH] memblock tests: compiling error
  2022-07-27  2:06   ` liuxp11
@ 2022-07-27  2:20     ` Kefeng Wang
  0 siblings, 0 replies; 8+ messages in thread
From: Kefeng Wang @ 2022-07-27  2:20 UTC (permalink / raw)
  To: liuxp11, mawupeng, rppt, linux-mm, david; +Cc: linux-kernel

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


On 2022/7/27 10:06, liuxp11@chinatelecom.cn wrote:
>> On 2022/7/26 11:05, Liu Xinpeng wrote:
>>> memblock.o: In function `memblock_find_in_range.constprop.9':
>>> memblock.c:(.text+0x4651): undefined reference to `pr_warn_ratelimited'
>>> memblock.o: In function `memblock_mark_mirror':
>>> memblock.c:(.text+0x7171): undefined reference to `mirrored_kernelcore'
>>> Fixs: 902c2d91582 ("memblock: Disable mirror feature if kernelcore is not
>>> specified")
>> Fixs should not have word warp.
>
>> Tested-by: Ma Wupeng<mawupeng1@huawei.com>
> I will send patch v2, such as,
>
> Fixs: 902c2d91582 ("memblock: Disable mirror feature if kernelcore is not
> specified") and 14d9a675fd0 ("mm: Ratelimited mirrored memory related warning")

no,  see https://kernel.org/doc/html/latest/process/submitting-patches.html

please check fixes part.

>> +#define pr_warn_ratelimited(fmt, ...)    printf(fmt, ##__VA_ARGS__);
> And removed the comma.
>
>
> thanks your test,
> Liu Xinpeng
>

[-- Attachment #2: Type: text/html, Size: 3978 bytes --]

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

* [PATCH] memblock tests: compiling error
@ 2022-07-26  3:21 Liu Xinpeng
  0 siblings, 0 replies; 8+ messages in thread
From: Liu Xinpeng @ 2022-07-26  3:21 UTC (permalink / raw)
  To: rppt, david; +Cc: linux-mm, linux-kernel, Liu Xinpeng

memblock.o: In function `memblock_find_in_range.constprop.9':
memblock.c:(.text+0x4651): undefined reference to `pr_warn_ratelimited'
memblock.o: In function `memblock_mark_mirror':
memblock.c:(.text+0x7171): undefined reference to `mirrored_kernelcore'

Fixs: 902c2d91582 ("memblock: Disable mirror feature if kernelcore is not
specified")
Fixs: 14d9a675fd0 ("mm: Ratelimited mirrored memory related warning")

Signed-off-by: Liu Xinpeng <liuxp11@chinatelecom.cn>
---
 tools/testing/memblock/internal.h | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/tools/testing/memblock/internal.h b/tools/testing/memblock/internal.h
index c2a492c..cf67e6e 100644
--- a/tools/testing/memblock/internal.h
+++ b/tools/testing/memblock/internal.h
@@ -9,6 +9,9 @@
 static int memblock_debug = 1;
 #endif
 
+#define pr_warn_ratelimited(fmt, ...)    printf(fmt, ##__VA_ARGS__)
+
+bool mirrored_kernelcore = false;
 struct page {};
 
 void memblock_free_pages(struct page *page, unsigned long pfn,
-- 
1.8.3.1



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

end of thread, other threads:[~2022-07-27  2:20 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-07-26  3:05 [PATCH] memblock tests: compiling error Liu Xinpeng
2022-07-27  1:01 ` Souptick Joarder
2022-07-27  1:03   ` Randy Dunlap
2022-07-27  1:53   ` mawupeng
2022-07-27  1:53 ` mawupeng
2022-07-27  2:06   ` liuxp11
2022-07-27  2:20     ` Kefeng Wang
2022-07-26  3:21 Liu Xinpeng

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox