Vipul,
vipul kumar <vipulk0511(a)gmail.com> writes:
Please see https://people.kernel.org/tglx/notes-about-netiquette
and search for Top-posting.
> Please find attached logs with mainline kernel version 5.4.15 with
> patch.
Which patch? I'm not seing any of the debug prints from my patch in that
dmesg.
I assume it's your patch, right?
> [ 5.736689] tsc: Refined TSC clocksource calibration: 1833.333 MHz
Otherwise this would not show up.
Try again please with your patch removed an my debug patch applied.
Thanks,
tglx
Hi,
On 28-01-2020 12:47, vipul kumar wrote:
> Hi Thomas,
>
> Please find attached logs with mainline kernel version 5.4.15 with patch.
So the suggested change seems to not work, that is strange.
Can you double check you are running the correct kernel and
add the following change and gather debug output ? :
--- a/arch/x86/kernel/tsc_msr.c
+++ b/arch/x86/kernel/tsc_msr.c
@@ -102,6 +103,8 @@ unsigned long cpu_khz_from_msr(void)
/* Get FSB FREQ ID */
+ pr_err("tsc msr id match %ld lo 0x%02x\n", id - tsc_msr_cpu_ids, lo);
+
/* Map CPU reference clock freq ID(0-7) to CPU reference clock freq(KHz) */
freq = freq_desc->freqs[lo & 0x7];
Regards,
Hans
> On Fri, Jan 24, 2020 at 9:24 PM Hans de Goede <hdegoede(a)redhat.com <mailto:hdegoede@redhat.com>> wrote:
>
> Hi,
>
> On 1/24/20 12:55 PM, Thomas Gleixner wrote:
> > Hans,
> >
> > Hans de Goede <hdegoede(a)redhat.com <mailto:hdegoede@redhat.com>> writes:
> >> On 1/24/20 9:35 AM, Thomas Gleixner wrote:
> >>> Where does that number come from? Just math?
> >>
> >> Yes just math, but perhaps the Intel folks can see if they can find some
> >> datasheet to back this up ?
> >
> > Can you observe the issue on one of the machines in your zoo as well?
>
> I haven't tried yet. Looking at the thread sofar the problem was noticed on
> a system with a Celeron N2930, I don't have access to one of those, I
> do have access to a system with a closely related N2840 I will give that
> a try as well as see if I can reproduce this on one of the tablet
> oriented Z3735x SoCs.
>
> I'll report back when I have had a chance to test this.
>
> Regards,
>
> Hans
>
stable-rc 4.9 build failed due to these build error,
drivers/md/bitmap.c:1702:13: error: conflicting types for 'bitmap_free'
static void bitmap_free(struct bitmap *bitmap)
^~~~~~~~~~~
include/linux/bitmap.h:94:13: note: previous declaration of
'bitmap_free' was here
extern void bitmap_free(const unsigned long *bitmap);
^~~~~~~~~~~
scripts/Makefile.build:304: recipe for target 'drivers/md/bitmap.o' failed
suspecting this patch causing this build failure on stable-rc 4.9
bitmap: Add bitmap_alloc(), bitmap_zalloc() and bitmap_free()
commit c42b65e363ce97a828f81b59033c3558f8fa7f70 upstream.
A lot of code become ugly because of open coding allocations for bitmaps.
Introduce three helpers to allow users be more clear of intention
and keep their code neat.
Note, due to multiple circular dependencies we may not provide
the helpers as inliners. For now we keep them exported and, perhaps,
at some point in the future we will sort out header inclusion and
inheritance.
Signed-off-by: Andy Shevchenko <andriy.shevchenko(a)linux.intel.com>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov(a)gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh(a)linuxfoundation.org>
--
Linaro LKFT
https://lkft.linaro.org
On Sun, 1 Jul 2018 at 01:49, Andy Shevchenko
<andriy.shevchenko(a)linux.intel.com> wrote:
>
> A lot of code become ugly because of open coding allocations for bitmaps.
>
> Introduce three helpers to allow users be more clear of intention
> and keep their code neat.
>
> Note, due to multiple circular dependencies we may not provide
> the helpers as inliners. For now we keep them exported and, perhaps,
> at some point in the future we will sort out header inclusion and
> inheritance.
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko(a)linux.intel.com>
> ---
> include/linux/bitmap.h | 8 ++++++++
> lib/bitmap.c | 19 +++++++++++++++++++
> 2 files changed, 27 insertions(+)
>
> diff --git a/include/linux/bitmap.h b/include/linux/bitmap.h
> index 1ee46f492267..acf5e8df3504 100644
> --- a/include/linux/bitmap.h
> +++ b/include/linux/bitmap.h
> @@ -104,6 +104,14 @@
> * contain all bit positions from 0 to 'bits' - 1.
> */
>
> +/*
> + * Allocation and deallocation of bitmap.
> + * Provided in lib/bitmap.c to avoid circular dependency.
> + */
> +extern unsigned long *bitmap_alloc(unsigned int nbits, gfp_t flags);
> +extern unsigned long *bitmap_zalloc(unsigned int nbits, gfp_t flags);
> +extern void bitmap_free(const unsigned long *bitmap);
> +
> /*
> * lib/bitmap.c provides these functions:
> */
> diff --git a/lib/bitmap.c b/lib/bitmap.c
> index 33e95cd359a2..09acf2fd6a35 100644
> --- a/lib/bitmap.c
> +++ b/lib/bitmap.c
> @@ -13,6 +13,7 @@
> #include <linux/bitops.h>
> #include <linux/bug.h>
> #include <linux/kernel.h>
> +#include <linux/slab.h>
> #include <linux/string.h>
> #include <linux/uaccess.h>
>
> @@ -1125,6 +1126,24 @@ void bitmap_copy_le(unsigned long *dst, const unsigned long *src, unsigned int n
> EXPORT_SYMBOL(bitmap_copy_le);
> #endif
>
> +unsigned long *bitmap_alloc(unsigned int nbits, gfp_t flags)
> +{
> + return kmalloc_array(BITS_TO_LONGS(nbits), sizeof(unsigned long), flags);
> +}
> +EXPORT_SYMBOL(bitmap_alloc);
> +
> +unsigned long *bitmap_zalloc(unsigned int nbits, gfp_t flags)
> +{
> + return bitmap_alloc(nbits, flags | __GFP_ZERO);
> +}
> +EXPORT_SYMBOL(bitmap_zalloc);
> +
> +void bitmap_free(const unsigned long *bitmap)
> +{
> + kfree(bitmap);
> +}
> +EXPORT_SYMBOL(bitmap_free);
> +
> #if BITS_PER_LONG == 64
> /**
> * bitmap_from_arr32 - copy the contents of u32 array of bits to bitmap
stable-rc 4.14 build failed due to these build error,
lib/bitmap.c: In function 'bitmap_from_u32array':
lib/bitmap.c:1133:1: warning: ISO C90 forbids mixed declarations and
code [-Wdeclaration-after-statement]
unsigned long *bitmap_alloc(unsigned int nbits, gfp_t flags)
^~~~~~~~
In file included from
/srv/oe/build/tmp-lkft-glibc/work-shared/intel-corei7-64/kernel-source/lib/bitmap.c:8:0:
lib/bitmap.c:1138:15: error: non-static declaration of 'bitmap_alloc'
follows static declaration
EXPORT_SYMBOL(bitmap_alloc);
^
include/linux/export.h:65:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
lib/bitmap.c:1138:1: note: in expansion of macro 'EXPORT_SYMBOL'
EXPORT_SYMBOL(bitmap_alloc);
^~~~~~~~~~~~~
lib/bitmap.c:1133:16: note: previous definition of 'bitmap_alloc' was here
unsigned long *bitmap_alloc(unsigned int nbits, gfp_t flags)
^~~~~~~~~~~~
In file included from
/srv/oe/build/tmp-lkft-glibc/work-shared/intel-corei7-64/kernel-source/lib/bitmap.c:8:0:
lib/bitmap.c:1144:15: error: non-static declaration of 'bitmap_zalloc'
follows static declaration
EXPORT_SYMBOL(bitmap_zalloc);
^
include/linux/export.h:65:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
lib/bitmap.c:1144:1: note: in expansion of macro 'EXPORT_SYMBOL'
EXPORT_SYMBOL(bitmap_zalloc);
^~~~~~~~~~~~~
lib/bitmap.c:1140:16: note: previous definition of 'bitmap_zalloc' was here
unsigned long *bitmap_zalloc(unsigned int nbits, gfp_t flags)
^~~~~~~~~~~~~
In file included from
/srv/oe/build/tmp-lkft-glibc/work-shared/intel-corei7-64/kernel-source/lib/bitmap.c:8:0:
lib/bitmap.c:1150:15: error: non-static declaration of 'bitmap_free'
follows static declaration
EXPORT_SYMBOL(bitmap_free);
^
include/linux/export.h:65:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
lib/bitmap.c:1150:1: note: in expansion of macro 'EXPORT_SYMBOL'
EXPORT_SYMBOL(bitmap_free);
^~~~~~~~~~~~~
lib/bitmap.c:1146:6: note: previous definition of 'bitmap_free' was here
void bitmap_free(const unsigned long *bitmap)
^~~~~~~~~~~
CC drivers/char/random.o
scripts/Makefile.build:326: recipe for target 'lib/bitmap.o' failed
make[3]: *** [lib/bitmap.o] Error 1
Makefile:1052: recipe for target 'lib' failed
make[2]: *** [lib] Error 2
stable-rc 4.14 build failed due to these build error,
lib/bitmap.c: In function 'bitmap_from_u32array':
lib/bitmap.c:1133:1: warning: ISO C90 forbids mixed declarations and
code [-Wdeclaration-after-statement]
unsigned long *bitmap_alloc(unsigned int nbits, gfp_t flags)
^~~~~~~~
In file included from
/srv/oe/build/tmp-lkft-glibc/work-shared/intel-corei7-64/kernel-source/lib/bitmap.c:8:0:
lib/bitmap.c:1138:15: error: non-static declaration of 'bitmap_alloc'
follows static declaration
EXPORT_SYMBOL(bitmap_alloc);
^
include/linux/export.h:65:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
lib/bitmap.c:1138:1: note: in expansion of macro 'EXPORT_SYMBOL'
EXPORT_SYMBOL(bitmap_alloc);
^~~~~~~~~~~~~
lib/bitmap.c:1133:16: note: previous definition of 'bitmap_alloc' was here
unsigned long *bitmap_alloc(unsigned int nbits, gfp_t flags)
^~~~~~~~~~~~
In file included from
/srv/oe/build/tmp-lkft-glibc/work-shared/intel-corei7-64/kernel-source/lib/bitmap.c:8:0:
lib/bitmap.c:1144:15: error: non-static declaration of 'bitmap_zalloc'
follows static declaration
EXPORT_SYMBOL(bitmap_zalloc);
^
include/linux/export.h:65:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
lib/bitmap.c:1144:1: note: in expansion of macro 'EXPORT_SYMBOL'
EXPORT_SYMBOL(bitmap_zalloc);
^~~~~~~~~~~~~
lib/bitmap.c:1140:16: note: previous definition of 'bitmap_zalloc' was here
unsigned long *bitmap_zalloc(unsigned int nbits, gfp_t flags)
^~~~~~~~~~~~~
In file included from
/srv/oe/build/tmp-lkft-glibc/work-shared/intel-corei7-64/kernel-source/lib/bitmap.c:8:0:
lib/bitmap.c:1150:15: error: non-static declaration of 'bitmap_free'
follows static declaration
EXPORT_SYMBOL(bitmap_free);
^
include/linux/export.h:65:21: note: in definition of macro '___EXPORT_SYMBOL'
extern typeof(sym) sym; \
^~~
lib/bitmap.c:1150:1: note: in expansion of macro 'EXPORT_SYMBOL'
EXPORT_SYMBOL(bitmap_free);
^~~~~~~~~~~~~
lib/bitmap.c:1146:6: note: previous definition of 'bitmap_free' was here
void bitmap_free(const unsigned long *bitmap)
^~~~~~~~~~~
CC drivers/char/random.o
scripts/Makefile.build:326: recipe for target 'lib/bitmap.o' failed
make[3]: *** [lib/bitmap.o] Error 1
Makefile:1052: recipe for target 'lib' failed
make[2]: *** [lib] Error 2
--
Linaro LKFT
https://lkft.linaro.org