linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: hanjun.guo@linaro.org (Hanjun Guo)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v9 00/21] Introduce ACPI for ARM64 based on ACPI 5.1
Date: Fri, 27 Feb 2015 16:37:10 +0800	[thread overview]
Message-ID: <54F02CB6.2090508@linaro.org> (raw)
In-Reply-To: <54EFE282.1090305@codeaurora.org>

Hi Timur,

On 2015?02?27? 11:20, Timur Tabi wrote:
> On 02/25/2015 02:39 AM, Hanjun Guo wrote:
>> Changes since v8:
>
> I'm still debugging it, but v9 on the 4.0-rc1 kernel crashes after
> calling the UEFI boot time services exit function.  That is, this line:
>
> status = sys_table->boottime->exit_boot_services(handle, mmap_key);
>
> in allocate_new_fdt_and_exit_boot() gets called, and then soon after it
> returns, the kernel crashes.  It's really early because the UEFI
> exception handler is called.
>
> I did not have this problem with v8 patchset on 3.19.

I tested this patch set on FVP base model with UEFI, and booted ok.
Can you boot the system without this ACPI patch set?

>
>
> Also, you still have a couple whitespace problems in these patches:
>
> Applying: ARM64 / ACPI: additions of ACPI documentation for arm64
> /local/mnt/workspace/linux.0/.git/rebase-apply/patch:607: new blank line
> at EOF.
> +
> /local/mnt/workspace/linux.0/.git/rebase-apply/patch:844: new blank line
> at EOF.
> +
> warning: 2 lines add whitespace errors.
> Applying: Documentation: ACPI for ARM64
> /local/mnt/workspace/linux.0/.git/rebase-apply/patch:517: new blank line
> at EOF.
> +
> warning: 1 line adds whitespace errors.

Weired, When I checked the patch locally there were no whitespace
problems:

./scripts/checkpatch.pl 0020-Documentation-ACPI-for-ARM64.patch
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#23:
new file mode 100644

WARNING: use relative pathname instead of absolute in changelog text
#25:
--- /dev/null

total: 0 errors, 2 warnings, 506 lines checked

0020-Documentation-ACPI-for-ARM64.patch has style problems, please review.

./scripts/checkpatch.pl 
0021-ARM64-ACPI-additions-of-ACPI-documentation-for-arm64.patch
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#32:
new file mode 100644

WARNING: use relative pathname instead of absolute in changelog text
#34:
--- /dev/null

WARNING: use relative pathname instead of absolute in changelog text
#634:
--- /dev/null

total: 0 errors, 3 warnings, 825 lines checked

0021-ARM64-ACPI-additions-of-ACPI-documentation-for-arm64.patch has 
style problems, please review.

Can you provide me the exact line of code which have whitespace problems?

Thanks
Hanjun

  reply	other threads:[~2015-02-27  8:37 UTC|newest]

Thread overview: 106+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-25  8:39 [PATCH v9 00/21] Introduce ACPI for ARM64 based on ACPI 5.1 Hanjun Guo
2015-02-25  8:39 ` [PATCH v9 01/21] ACPI / table: Use pr_debug() instead of pr_info() for MADT table scanning Hanjun Guo
2015-03-04 22:33   ` Rafael J. Wysocki
2015-03-05 17:55   ` Olof Johansson
2015-03-06 20:17   ` Grant Likely
2015-03-06 20:31     ` Joe Perches
2015-03-10  2:35       ` Hanjun Guo
2015-02-25  8:39 ` [PATCH v9 02/21] ACPI / processor: Introduce phys_cpuid_t for CPU hardware ID Hanjun Guo
2015-03-04 22:29   ` Rafael J. Wysocki
2015-03-05  7:44     ` Hanjun Guo
2015-03-05 13:23       ` Rafael J. Wysocki
2015-03-06  6:48         ` Hanjun Guo
2015-03-06 20:19   ` Grant Likely
2015-02-25  8:39 ` [PATCH v9 03/21] ACPI: add arm64 to the platforms that use ioremap Hanjun Guo
2015-03-04 22:33   ` Rafael J. Wysocki
2015-03-06 20:20   ` Grant Likely
2015-02-25  8:39 ` [PATCH v9 04/21] ARM64: allow late use of early_ioremap Hanjun Guo
2015-03-06 20:24   ` Grant Likely
2015-02-25  8:39 ` [PATCH v9 05/21] ARM64 / ACPI: Get RSDP and ACPI boot-time tables Hanjun Guo
2015-03-05 18:10   ` Olof Johansson
2015-03-05 18:51   ` Lorenzo Pieralisi
2015-03-10  8:01     ` Hanjun Guo
2015-03-10  9:32       ` Lorenzo Pieralisi
2015-03-10 11:19       ` Leif Lindholm
2015-03-10 11:36         ` Hanjun Guo
2015-03-06 20:28   ` Grant Likely
2015-02-25  8:39 ` [PATCH v9 06/21] ACPI: fix acpi_os_ioremap for arm64 Hanjun Guo
2015-03-04 22:36   ` Rafael J. Wysocki
2015-03-06 20:30   ` Grant Likely
2015-02-25  8:39 ` [PATCH v9 07/21] ACPI / sleep: Introduce arm64 specific acpi_sleep.c Hanjun Guo
2015-03-04 22:38   ` Rafael J. Wysocki
2015-03-04 22:49     ` G Gregory
2015-03-04 23:25       ` Rafael J. Wysocki
2015-03-05  0:16         ` Rafael J. Wysocki
2015-03-06 12:36           ` Lorenzo Pieralisi
2015-03-06 20:34   ` Grant Likely
2015-02-25  8:39 ` [PATCH v9 08/21] ARM64 / ACPI: Introduce PCI stub functions for ACPI Hanjun Guo
2015-03-06 18:31   ` Lorenzo Pieralisi
2015-03-10  9:21     ` Hanjun Guo
2015-03-06 20:36   ` Grant Likely
2015-03-09 15:01   ` Liviu Dudau
2015-03-10  9:34     ` Hanjun Guo
2015-02-25  8:39 ` [PATCH v9 09/21] ARM64 / ACPI: Introduce early_param "acpi=" to enable/disable ACPI Hanjun Guo
2015-03-05 18:11   ` Olof Johansson
2015-03-06 20:37   ` Grant Likely
2015-02-25  8:39 ` [PATCH v9 10/21] ARM64 / ACPI: If we chose to boot from acpi then disable FDT Hanjun Guo
2015-03-05 18:12   ` Olof Johansson
2015-03-06 20:38   ` Grant Likely
2015-02-25  8:39 ` [PATCH v9 11/21] ARM64 / ACPI: Get PSCI flags in FADT for PSCI init Hanjun Guo
2015-03-05 18:19   ` Olof Johansson
2015-03-06 20:40   ` Grant Likely
2015-02-25  8:39 ` [PATCH v9 12/21] ACPI / table: Print GIC information when MADT is parsed Hanjun Guo
2015-03-04 22:40   ` Rafael J. Wysocki
2015-03-06 18:06   ` Lorenzo Pieralisi
2015-03-06 20:40   ` Grant Likely
2015-02-25  8:39 ` [PATCH v9 13/21] ARM64 / ACPI: Parse MADT for SMP initialization Hanjun Guo
2015-03-05 18:49   ` Olof Johansson
2015-03-10 11:33     ` Hanjun Guo
2015-03-07 22:49   ` Grant Likely
2015-02-25  8:39 ` [PATCH v9 14/21] ACPI / processor: Make it possible to get CPU hardware ID via GICC Hanjun Guo
2015-03-04 22:46   ` Rafael J. Wysocki
2015-03-05  8:03     ` Hanjun Guo
2015-03-05 11:27       ` Catalin Marinas
2015-03-05 13:13         ` Rafael J. Wysocki
2015-03-05 15:19           ` Catalin Marinas
2015-03-06  6:51             ` Hanjun Guo
2015-03-07 22:52   ` Grant Likely
2015-02-25  8:39 ` [PATCH v9 15/21] ARM64 / ACPI: Introduce ACPI_IRQ_MODEL_GIC and register device's gsi Hanjun Guo
2015-03-04 22:47   ` Rafael J. Wysocki
2015-03-07 23:05   ` Grant Likely
2015-02-25  8:39 ` [PATCH v9 16/21] irqchip: Add GICv2 specific ACPI boot support Hanjun Guo
2015-03-04 22:50   ` Rafael J. Wysocki
2015-03-05  9:06     ` Hanjun Guo
2015-03-05 11:53     ` Catalin Marinas
2015-03-06  0:42       ` Rafael J. Wysocki
2015-03-05  8:21   ` Hanjun Guo
2015-03-07 23:14   ` Grant Likely
2015-02-25  8:39 ` [PATCH v9 17/21] clocksource / arch_timer: Parse GTDT to initialize arch timer Hanjun Guo
2015-03-07 23:16   ` Grant Likely
2015-02-25  8:39 ` [PATCH v9 18/21] ARM64 / ACPI: Select ACPI_REDUCED_HARDWARE_ONLY if ACPI is enabled on ARM64 Hanjun Guo
2015-03-06 17:47   ` Lorenzo Pieralisi
2015-03-10 12:23     ` Hanjun Guo
2015-03-10 14:16       ` Lorenzo Pieralisi
2015-03-07 23:16   ` Grant Likely
2015-02-25  8:39 ` [PATCH v9 19/21] ARM64 / ACPI: Enable ARM64 in Kconfig Hanjun Guo
2015-03-04 22:52   ` Rafael J. Wysocki
2015-03-07 23:17   ` Grant Likely
2015-02-25  8:40 ` [PATCH v9 20/21] Documentation: ACPI for ARM64 Hanjun Guo
2015-02-27 10:53   ` Shannon Zhao
2015-02-27 11:13     ` Shannon Zhao
2015-02-27 11:20   ` Shannon Zhao
2015-02-25  8:40 ` [PATCH v9 21/21] ARM64 / ACPI: additions of ACPI documentation for arm64 Hanjun Guo
2015-02-27 11:22   ` Shannon Zhao
2015-02-27 14:19     ` Hanjun Guo
2015-03-05 18:54   ` Olof Johansson
2015-02-27  3:20 ` [PATCH v9 00/21] Introduce ACPI for ARM64 based on ACPI 5.1 Timur Tabi
2015-02-27  8:37   ` Hanjun Guo [this message]
2015-02-27 10:51     ` Shannon Zhao
2015-02-27  8:50   ` Ard Biesheuvel
2015-02-27 10:36     ` Mark Rutland
2015-02-27 21:05     ` Timur Tabi
2015-03-04 23:18     ` Timur Tabi
2015-03-04 22:56 ` Rafael J. Wysocki
2015-03-05  7:03   ` Hanjun Guo
2015-03-05 18:57 ` Olof Johansson
2015-03-06  4:26   ` Hanjun Guo

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=54F02CB6.2090508@linaro.org \
    --to=hanjun.guo@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).