From: Julien Grall <julien.grall@linaro.org>
To: Manish Jaggi <mjaggi@caviumnetworks.com>,
Julien Grall <julien.grall@arm.com>,
Andre Przywara <andre.przywara@arm.com>
Cc: xen-devel <xen-devel@lists.xenproject.org>,
sstabellini@kernel.org, Manish Jaggi <mjaggi@cavium.com>,
tomasz.nowicki@cavium.com
Subject: Re: [PATCH v2 0/2] ARM: ACPI: IORT: Hide SMMU from hardware domain's IORT table
Date: Thu, 12 Oct 2017 12:44:57 +0100 [thread overview]
Message-ID: <2c4fe6a3-e3dc-8cf6-dd3d-ccf10641fa12@linaro.org> (raw)
In-Reply-To: <55d35b00-af71-b239-d782-61ef74a96b6e@caviumnetworks.com>
On 12/10/17 12:22, Manish Jaggi wrote:
> Hi Julien,
>
> Why do you omit parts of mail where I have asked a question , please
> avoid skiping that removes the context.
I believe I answered it just after because you asked twice the same
thing. So may I dropped the context but the answer was there...
For your convenience here the replicated answer.
"Why? The generation of IORT is fairly standalone.
And again, this was suggestion to share in the future and an expectation
for this series. What I care the most is the generation to be fully
separated from the rest."
> I raised a valid point and it was totally ignored and you asked me to
> explain the rationale on a later point.
> So if you choose to ignore my first point, how can I put any point.
Well, maybe you should read the e-mail more carefully because your point
have been addressed. If they are not, then please say it rather than
accusing the reviewers on spending not enough time on your series...
[...]
> Now if you see both the codes are quite similar and there is redundancy
> in libxl and in xen code for preparing ACPI tables for dom0 and domU.
> The point I am raising is quite clear, if all other tables like MADT,
> XSDT, RSDP, GTDT etc does not share a common generation code with xen
> what is so special about IORT.
> Either we move all generation into a common code or keep redundancy for
> IORT.
>
> I hope I have shown the code and made the point quite clear.
> Please provide a technical answer rather than a simple "Why".
Why do you still continue arguing on how this is going to interact with
libxl when your only work now (as I stated in every single e-mail) is
for Dom0.
If the generation is generic enough, it will require little code to
interface. After all, you only need:
- informations (e.g DeviceID, MasterID...)
- buffer for writing the generated IORT
So now it is maybe time for you to suggest an interface we can discuss on.
Cheers,
--
Julien Grall
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
next prev parent reply other threads:[~2017-10-12 11:45 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-11 21:33 [PATCH v2 0/2] ARM: ACPI: IORT: Hide SMMU from hardware domain's IORT table mjaggi
2017-09-11 21:33 ` [PATCH 1/2] ARM: ACPI: IORT: Estimate the size of hardware domain " mjaggi
2017-09-11 21:33 ` [PATCH 2/2] ARM: ACPI: IORT: Write Hardware domain's " mjaggi
2017-09-22 14:12 ` [PATCH v2 0/2] ARM: ACPI: IORT: Hide SMMU from hardware " Andre Przywara
2017-09-25 4:22 ` Manish Jaggi
2017-10-03 18:42 ` Julien Grall
2017-10-04 5:22 ` Manish Jaggi
2017-10-06 14:24 ` Julien Grall
2017-10-12 6:11 ` Manish Jaggi
2017-10-12 11:04 ` Julien Grall
2017-10-12 11:22 ` Manish Jaggi
2017-10-12 11:44 ` Julien Grall [this message]
2017-10-12 21:08 ` Manish Jaggi
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=2c4fe6a3-e3dc-8cf6-dd3d-ccf10641fa12@linaro.org \
--to=julien.grall@linaro.org \
--cc=andre.przywara@arm.com \
--cc=julien.grall@arm.com \
--cc=mjaggi@cavium.com \
--cc=mjaggi@caviumnetworks.com \
--cc=sstabellini@kernel.org \
--cc=tomasz.nowicki@cavium.com \
--cc=xen-devel@lists.xenproject.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).