linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: vijay.kilari@gmail.com (vijay.kilari at gmail.com)
To: linux-arm-kernel@lists.infradead.org
Subject: [RFC v2 PATCH 0/1] ARM64: KGDB: Add FPSIMD debug support
Date: Mon, 18 Nov 2013 17:36:56 +0530	[thread overview]
Message-ID: <1384776417-29076-1-git-send-email-vijay.kilari@gmail.com> (raw)

From: Vijaya Kumar K <Vijaya.Kumar@caviumnetworks.com>

Based on ARM64 KGDB support patches, KGDB support for
FPSIMD is added. Only debugging of FPSIMD kernel context
is supported.

This patch requires Ard's patches where in kernel support and
below patch for holding thread's fpsimd state.

http://permalink.gmane.org/gmane.linux.ports.arm.kernel/277228

So CONFIG_KERNEL_MODE_NEON should  be enabled.
with this, FPSIMD registers can be viewed or set from gdb tool.

Unlike CPU registers, the FPSIMD registers are not saved on
exception entry. With the known restriction that FPSIMD  should not be
touched in interrupt/exception context, in this patch the FPSIMD
registers are directly read/written on gdb tool request

Here, the FPSIMD registers are read and restored for every FPSIMD
register read and write by GDB tool. So this has impact on
gdb tool response which is neglible. Other architectures like
mips are also implemented similarly

v2 changes:
  - Added API to know thread fpsimd state by checking
    TIF_FOREIGN_FPSTATE flag. This is based on below patch

    http://permalink.gmane.org/gmane.linux.ports.arm.kernel/277228

  - Allow FPSIMD registers access only when FPSIMD is under use
    by current thread 

v1 changes: 
  - Initial patch

Tested on ARM64 simulator

Vijaya Kumar K (1):
  ARM64: KGDB: Add FP/SIMD debug support

 arch/arm64/include/asm/fpsimd.h |    1 +
 arch/arm64/kernel/fpsimd.c      |    5 ++
 arch/arm64/kernel/kgdb.c        |  105 +++++++++++++++++++++++++--------------
 3 files changed, 73 insertions(+), 38 deletions(-)

-- 
1.7.9.5

             reply	other threads:[~2013-11-18 12:06 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-18 12:06 vijay.kilari at gmail.com [this message]
2013-11-18 12:06 ` [RFC v2 PATCH 1/1] ARM64: KGDB: Add FP/SIMD debug support vijay.kilari at gmail.com
2013-11-25 18:52   ` Will Deacon
2013-11-25 19:03     ` Ard Biesheuvel
2013-11-26 11:57     ` Vijay Kilari
2013-11-29  8:53   ` Ard Biesheuvel
2013-11-29  9:20     ` Vijay Kilari
2013-11-29  9:27       ` Ard Biesheuvel
2013-11-29  9:57         ` Vijay Kilari
2014-02-28 11:08           ` Vijay Kilari
2014-02-28 17:36             ` Will Deacon
2014-03-02 13:08               ` Ard Biesheuvel
2014-03-03 11:43                 ` Will Deacon

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=1384776417-29076-1-git-send-email-vijay.kilari@gmail.com \
    --to=vijay.kilari@gmail.com \
    --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).