Jiang Liu | 1 Sep 09:12 2015
Picon

Possible deadlock related to CPU hotplug and kernfs

Hi Rafael and Tejun,
	When running CPU hotplug tests, it triggers an lockdep warning
as follow. The two possible deadlock paths are:
1) echo x > /sys/devices/system/cpu/cpux/online
   ->kernfs_fop_write()
     ->kernfs_get_active()
1.a)   ->rwsem_acquire_read(&kn->dep_map, 0, 1, _RET_IP_);
         ->cpu_up()
1.b)       ->cpu_hotplug_begin()[lock_map_acquire(&cpu_hotplug.dep_map)]
2) hardware triggers hotplug evetns
   ->acpi_device_hotplug()
     ->acpi_processor_remove()
2.a)   ->cpu_hotplug_begin()[lock_map_acquire(&cpu_hotplug.dep_map)]
         ->unregister_cpu()
           ->device_del()
             ->kernfs_remove_by_name_ns()
               ->__kernfs_remove()
                 ->kernfs_drain()
2.b)               ->rwsem_acquire(&kn->dep_map, 0, 0, _RET_IP_)

So there is a possible deadlock scenario among 1.a, 1.b, 2.a and 2.b.
I'm not familiar with kernfs, so could you please help to comment:
1) whether is a real deadlock issue?
2) any recommended way to get it fixed?
Thanks!
Gerry

Full lockdep warnings:
[  310.309391] [ INFO: possible circular locking dependency detected ]
[  310.316462] 4.2.0-rc8+ #7 Not tainted
(Continue reading)

Unknown | 31 Aug 21:19 2015

Re: Do you need a loan?

We offer private, commercial and personal loans with very low annualinterest rates as low as 2% in one year
to 50 years repayment period anywhere in the world. We offer loans ranging from $5000 to $100 million. Our
loans are well insured for maximum security is our priority. Are you losing sleep at night worrying how to
get a legitimate loan lender? You bite your nails that fast? Instead of beating you, contact JM Financial
Services Ltd now, specialists who help stop loans bad credit history to find a solution that victory is our
mission. Applicants must fill out a loan application form below:

FORM credit application

loan applications
Your full name *
Your e-mail *
Your phone *
Your address *
Your city *
State / Province *
Country *
Fax*
Date of birth *
Do you have an account? *
Have you applied before? *
The loan amount is needed *
The Loan Duration*
The life expectancy *
The purpose of the loan *
Send me a scanned copy of your passport: *

Creditor: Mr. Prakash Lass Dickson.
Company: JM Financial Services Ltd.
Copyright JM Financial Ltd. All rights reserved.
(Continue reading)

Trotter family | 30 Aug 14:03 2015
Picon

(unknown)


--

-- 
The Trotter family are donating the sum of 1,500,000.00GBP to you From
the 108 Million Pounds they won on 18 March 2014 . To confirm this kindly
click on
http://www.bbc.com/news/uk-england-london-26627075
Please provide the following information
*Full Name:*Telephone Number:*Age:*Country:
reply to us on: neiltrotterfoundations <at> yahoo.com.hk

Regards;
Neil Trotter

--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo <at> vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

jerryfunds6 | 30 Aug 04:21 2015

(unknown)

We Give Out Loans For 3% Interest Rate And We Offer Loans From $5,000 To $50,000,000.00, Are You Looking To
Buy A House Car Or Company Or Start Up A Truck Company or Buy A Truck Or Personal Loans, Email Us At
j.funds2000000 <at> inbox.lv  With Amount Needed And Phone Number.
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo <at> vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

jerryfunds24 | 29 Aug 20:43 2015

(unknown)

We Give Out Loans For 3% Interest Rate And We Offer Loans From $5,000 To $50,000,000.00, Are You Looking To
Buy A House Car Or Company Or Start Up A Truck Company or Buy A Truck Or Personal Loans, Email Us At
j.funds2000000 <at> inbox.lv  With Amount Needed And Phone Number.
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo <at> vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Toshi Kani | 28 Aug 16:40 2015
Picon

[PATCH]: acpica/nfit: Rename not-armed bit definition

ACPI 6.0 NFIT Memory Device State Flags in Table 5-129 defines
bit 3 as follows.

  Bit [3] set to 1 to indicate that the Memory Device is observed
  to be not armed prior to OSPM hand off. A Memory Device is
  considered armed if it is able to accept persistent writes.

This bit is currently defined as ACPI_NFIT_MEM_ARMED, which can be
confusing as if the Memory Device is armed when this bit is set.

Change the name to ACPI_NFIT_MEM_NOT_ARMED per the spec.

Signed-off-by: Toshi Kani <toshi.kani <at> hp.com>
Cc: Bob Moore <robert.moore <at> intel.com>
Cc: Dan Williams <dan.j.williams <at> intel.com>
Cc: Rafael J. Wysocki <rafael.j.wysocki <at> intel.com>
---
This patch is based on the linus's tree.
---
 drivers/acpi/nfit.c              |    6 +++---
 drivers/acpi/nfit.h              |    2 +-
 include/acpi/actbl1.h            |    2 +-
 tools/testing/nvdimm/test/nfit.c |    2 +-
 4 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/acpi/nfit.c b/drivers/acpi/nfit.c
index cf0fd96..4596e0a 100644
--- a/drivers/acpi/nfit.c
+++ b/drivers/acpi/nfit.c
 <at>  <at>  -705,7 +705,7  <at>  <at>  static ssize_t flags_show(struct device *dev,
(Continue reading)

Vicky | 28 Aug 14:51 2015
Picon

1

What time can i call you?

--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo <at> vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Dennis Chen | 28 Aug 14:49 2015

[PATCH] ACPI / ARM64: Get configuration base address of ECAM via ACPI MCFG table

This patch will fall back to ACPI MCFG table if _CBA method fails
to get the configuration base address of ECAM. Firmware on ARM
platform uses MCFG table instead of _CBA method. This is needed
to scan the PCIe root complex for ARM SoC.

Signed-off-by: Dennis Chen <dennis.chen <at> arm.com>
---
 drivers/pci/pci-acpi.c | 102 ++++++++++++++++++++++++++++++++++++++++++++++---
 1 file changed, 97 insertions(+), 5 deletions(-)

diff --git a/drivers/pci/pci-acpi.c b/drivers/pci/pci-acpi.c
index 314a625..211b9d9 100644
--- a/drivers/pci/pci-acpi.c
+++ b/drivers/pci/pci-acpi.c
 <at>  <at>  -27,18 +27,110  <at>  <at>  const u8 pci_acpi_dsm_uuid[] = {
 	0x91, 0x17, 0xea, 0x4d, 0x19, 0xc3, 0x43, 0x4d
 };

+static int __init acpi_mcfg_check_entry(struct acpi_table_mcfg *mcfg)
+{
+	if (mcfg->header.revision != 1) {
+		pr_err("invalid header revision:%d in ACPI MCFG table\n",
+			mcfg->header.revision);
+		return -EINVAL;
+	}
+
+	return 0;
+}
+
+static phys_addr_t __init acpi_get_mcfg_cba(u16 segment, u8 bus)
(Continue reading)

Unknown | 28 Aug 04:10 2015

LOAN OFFER OF ALL TYPES

Good day, PAUL FREDRICK FINANCE offer loans at 2% annual rate. Apply with amount needed, loan duration and
phone number Via: 
:paulfredrickfinanceservice <at> gmail.com 
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo <at> vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Ross Zwisler | 27 Aug 21:14 2015
Picon

[PATCH v3] change aperture mapping from WC to WB

The patch formerly known as `nd_blk: add support for "read flush" DSM flag'.

I changed the patch title and updated the description to place more emphasis on
the real purpose of the change, which is to move the BLK aperture mapping to
WB.  Really the implementation of the "read flush" flag was just to enable this
mapping change.

Ross Zwisler (1):
  nd_blk: change aperture mapping from WC to WB

 arch/x86/Kconfig                  |  1 +
 arch/x86/include/asm/cacheflush.h |  2 ++
 arch/x86/include/asm/io.h         |  2 --
 arch/x86/include/asm/pmem.h       |  2 ++
 drivers/acpi/Kconfig              |  1 +
 drivers/acpi/nfit.c               | 55 ++++++++++++++++++++++-----------------
 drivers/acpi/nfit.h               | 16 ++++++++----
 lib/Kconfig                       |  3 +++
 tools/testing/nvdimm/Kbuild       |  2 ++
 tools/testing/nvdimm/test/iomap.c | 30 +++++++++++++++++++--
 tools/testing/nvdimm/test/nfit.c  | 10 ++++---
 11 files changed, 88 insertions(+), 36 deletions(-)

--

-- 
2.1.0

--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo <at> vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
(Continue reading)

Jiang Liu | 27 Aug 09:31 2015
Picon

Lockdep warnings triggered by CPU hotplug

Hi Rafael,
	With the latest v4.2-rc8, socket hot-removal triggers following
lockdep warning. I will dig more, but any help are welcomed!
Thanks!
Gerry
[  294.957923] ======================================================
[  294.966401] [ INFO: possible circular locking dependency detected ]
[  294.974959] 4.2.0-rc8+ #6 Not tainted
[  294.979614] -------------------------------------------------------
[  294.986696] kworker/u288:3/627 is trying to acquire lock:
[  294.992799]  (cpu_add_remove_lock){+.+.+.}, at: [<ffffffff818536c2>]
cpu_down+0x22/0x50
[  295.007000]
[  295.007000] but task is already holding lock:
[  295.013593]  (&device->physical_node_lock){+.+.+.}, at:
[<ffffffff814b7acb>] acpi_bus_offline+0x73/0x107
[  295.024706]
[  295.024706] which lock already depends on the new lock.
[  295.024706]
[  295.034007]
[  295.034007] the existing dependency chain (in reverse order) is:
[  295.042503]
[  295.042503] -> #3 (&device->physical_node_lock){+.+.+.}:
[  295.049034]        [<ffffffff810df04d>] lock_acquire+0xdd/0x2a0
[  295.055831]        [<ffffffff81864430>] mutex_lock_nested+0x70/0x3e0
[  295.063111]        [<ffffffff814b713d>] acpi_companion_match+0x49/0x80
[  295.070581]        [<ffffffff814b8482>] acpi_match_device+0x16/0x27
[  295.077762]        [<ffffffff814bf719>]
acpi_lpss_platform_notify+0x27/0x108
[  295.085910]        [<ffffffff810a873d>] notifier_call_chain+0x5d/0x80
(Continue reading)


Gmane