From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 3743 invoked by alias); 11 Jan 2009 03:38:20 -0000 Received: (qmail 3734 invoked by uid 22791); 11 Jan 2009 03:38:19 -0000 X-SWARE-Spam-Status: No, hits=-2.4 required=5.0 tests=AWL,BAYES_00 X-Spam-Check-By: sourceware.org Received: from rock.gnat.com (HELO rock.gnat.com) (205.232.38.15) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Sun, 11 Jan 2009 03:37:47 +0000 Received: from localhost (localhost.localdomain [127.0.0.1]) by filtered-rock.gnat.com (Postfix) with ESMTP id D84822A969C; Sat, 10 Jan 2009 22:37:45 -0500 (EST) Received: from rock.gnat.com ([127.0.0.1]) by localhost (rock.gnat.com [127.0.0.1]) (amavisd-new, port 10024) with LMTP id zwFAhumyyho2; Sat, 10 Jan 2009 22:37:45 -0500 (EST) Received: from joel.gnat.com (localhost.localdomain [127.0.0.1]) by rock.gnat.com (Postfix) with ESMTP id 03D0F2A969B; Sat, 10 Jan 2009 22:37:45 -0500 (EST) Received: by joel.gnat.com (Postfix, from userid 1000) id E33EDE7ACD; Sun, 11 Jan 2009 07:37:37 +0400 (RET) Date: Sun, 11 Jan 2009 03:38:00 -0000 From: Joel Brobecker To: Caz Yokoyama Cc: gdb@sourceware.org Subject: Re: symbolic debug of loadable modules with kgdb light Message-ID: <20090111033737.GL31296@adacore.com> References: <20090109041826.GD20227@adacore.com> <20090109181720.GD31296@adacore.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.4.2.2i Mailing-List: contact gdb-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-owner@sourceware.org X-SW-Source: 2009-01/txt/msg00050.txt.bz2 > About 25 lines come from > http://kgdb.cvs.sourceforge.net/viewvc/kgdb/gdb/. These are an > essential part. I could not delete it. From your comments, I have 3 > options in the order of most aggressive to conservative, 1) re-write > the code which come from kgdb, 2) ask the author of the code for his > permission and 3) don't submit the patch to gdb. I am wondering what > is right choice. It's hard to tell without knowing what the code is doing. If you can easily track down the author (all the authors, actually), then you can ask them to contribute this code. I think the safest for everyone would be if they have a copyright assignment on file with the FSF. On the other hand, if it's easy for you to rewrite the code in a different way, doing so allows you do remove a dependency on external factors. The other alternative is if we can determine that the change is not a "legally significant change", as our guidelines call it: http://www.gnu.org/prep/maintain/maintain.html#Legally-Significant > Regarding testsuite, > - gdb-6.8 > # of expected passes 11844 > # of unexpected failures 82 > # of unexpected successes 2 > # of expected failures 43 > # of known failures 39 > # of untested testcases 10 > # of unsupported tests 47 > - Modified gdb-6.8, > # of expected passes 11841 > # of unexpected failures 83 > # of unexpected successes 2 > # of expected failures 43 > # of known failures 39 > # of untested testcases 10 > # of unsupported tests 47 > So I believe the modification does not break anything. Looking at pure numbers can be misleading sometimes (which is why I diff the .sum files before and after applying my patch), but the numbers you published show that you have 3 less PASSes and one more FAIL. Sometimes, this does not indicate any regression, but often it does... Did you check were the difference come from? > All modifications are clear to me. I can write explanation for each > modifications and files. Also making a patch against cvd HEAD is not a > issue. Great! My recommendation is to first bring your patches up to date with CVS HEAD and, once done, to break them down into individual pieces. Then send each piece for review to the gdb-patches mailing list, explaining what the problem is and how you're solving it. The more information you provide, the easier it is for us to review the patches. -- Joel