Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: "Maciej W. Rozycki" <macro@orcam.me.uk>
To: Milica Matic <milica.matic@htecgroup.com>
Cc: Djordje Todorovic <Djordje.Todorovic@htecgroup.com>,
	 Milos Kalicanin <milos.kalicanin@htecgroup.com>,
	 Simon Marchi <simark@simark.ca>, Chao-ying Fu <cfu@wavecomp.com>,
	 Andrew Burgess <aburgess@redhat.com>,
	Kevin Buettner <kevinb@redhat.com>,
	 gdb-patches@sourceware.org
Subject: Re: [PATCH v12 2/2] gdb: mips: Add MIPSR6 support
Date: Tue, 25 Feb 2025 22:35:38 +0000 (GMT)	[thread overview]
Message-ID: <alpine.DEB.2.21.2502231646180.65342@angie.orcam.me.uk> (raw)
In-Reply-To: <20250219130125.356900-4-milica.matic@htecgroup.com>

On Wed, 19 Feb 2025, Milica Matic wrote:

> Introduce new instruction encodings from Release 6 of the MIPS
> architecture [1]. Support breakpoints and single stepping with
> compact branches, forbidden slots, new branch instruction and
> new atomic load-store instruction encodings.

 I have now skimmed over your change and obviously branch decoding is the 
huge update and the most problematic part.  At high level I find new code 
a little bit hairy and hard to follow in some places, and will go over it 
in details double-checking all is in order, and maybe trying to untangle 
it a little.  I won't be able to do it before this coming weekend though 
at which point I'm going on holiday for a fortnight.  Please be a little 
patient meanwhile.

 Also you may have seen my response to an earlier version where I reported 
how I tried to do verification against my 74K Malta only to see it run 
unreliably.  Sadly the 5Kc core card I previously used broke a couple of 
years ago and the replacement newer module does not seem to run equally 
well (also it's 32-bit only while the 5Kc was 64-bit).

 I suspect it just wasn't debugged as thoroughly as the older one, there
are issues with PCI handling both in Linux and even in the YAMON firmware, 
and I think this instability in running GDB verification can be attributed 
to cache coherency maintenance issues in Linux, which can be hairy.  I may 
not have the resources available at hand though to get it sorted quickly.

 It doesn't appear to me that the 74K core card is at fault as I have two 
of them, slightly different and carrying a 74Kc and a 74Kf each, so if it 
was owing to a faulty card, then I imagine they wouldn't behave exactly 
the same.  Also glibc and GCC verification seems more reproducible.  The 
old 5Kc core card was rock-solid, I used it for 20 years until it failed.

 So instead I made an attempt the other day to use alternate test machines 
I have in my remote lab, and that also hit obstacles.  One of the systems 
refused to respond on the serial console.  I've seen it before and it is 
likely owing to its lithium coin cell powering the configuration memory 
having run out of energy, making the system revert to the video console, 
which I have no remote access to.  So I cannot make use of that machine 
for the time being, until the cell has been replaced and configuration 
restored.

 The other system booted just fine and completed reference GDB testing.  
On second iteration however it broke with uncorrectable multiple-bit ECC 
errors reported by DRAM making the system unusable.  The errors persisted 
after a reboot and firmware testing pointed at just one problematic DRAM 
module (out of eight installed total).  I have seen this before as well 
and cleaning contacts and/or reseating the module should clear the issue.  
At worst the module can be removed, reducing system RAM from 256MiB down 
to 224MiB, which should still be fine for the purpose of this testing.

 I have a maintenance visit scheduled to my lab tomorrow, so I'm going to 
address these issues.  I'll report back and, fingers crossed, I should at 
least be able to verify the lack of regressions from your change for older 
systems, possibly by the end of this week.

 I will appreciate your understanding of my constraints.

  Maciej

  reply	other threads:[~2025-02-25 22:36 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-02-19 13:01 [PATCH 0/2] " Milica Matic
2025-02-19 13:01 ` [PATCH v12 " Milica Matic
2025-02-19 13:01 ` [PATCH v12 1/2] Apply coding guidelines Milica Matic
2025-02-19 23:21   ` Kevin Buettner
2025-02-23 16:40     ` Maciej W. Rozycki
2025-02-19 13:01 ` [PATCH v12 2/2] gdb: mips: Add MIPSR6 support Milica Matic
2025-02-25 22:35   ` Maciej W. Rozycki [this message]
2025-03-02  0:17     ` Maciej W. Rozycki
2025-03-28 17:09       ` Maciej W. Rozycki

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=alpine.DEB.2.21.2502231646180.65342@angie.orcam.me.uk \
    --to=macro@orcam.me.uk \
    --cc=Djordje.Todorovic@htecgroup.com \
    --cc=aburgess@redhat.com \
    --cc=cfu@wavecomp.com \
    --cc=gdb-patches@sourceware.org \
    --cc=kevinb@redhat.com \
    --cc=milica.matic@htecgroup.com \
    --cc=milos.kalicanin@htecgroup.com \
    --cc=simark@simark.ca \
    /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