Mirror of the gdb mailing list
 help / color / mirror / Atom feed
From: Joel Brobecker <brobecker@adacore.com>
To: Jim Wilson <jimw@sifive.com>
Cc: gdb@sourceware.org
Subject: Re: contributing the RISC-V linux native port
Date: Tue, 07 Aug 2018 13:01:00 -0000	[thread overview]
Message-ID: <20180807130122.GI2432@adacore.com> (raw)
In-Reply-To: <CAFyWVaauzsTVmcjGuPXPPzJ4AuLbyaE4oc5MLTG0wq1DEXZmmw@mail.gmail.com>

> There are some obvious things broken.  32-bit float values in 64-bit
> FP registers aren't printed properly because they are NaN-boxed, and
> gdb doesn't know about that yet.  I don't have support for signal
> handler frames yet.  I only have integer register support for core
> files.  I haven't tried to implement watchpoints yet.  There are a lot
> of thread failures, 358 in gdb.thread, and apparently some in gdb.mi
> also, but there is a glibc patch from Andreas Schwab that I hope fixes
> some of them.  Most of the rest of the failures I haven't categorized
> yet.
> 
> Anyways, I'm wondering how much more work I need to do out of tree
> before I can try submitting my work.

I don't think we have a specific guideline for this; my opinion is
that you don't have to wait for something to be perfect before you
can submit it for inclusion. What you want to ask yourself is,
assuming your patch is improving things already (and therefore
being helpful to you and probably others), does it have a cost
elsewhere? For instance, will it make harder to fix something later,
or does it break something else, etc? If, as is often the case,
you find that the code is self contained and doesn't hurt anything
else, why not submit it already, so others can benefit from it?
As long as the code passes the coding standards, I don't see how
it could hurt.

> Also, I'm wondering about policy on branches in the gdb tree.  Maybe
> it would make sense to move my work into a branch in the FSF gdb tree.
> I have my own github tree at the moment where I am doing work which is
> working fine for me, but it isn't easy for other people to find it.

Users can have personal branches under "users/<username>/<branch-name>".

-- 
Joel


  parent reply	other threads:[~2018-08-07 13:01 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-08-07  2:22 Jim Wilson
2018-08-07  6:27 ` Palmer Dabbelt
2018-08-07 13:01 ` Joel Brobecker [this message]
2018-08-07 16:14   ` Jim Wilson
2018-08-07 16:27     ` Andreas Schwab

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=20180807130122.GI2432@adacore.com \
    --to=brobecker@adacore.com \
    --cc=gdb@sourceware.org \
    --cc=jimw@sifive.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox