qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Aggeler  Fabian" <aggelerf@student.ethz.ch>
To: Greg Bellows <greg.bellows@linaro.org>
Cc: "Peter Maydell" <peter.maydell@linaro.org>,
	"Peter Crosthwaite" <peter.crosthwaite@xilinx.com>,
	"Aggeler Fabian" <aggelerf@student.ethz.ch>,
	"Rob Herring" <rob.herring@linaro.org>,
	"QEMU Developers" <qemu-devel@nongnu.org>,
	"Alexander Graf" <agraf@suse.de>,
	"John Williams" <john.williams@xilinx.com>,
	"Edgar E. Iglesias" <edgar.iglesias@gmail.com>,
	"Alex Bennée" <alex.bennee@linaro.org>
Subject: Re: [Qemu-devel] [PATCH v1 00/22] target-arm: Preparations for A64 EL2 and 3
Date: Thu, 15 May 2014 09:28:09 +0000	[thread overview]
Message-ID: <5153C5A7-55BE-466C-859F-806F76457075@ethz.ch> (raw)
In-Reply-To: <CAOgzsHUHBbx1ix4i6m8j_nosFTk7jga2=uSh+z0p-0=c_geq0Q@mail.gmail.com>

Hi Greg

Thanks for your comments. I still have to work through them. I am using OpenVirtualization in secure world, which then switches to a Linux kernel in non-secure world to test the patches. What about you?

Best,
Fabian

On 14 May 2014, at 15:55, Greg Bellows <greg.bellows@linaro.org<mailto:greg.bellows@linaro.org>> wrote:

Hi Fabian,

I too had been updating the core TZ patches provided by Samsung.  From looking at your changes I see a lot of similarities in our code with the exception being the mechanism for banked register support.  The difference being that your approach is a bit more explicit in the declaration of the banked registers.  Whereas my approach was to update the banked registers once all the other registers were registered.  Both approaches I believe work.

I spoke with Peter M. and he and I are okay with your approach.  I will be looking closer at your patches today and making comments.

One thing that held me up from committing sooner was testing my changes.  Do you have a good approach for testing the changes?

Regards,

Greg


On 14 May 2014 03:58, Aggeler Fabian <aggelerf@student.ethz.ch<mailto:aggelerf@student.ethz.ch>> wrote:
I see. What is Greg Bellows working on exactly? Also peripherals like TZASC, TZPC,...? My plan is to focus on them now if no one else is working on them. What do you suggest to minimize overlap?

Thanks,
Fabian
________________________________________
From: Peter Maydell [peter.maydell@linaro.org<mailto:peter.maydell@linaro.org>]
Sent: Monday, May 12, 2014 10:39 PM
To: Aggeler  Fabian
Cc: Edgar E. Iglesias; Rob Herring; Peter Crosthwaite; QEMU Developers; Alexander Graf; John Williams; Alex Bennée; Greg Bellows
Subject: Re: [Qemu-devel] [PATCH v1 00/22] target-arm: Preparations for A64 EL2 and 3

On 12 May 2014 20:13, Aggeler  Fabian <aggelerf@student.ethz.ch<mailto:aggelerf@student.ethz.ch>> wrote:
> I’ve been reworking the Samsung patches as part of my Master thesis and I wanted to
> send them some time this week. I am currently rebasing them when I noticed Edgar’s
> patches. Is there some branch with the patches so I could rebase on them?

Hmm, that makes about three lots of people trying to do similar things
at this point. We should try to coordinate so we don't duplicate work.

thanks
-- PMM

  reply	other threads:[~2014-05-15  9:28 UTC|newest]

Thread overview: 74+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-06  6:08 [Qemu-devel] [PATCH v1 00/22] target-arm: Preparations for A64 EL2 and 3 Edgar E. Iglesias
2014-05-06  6:08 ` [Qemu-devel] [PATCH v1 01/22] target-arm: A64: Add friendly logging of PSTATE A and I flags Edgar E. Iglesias
2014-05-07  5:32   ` Peter Crosthwaite
2014-05-07  8:50   ` Peter Maydell
2014-05-08  0:08     ` Edgar E. Iglesias
2014-05-06  6:08 ` [Qemu-devel] [PATCH v1 02/22] target-arm: Make elr_el1 an array Edgar E. Iglesias
2014-05-07  5:10   ` Peter Crosthwaite
2014-05-08  0:13     ` Edgar E. Iglesias
2014-05-16 14:19       ` Peter Maydell
2014-05-16 22:19         ` Edgar E. Iglesias
2014-05-16 14:22   ` Peter Maydell
2014-05-16 22:18     ` Edgar E. Iglesias
2014-05-06  6:08 ` [Qemu-devel] [PATCH v1 03/22] target-arm: Make esr_el1 " Edgar E. Iglesias
2014-05-06  6:08 ` [Qemu-devel] [PATCH v1 04/22] target-arm: c12_vbar -> vbar_el[] Edgar E. Iglesias
2014-05-06  6:08 ` [Qemu-devel] [PATCH v1 05/22] target-arm: Add arm_el_to_mmu_idx() Edgar E. Iglesias
2014-05-07  5:43   ` Peter Crosthwaite
2014-05-16 14:24   ` Peter Maydell
2014-05-16 22:10     ` Edgar E. Iglesias
2014-05-16 22:13       ` Alexander Graf
2014-05-17  1:41         ` Edgar E. Iglesias
2014-05-06  6:08 ` [Qemu-devel] [PATCH v1 06/22] target-arm: Move get_mem_index to translate.h Edgar E. Iglesias
2014-05-06  6:08 ` [Qemu-devel] [PATCH v1 07/22] target-arm: A64: Add SP entries for EL2 and 3 Edgar E. Iglesias
2014-05-06  6:08 ` [Qemu-devel] [PATCH v1 08/22] target-arm: A64: Add ELR " Edgar E. Iglesias
2014-05-06  6:08 ` [Qemu-devel] [PATCH v1 09/22] target-arm: Add SPSR entries for EL2/HYP and EL3/MON Edgar E. Iglesias
2014-05-07  5:31   ` Edgar E. Iglesias
2014-05-06  6:08 ` [Qemu-devel] [PATCH v1 10/22] target-arm: A64: Introduce arm64_banked_spsr_index() Edgar E. Iglesias
2014-05-07  5:50   ` Peter Crosthwaite
2014-05-16 14:31   ` Peter Maydell
2014-05-17  2:21     ` Edgar E. Iglesias
2014-05-06  6:08 ` [Qemu-devel] [PATCH v1 11/22] target-arm: Add a feature flag for EL2 Edgar E. Iglesias
2014-05-07  5:50   ` Peter Crosthwaite
2014-05-06  6:08 ` [Qemu-devel] [PATCH v1 12/22] target-arm: Add a feature flag for EL3 Edgar E. Iglesias
2014-05-07  5:51   ` Peter Crosthwaite
2014-05-06  6:08 ` [Qemu-devel] [PATCH v1 13/22] target-arm: Register EL2 versions of ELR and SPSR Edgar E. Iglesias
2014-05-07  6:02   ` Peter Crosthwaite
2014-05-16 14:36     ` Peter Maydell
2014-05-06  6:08 ` [Qemu-devel] [PATCH v1 14/22] target-arm: Register EL3 " Edgar E. Iglesias
2014-05-07  6:02   ` Peter Crosthwaite
2014-05-06  6:08 ` [Qemu-devel] [PATCH v1 15/22] target-arm: A64: Forbid ERET to increase the EL Edgar E. Iglesias
2014-05-07  6:03   ` Peter Crosthwaite
2014-05-06  6:08 ` [Qemu-devel] [PATCH v1 16/22] target-arm: A64: Forbid ERET to unimplemented ELs Edgar E. Iglesias
2014-05-07  6:04   ` Peter Crosthwaite
2014-05-07  9:00   ` Peter Maydell
2014-05-08  0:14     ` Edgar E. Iglesias
2014-05-06  6:08 ` [Qemu-devel] [PATCH v1 17/22] target-arm: A64: Generalize ERET to various ELs Edgar E. Iglesias
2014-05-07  6:09   ` Peter Crosthwaite
2014-05-06  6:08 ` [Qemu-devel] [PATCH v1 18/22] target-arm: A64: Generalize update_spsel for the " Edgar E. Iglesias
2014-05-07  6:13   ` Peter Crosthwaite
2014-05-13 17:32   ` Richard Henderson
2014-05-14  1:18     ` Edgar E. Iglesias
2014-05-14 15:57       ` Richard Henderson
2014-05-06  6:08 ` [Qemu-devel] [PATCH v1 19/22] target-arm: Add storage for VBAR_EL2 and 3 Edgar E. Iglesias
2014-05-16 14:40   ` Peter Maydell
2014-05-17  1:42     ` Edgar E. Iglesias
2014-05-06  6:08 ` [Qemu-devel] [PATCH v1 20/22] target-arm: Make vbar_write writeback to any CPREG Edgar E. Iglesias
2014-05-07  6:19   ` Peter Crosthwaite
2014-05-06  6:08 ` [Qemu-devel] [PATCH v1 21/22] target-arm: A64: Register VBAR_EL2 Edgar E. Iglesias
2014-05-07  6:22   ` Peter Crosthwaite
2014-05-16 14:43   ` Peter Maydell
2014-05-06  6:08 ` [Qemu-devel] [PATCH v1 22/22] target-arm: A64: Register VBAR_EL3 Edgar E. Iglesias
2014-05-07  6:23   ` Peter Crosthwaite
2014-05-06  7:58 ` [Qemu-devel] [PATCH v1 00/22] target-arm: Preparations for A64 EL2 and 3 Peter Maydell
2014-05-07  3:46   ` Edgar E. Iglesias
2014-05-12 19:13     ` Aggeler  Fabian
2014-05-12 20:39       ` Peter Maydell
2014-05-14  8:58         ` Aggeler  Fabian
2014-05-14 13:55           ` Greg Bellows
2014-05-15  9:28             ` Aggeler  Fabian [this message]
2014-05-15  9:45               ` Sergey Fedorov
2014-05-15 12:44                 ` Christopher Covington
2014-05-14 14:56           ` Edgar E. Iglesias
2014-05-12 23:41       ` Peter Crosthwaite
2014-05-13  3:31       ` Edgar E. Iglesias
2014-05-06  8:24 ` Alexander Graf

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=5153C5A7-55BE-466C-859F-806F76457075@ethz.ch \
    --to=aggelerf@student.ethz.ch \
    --cc=agraf@suse.de \
    --cc=alex.bennee@linaro.org \
    --cc=edgar.iglesias@gmail.com \
    --cc=greg.bellows@linaro.org \
    --cc=john.williams@xilinx.com \
    --cc=peter.crosthwaite@xilinx.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=rob.herring@linaro.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).