Thread (4 messages) 4 messages, 3 authors, 2018-08-01

RE: [PATCH 1/1] x86, pmem, acpi: Remove excessive ACPI Large Reference Count warnings

From: Moore, Robert <hidden>
Date: 2018-08-01 21:24:26
Also in: lkml, nvdimm

-----Original Message-----
From: Russ Anderson [mailto:rja-ZPxbGqLxI0U@public.gmane.org]
Sent: Wednesday, August 1, 2018 1:38 PM
To: Moore, Robert <redacted>
Cc: Mike Travis <redacted>; Schmauss, Erik
[off-list ref]; Wysocki, Rafael J
[off-list ref]; Len Brown [off-list ref]; Russ Anderson
[off-list ref]; Dimitri Sivanich [off-list ref];
Ross Zwisler [off-list ref]; Williams, Dan J
[off-list ref]; Verma, Vishal L [off-list ref];
Jiang, Dave [off-list ref]; linux-acpi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org;
devel-E0kO6a4B6psdnm+yROfE0A@public.gmane.org; linux-nvdimm-hn68Rpc1hR1g9hUCZPvPmw@public.gmane.org; linux-
kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH 1/1] x86, pmem, acpi: Remove excessive ACPI Large
Reference Count warnings

On Wed, Aug 01, 2018 at 07:49:14PM +0000, Moore, Robert wrote:
quoted
Are there any actual memory leaks, or this a consequence of the number
of sockets?

It is a consequence of the number of sockets.
It is not an actual memory leak.

Thanks.
[Moore, Robert] 

It does bother me a bit that there are over 4096 references to a single object, however. Could you send us the ACPI dump for the machine?

This issue sounds vaguely familiar though. Perhaps Dan remembers, it was something about large reference counts and the NFIT/NVDIMM stuff. If so, this has been fixed for some time now.

Bob

quoted
quoted
-----Original Message-----
From: Mike Travis [mailto:mike.travis-ZPxbGqLxI0U@public.gmane.org]
Sent: Monday, July 30, 2018 12:56 PM
To: Moore, Robert <redacted>; Schmauss, Erik
[off-list ref]; Wysocki, Rafael J
[off-list ref]; Len Brown [off-list ref]
Cc: Russ Anderson <redacted>; Dimitri Sivanich
[off-list ref]; Ross Zwisler
[off-list ref]; Williams, Dan J
[off-list ref]; Verma, Vishal L
[off-list ref]; Jiang, Dave [off-list ref];
linux- acpi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; devel-E0kO6a4B6psdnm+yROfE0A@public.gmane.org;
linux-nvdimm-hn68Rpc1hR1g9hUCZPvPmw@public.gmane.org; linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: [PATCH 1/1] x86, pmem, acpi: Remove excessive ACPI Large
Reference Count warnings

With the Intel BIOS support for 8 processor sockets with a full
complement of NVDIMMS potentially installable, and there are empty
sockets without NVDIMMS, there is an extremely large amount of the
following warnings:

ACPI Warning: Large Reference Count (0x1001) in object
ffff99453fc71750,
quoted
quoted
	Type=0x0A

On a 4 socket system with 4 NVDIMMs there were over 6000 of these
warning messages and it has been seem on systems from 4 to 32
sockets.
quoted
quoted
Through some guidance from the BIOS developers and testing, it
appears that simply bumping up the threshold for warnings from
0x1000 to 0x2000 eliminates these messages.  Changing them to be
ACPI DEBUG messages, or even removing them are other options, but it
would then defeat the purpose of the warnings as the problem would
effectively be hidden.
quoted
quoted
Reviewed-by: Dimitri Sivanich <redacted>
Tested-by: Russ Anderson <redacted>
Signed-off-by: Mike Travis <redacted>
---
 include/acpi/acconfig.h |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
--- linux-4.12.orig/include/acpi/acconfig.h
+++ linux-4.12/include/acpi/acconfig.h
@@ -123,7 +123,7 @@

 /* Maximum object reference count (detects object deletion issues)
*/

-#define ACPI_MAX_REFERENCE_COUNT        0x1000
+#define ACPI_MAX_REFERENCE_COUNT        0x2000

 /* Default page size for use in mapping memory for operation
regions */


--
--
Russ Anderson,  SuperDome Flex Linux Kernel Group Manager HPE - Hewlett
Packard Enterprise (formerly SGI)  rja-ZPxbGqLxI0U@public.gmane.org
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help