All of lore.kernel.org
 help / color / mirror / Atom feed
From: Laszlo Ersek <lersek@redhat.com>
To: Auger Eric <eric.auger@redhat.com>, Igor Mammedov <imammedo@redhat.com>
Cc: "peter.maydell@linaro.org" <peter.maydell@linaro.org>,
	"sameo@linux.intel.com" <sameo@linux.intel.com>,
	Ard Biesheuvel <ard.biesheuvel@linaro.org>,
	"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
	Shameerali Kolothum Thodi <shameerali.kolothum.thodi@huawei.com>,
	Linuxarm <linuxarm@huawei.com>,
	"shannon.zhaosl@gmail.com" <shannon.zhaosl@gmail.com>,
	"qemu-arm@nongnu.org" <qemu-arm@nongnu.org>,
	"xuwei \(O\)" <xuwei5@huawei.com>,
	"sebastien.boeuf@intel.com" <sebastien.boeuf@intel.com>,
	Leif Lindholm <Leif.Lindholm@arm.com>
Subject: Re: [Qemu-arm] [Qemu-devel] [PATCH v3 07/10] hw/arm/virt: Introduce opt-in feature "fdt"
Date: Tue, 2 Apr 2019 17:56:00 +0200	[thread overview]
Message-ID: <59dfd9cd-3762-cbb6-0823-e1247ec5fcd7@redhat.com> (raw)
In-Reply-To: <91a906c2-ef90-3aec-2b2c-fbb63c41f7c2@redhat.com>

On 04/02/19 17:52, Laszlo Ersek wrote:
> On 04/02/19 17:42, Auger Eric wrote:

>>>>> The firmware does consume DT:
>>>>>
>>>>> - If you start QEMU *with* "-no-acpi", then the DT is both consumed by
>>>>> the firmware (for its own information needs), and passed on to the OS.
>>>>>
>>>>> - If you start QEMU *without* "-no-acpi" (the default), then the DT is
>>>>> consumed only by the firmware (for its own information needs), and the
>>>>> DT is hidden from the OS. The OS gets only the ACPI content
>>>>> (processed/prepared as described above).
> 
>> I am confused by the above statement actually. In the above case what
>> does happen if you pass the acpi=off in the kernel boot parameters?
> 
> If you launch QEMU with "-no-acpi" and you pass "acpi=off" to the guest
> kernel, then the kernel will not boot successfully, as it will not get
> DT from the firmware, and it will ignore the ACPI tables that it does
> get from the firmware.

Sorry, I ended up answering "what happens when you run QEMU *without*
-no-acpi and pass acpi=off to the guest kernel".

To explain what happens when you boot *with* -no-acpi: in that case,
"acpi=off" doesn't matter, since the guest kernel doesn't get ACPI
tables anyway. The kernel will go for DT.

Thanks
Laszlo

WARNING: multiple messages have this Message-ID (diff)
From: Laszlo Ersek <lersek@redhat.com>
To: Auger Eric <eric.auger@redhat.com>, Igor Mammedov <imammedo@redhat.com>
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>,
	"peter.maydell@linaro.org" <peter.maydell@linaro.org>,
	"sameo@linux.intel.com" <sameo@linux.intel.com>,
	"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
	Shameerali Kolothum Thodi <shameerali.kolothum.thodi@huawei.com>,
	Linuxarm <linuxarm@huawei.com>,
	"shannon.zhaosl@gmail.com" <shannon.zhaosl@gmail.com>,
	"qemu-arm@nongnu.org" <qemu-arm@nongnu.org>,
	"xuwei (O)" <xuwei5@huawei.com>,
	"sebastien.boeuf@intel.com" <sebastien.boeuf@intel.com>,
	Leif Lindholm <Leif.Lindholm@arm.com>
Subject: Re: [Qemu-devel] [PATCH v3 07/10] hw/arm/virt: Introduce opt-in feature "fdt"
Date: Tue, 2 Apr 2019 17:56:00 +0200	[thread overview]
Message-ID: <59dfd9cd-3762-cbb6-0823-e1247ec5fcd7@redhat.com> (raw)
In-Reply-To: <91a906c2-ef90-3aec-2b2c-fbb63c41f7c2@redhat.com>

On 04/02/19 17:52, Laszlo Ersek wrote:
> On 04/02/19 17:42, Auger Eric wrote:

>>>>> The firmware does consume DT:
>>>>>
>>>>> - If you start QEMU *with* "-no-acpi", then the DT is both consumed by
>>>>> the firmware (for its own information needs), and passed on to the OS.
>>>>>
>>>>> - If you start QEMU *without* "-no-acpi" (the default), then the DT is
>>>>> consumed only by the firmware (for its own information needs), and the
>>>>> DT is hidden from the OS. The OS gets only the ACPI content
>>>>> (processed/prepared as described above).
> 
>> I am confused by the above statement actually. In the above case what
>> does happen if you pass the acpi=off in the kernel boot parameters?
> 
> If you launch QEMU with "-no-acpi" and you pass "acpi=off" to the guest
> kernel, then the kernel will not boot successfully, as it will not get
> DT from the firmware, and it will ignore the ACPI tables that it does
> get from the firmware.

Sorry, I ended up answering "what happens when you run QEMU *without*
-no-acpi and pass acpi=off to the guest kernel".

To explain what happens when you boot *with* -no-acpi: in that case,
"acpi=off" doesn't matter, since the guest kernel doesn't get ACPI
tables anyway. The kernel will go for DT.

Thanks
Laszlo

  reply	other threads:[~2019-04-02 15:56 UTC|newest]

Thread overview: 95+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-03-21 10:47 [Qemu-arm] [PATCH v3 00/10] ARM virt: ACPI memory hotplug support Shameer Kolothum
2019-03-21 10:47 ` [Qemu-devel] [PATCH v3 01/10] hw/acpi: Make ACPI IO address space configurable Shameer Kolothum
2019-04-01 12:58   ` [Qemu-arm] " Igor Mammedov
2019-04-01 12:58     ` [Qemu-devel] " Igor Mammedov
2019-03-21 10:47 ` [Qemu-devel] [PATCH v3 02/10] hw/acpi: Do not create memory hotplug method when handler is not defined Shameer Kolothum
2019-03-28 14:14   ` [Qemu-arm] " Auger Eric
2019-03-21 10:47 ` [Qemu-arm] [PATCH v3 03/10] hw/arm/virt: Add virtual ACPI device Shameer Kolothum
2019-03-28 14:14   ` Auger Eric
2019-03-29 11:22     ` Shameerali Kolothum Thodi
2019-04-01 13:08       ` [Qemu-arm] [Qemu-devel] " Igor Mammedov
2019-04-01 13:08         ` Igor Mammedov
2019-04-01 14:21         ` Shameerali Kolothum Thodi
2019-04-01 14:21           ` Shameerali Kolothum Thodi
2019-04-02  6:31           ` [Qemu-arm] " Igor Mammedov
2019-04-02  6:31             ` Igor Mammedov
2019-03-21 10:47 ` [Qemu-arm] [PATCH v3 04/10] hw/arm/virt: Add memory hotplug framework Shameer Kolothum
2019-03-28 15:37   ` Auger Eric
2019-03-29 12:03     ` Shameerali Kolothum Thodi
2019-03-21 10:47 ` [Qemu-devel] [PATCH v3 05/10] hw/arm/virt: Add ACPI support for device memory cold-plug Shameer Kolothum
2019-03-29  9:31   ` [Qemu-arm] " Auger Eric
2019-03-29 10:54     ` Shameerali Kolothum Thodi
2019-04-01 13:43     ` Igor Mammedov
2019-04-01 13:43       ` [Qemu-devel] " Igor Mammedov
2019-04-01 14:51       ` [Qemu-arm] " Shameerali Kolothum Thodi
2019-04-01 14:51         ` [Qemu-devel] " Shameerali Kolothum Thodi
2019-04-02  7:19         ` [Qemu-arm] " Igor Mammedov
2019-04-02  7:19           ` Igor Mammedov
2019-04-01 14:59       ` [Qemu-arm] " Auger Eric
2019-04-01 14:59         ` Auger Eric
2019-04-01 13:34   ` [Qemu-arm] " Igor Mammedov
2019-04-01 13:34     ` [Qemu-devel] " Igor Mammedov
2019-04-01 16:24     ` [Qemu-arm] " Shameerali Kolothum Thodi
2019-04-01 16:24       ` [Qemu-devel] " Shameerali Kolothum Thodi
2019-04-02  7:22       ` Igor Mammedov
2019-04-02  7:22         ` Igor Mammedov
2019-03-21 10:47 ` [Qemu-devel] [PATCH v3 06/10] hw/arm/virt-acpi-build: Add PC-DIMM in SRAT Shameer Kolothum
2019-03-21 10:47 ` [Qemu-arm] [PATCH v3 07/10] hw/arm/virt: Introduce opt-in feature "fdt" Shameer Kolothum
2019-03-29  9:31   ` Auger Eric
2019-03-29  9:41     ` Shameerali Kolothum Thodi
2019-03-29 13:41       ` Auger Eric
2019-03-29  9:59     ` Shameerali Kolothum Thodi
2019-03-29 13:12       ` Auger Eric
2019-03-29 13:14         ` Ard Biesheuvel
2019-03-29 13:56           ` [Qemu-arm] [Qemu-devel] " Auger Eric
2019-03-29 14:08             ` Shameerali Kolothum Thodi
2019-04-01 13:07             ` Laszlo Ersek
2019-04-01 13:07               ` Laszlo Ersek
2019-04-02  7:42               ` Igor Mammedov
2019-04-02  7:42                 ` Igor Mammedov
2019-04-02 10:33                 ` [Qemu-arm] " Laszlo Ersek
2019-04-02 10:33                   ` Laszlo Ersek
2019-04-02 15:42                   ` [Qemu-arm] " Auger Eric
2019-04-02 15:42                     ` Auger Eric
2019-04-02 15:52                     ` Laszlo Ersek
2019-04-02 15:52                       ` Laszlo Ersek
2019-04-02 15:56                       ` Laszlo Ersek [this message]
2019-04-02 15:56                         ` Laszlo Ersek
2019-04-02 16:07                       ` [Qemu-arm] " Auger Eric
2019-04-02 16:07                         ` Auger Eric
2019-04-02 14:26               ` [Qemu-arm] " Shameerali Kolothum Thodi
2019-04-02 14:26                 ` Shameerali Kolothum Thodi
2019-04-02 15:29               ` [Qemu-arm] " Auger Eric
2019-04-02 15:29                 ` Auger Eric
2019-04-02 15:38                 ` Laszlo Ersek
2019-04-02 15:38                   ` Laszlo Ersek
2019-04-02 15:50                   ` Auger Eric
2019-04-02 15:50                     ` Auger Eric
2019-04-03  9:49                   ` [Qemu-arm] " Igor Mammedov
2019-04-03  9:49                     ` Igor Mammedov
2019-04-03 12:10                     ` [Qemu-arm] " Shameerali Kolothum Thodi
2019-04-03 12:10                       ` Shameerali Kolothum Thodi
2019-04-03 13:29                       ` [Qemu-arm] " Laszlo Ersek
2019-04-03 13:29                         ` Laszlo Ersek
2019-04-03 16:25                         ` [Qemu-arm] " Shameerali Kolothum Thodi
2019-04-03 16:25                           ` Shameerali Kolothum Thodi
2019-04-08  8:11                           ` [Qemu-arm] " Igor Mammedov
2019-04-08  8:11                             ` Igor Mammedov
2019-04-08  8:11                             ` Igor Mammedov
2019-04-09 10:43                             ` [Qemu-arm] " Shameerali Kolothum Thodi
2019-04-09 10:43                               ` Shameerali Kolothum Thodi
2019-04-09 10:43                               ` Shameerali Kolothum Thodi
2019-04-03 13:19                     ` [Qemu-arm] " Laszlo Ersek
2019-04-03 13:19                       ` Laszlo Ersek
2019-04-08  8:13                       ` [Qemu-arm] " Igor Mammedov
2019-04-08  8:13                         ` Igor Mammedov
2019-04-08  8:13                         ` Igor Mammedov
2019-04-02  8:39             ` [Qemu-arm] " Peter Maydell
2019-04-02  8:39               ` Peter Maydell
2019-03-21 10:47 ` [Qemu-arm] [PATCH v3 08/10] hw/arm/boot: Expose the PC-DIMM nodes in the DT Shameer Kolothum
2019-03-21 10:47 ` [Qemu-devel] [PATCH v3 09/10] hw/acpi: Add ACPI Generic Event Device Support Shameer Kolothum
2019-03-29 13:09   ` [Qemu-arm] " Auger Eric
2019-03-29 13:44     ` Shameerali Kolothum Thodi
2019-03-21 10:47 ` [Qemu-devel] [PATCH v3 10/10] hw/arm/virt: Init GED device and enable memory hotplug Shameer Kolothum
2019-03-29 14:16   ` [Qemu-arm] " Auger Eric
2019-03-21 11:06 ` [Qemu-arm] [Qemu-devel] [PATCH v3 00/10] ARM virt: ACPI memory hotplug support no-reply

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=59dfd9cd-3762-cbb6-0823-e1247ec5fcd7@redhat.com \
    --to=lersek@redhat.com \
    --cc=Leif.Lindholm@arm.com \
    --cc=ard.biesheuvel@linaro.org \
    --cc=eric.auger@redhat.com \
    --cc=imammedo@redhat.com \
    --cc=linuxarm@huawei.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-arm@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=sameo@linux.intel.com \
    --cc=sebastien.boeuf@intel.com \
    --cc=shameerali.kolothum.thodi@huawei.com \
    --cc=shannon.zhaosl@gmail.com \
    --cc=xuwei5@huawei.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.