From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 1839 invoked by alias); 21 Oct 2014 05:22:41 -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 1830 invoked by uid 89); 21 Oct 2014 05:22:40 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.2 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 X-HELO: mail-qa0-f44.google.com Received: from mail-qa0-f44.google.com (HELO mail-qa0-f44.google.com) (209.85.216.44) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-SHA encrypted) ESMTPS; Tue, 21 Oct 2014 05:22:38 +0000 Received: by mail-qa0-f44.google.com with SMTP id x12so320060qac.31 for ; Mon, 20 Oct 2014 22:22:36 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:date :message-id:subject:from:to:cc:content-type; bh=cVB+olRRYC6UW5Q+lr3j/N41fpUAmU6OvmZaACbCmac=; b=NDrSehH4a7SALSc8vuY68iU9xkFNrG/LdMPepsMTfSOSUPPF8HYQBj+nwSN1rymYTq iGRgwIDBvkvwiumsEBfoz7YoeycTGJlg5keND4LZE0ykCJBj0nzfeVPudnQdHQHBnLRy je/wqk0etk42E5Q/DWf+plicHnTF+ULYZwI6Hz2z147EQ6QNjcOqe74bt+JIqxdrjALN JeGQcshVRPBOesN9IrVOw+oPQV8KnUk4SZaDXvgXX3ROcLzVb2Ei3PGxN9IZWnm7Rpid wbCVdMwgpFzfHQBjavSaZNxrBe2H4rVALGYO/uTwJRmxoTGoJrviO8ZZvx4/7Gdhq+ON SBIA== X-Gm-Message-State: ALoCoQnnSnbrmzKazAwwVv0+VX3aJs0a22kINst+APm56RrUk4+3rKt8AuLebUuazbJVESlzXc2A MIME-Version: 1.0 X-Received: by 10.224.166.6 with SMTP id k6mr43066617qay.57.1413868956742; Mon, 20 Oct 2014 22:22:36 -0700 (PDT) Received: by 10.229.93.203 with HTTP; Mon, 20 Oct 2014 22:22:36 -0700 (PDT) In-Reply-To: References: <1413853021-4393-1-git-send-email-victor.kamensky@linaro.org> Date: Tue, 21 Oct 2014 05:22:00 -0000 Message-ID: Subject: Re: [PATCH 0/5] arm: set of big endian related fixes for armeb (v7) From: Victor Kamensky To: Andrew Pinski Cc: "gdb-patches@sourceware.org" Content-Type: text/plain; charset=UTF-8 X-SW-Source: 2014-10/txt/msg00533.txt.bz2 On 20 October 2014 18:12, Andrew Pinski wrote: > On Mon, Oct 20, 2014 at 5:56 PM, Victor Kamensky > wrote: >> Hi Folks, >> >> Please find five patches following this cover letter that address several >> big endian related issues for ARM V7 target. Problems were discovered by >> running gdb/testsuite on armeb (big endian) target rootfs/kernel and >> comparing results from the same source against arm (little endian). >> >> Note there are several failures remain that are not addressed by this >> series. I.e valgrind related test cases all failed, because valgrind does >> not support armeb target. There are other failures in a bit obscure >> places like gdb.dwarf2/implptrpiece.exp, gdb.python/py-value-cc.exp, >> but over all number of passes/failures significantly improved in armeb >> case. >> >> Please note it is my first attempt to contribute into gdb. If I missed >> something please guide me how I can correct that. > > Yes bfd patches need to be also sent to binutils@. Thanks, Andrew. I posted first patch of this series on suggested binutils mailing list [1]. Other four patches in gdb directory. - Victor [1] https://sourceware.org/ml/binutils/2014-10/msg00175.html > Thanks, > Andrew Pinski > > >> >> Thanks, >> Victor >> >> Victor Kamensky (5): >> ARM: plt_size functions need to read instructions in right byte order >> ARM: extract_arm_insn function need to read instrs correctly in be8 >> case >> ARM: arm_breakpoint should be little endian form in case for arm BE8 >> ARM: read_pieced_value do big endian processing only in case of valid >> gdb_regnum >> ARM: asm-source.exp link options in case of armv7b target >> >> bfd/ChangeLog | 9 +++++++ >> bfd/elf32-arm.c | 48 +++++++++++++++++++++++++++++++++--- >> gdb/ChangeLog | 16 ++++++++++++ >> gdb/arm-linux-tdep.c | 2 +- >> gdb/arm-tdep.c | 2 +- >> gdb/dwarf2loc.c | 30 +++++++++++----------- >> gdb/testsuite/ChangeLog | 4 +++ >> gdb/testsuite/gdb.asm/asm-source.exp | 4 +++ >> 8 files changed, 94 insertions(+), 21 deletions(-) >> >> -- >> 1.8.1.4 >>