From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 6654 invoked by alias); 15 Dec 2014 14:24:26 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 6642 invoked by uid 89); 15 Dec 2014 14:24:24 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.6 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 X-HELO: mail-pa0-f53.google.com Received: from mail-pa0-f53.google.com (HELO mail-pa0-f53.google.com) (209.85.220.53) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-SHA encrypted) ESMTPS; Mon, 15 Dec 2014 14:24:22 +0000 Received: by mail-pa0-f53.google.com with SMTP id kq14so11748193pab.26 for ; Mon, 15 Dec 2014 06:24:21 -0800 (PST) X-Received: by 10.66.119.193 with SMTP id kw1mr22858388pab.45.1418653460752; Mon, 15 Dec 2014 06:24:20 -0800 (PST) Received: from [192.168.1.103] ([223.72.65.33]) by mx.google.com with ESMTPSA id ug6sm9407122pbc.72.2014.12.15.06.24.17 for (version=TLSv1 cipher=RC4-SHA bits=128/128); Mon, 15 Dec 2014 06:24:19 -0800 (PST) Message-ID: <548EEF0E.3000307@gmail.com> Date: Mon, 15 Dec 2014 14:24:00 -0000 From: Chen Gang User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130625 Thunderbird/17.0.7 MIME-Version: 1.0 To: Joel Brobecker CC: Andreas Schwab , gdb-patches@sourceware.org Subject: Re: [PATCH v4] gdb/hppa-tdep.c: Fix logical working flow issues and check additional store instructions References: <548D93E4.2000405@gmail.com> <20141214193513.GT5457@adacore.com> <548E65B5.7040606@gmail.com> <20141215124918.GV5457@adacore.com> In-Reply-To: <20141215124918.GV5457@adacore.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit X-SW-Source: 2014-12/txt/msg00389.txt.bz2 On 12/15/2014 08:49 PM, Joel Brobecker wrote: >> Excuse me, I have no related test environments for parisc either. Maybe >> I can try to construct the related virtual environments for it, but I am >> still not quite sure whether it is enough. >> >> Can we change to another way for it: >> >> - try to keep the original code no touch. >> >> - only fix the related typo issues according to the related documents, >> do not add new features (e.g. do not check stby, stbdy, stwa, stda, >> only give the related comments for them). >> >> - let it pass compiling. >> >> Or, can we find related members in our gdb mailing list which has parisc >> environments? > > I think finding people who can test your change would be ideal, > but is not required. I was just making it clear what my review > could bring, or rather not bring. I think it's fine to increase > the scope of the routine as you've been doing, as I think the risk > of disturbing currently supported instructions is small. > OK, thanks. > I am curious - and you do not have to reply if you do not want to -, > how did you get involved in this change? If you have no testing > environment, it probably means you're not using the code. What > made you decide to try to fix it? > When I compiling binutils, and find the compiler warning, then try to fix it with other members. I have no environments, not familiar with parisc, no enough time resources ... but I should still try my best. Originally, I worked for coredump analysing (kdb for Linux kernel, gdb for user mode C++ program) -- which have no test environments, either not familiar many things, but the end time point is sensitive ... At present, after I really touch the open source in details, I know: - Most of persons and companies (include me) have already get benefit from open source, especially fundamental software (e.g. binutils, gdb), but most of them do not contribute back to open source. - But open source still continue, and still benefit to most of persons and companies. - What I have done is just only try to contribute back to open source. Thanks. -- Chen Gang Open share and attitude like air water and life which God blessed