From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from simark.ca (simark.ca [158.69.221.121]) by sourceware.org (Postfix) with ESMTPS id E5DF23857C52 for ; Thu, 24 Sep 2020 15:42:22 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org E5DF23857C52 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=simark.ca Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=simark@simark.ca Received: from [10.0.0.11] (173-246-6-90.qc.cable.ebox.net [173.246.6.90]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by simark.ca (Postfix) with ESMTPSA id 857D31E509; Thu, 24 Sep 2020 11:42:22 -0400 (EDT) Subject: Re: [PATCH] [gdb] Add Z80 CPU basic support To: Sergey Belyashov Cc: gdb-patches@sourceware.org References: <28250680-67b9-7551-72fb-bb5826eeecbd@simark.ca> <20200924082621.19370-1-sergey.belyashov@gmail.com> <34795027-0954-e286-dc1a-1f810a8ff6e1@simark.ca> From: Simon Marchi Message-ID: <66e8ab57-0603-56d8-49cc-c45ce2ba24ef@simark.ca> Date: Thu, 24 Sep 2020 11:42:22 -0400 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.12.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: fr Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-7.4 required=5.0 tests=BAYES_00, BODY_8BITS, GIT_PATCH_0, KAM_DMARC_STATUS, NICE_REPLY_A, SPF_HELO_PASS, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 24 Sep 2020 15:42:24 -0000 On 2020-09-24 11:21 a.m., Sergey Belyashov wrote: > чт, 24 сент. 2020 г. в 17:08, Simon Marchi : > This code is written by me from scratch. Some code parts are copied from > other GDB source files. Thanks. >> - Did you manage to run parts of the GDB testsuite against this? > > GDB testsuit cannot be used, because there are no GCC port to Z80. GCC is not required, it is possible to write a board file that specifies another compiler. It would be good to have a board file that allows a developer to test a change against Z80. A bit like testsuite/boards/simavr.exp for AVR. > >> - If I want more information about the Z80 architecture, such as the ISA >> details, ABI, etc, where do I look? It would also be a good idea to >> add this to the commit message. > > It can be downloaded on Zilog offical site. Thanks, I see you provided direct links in v3. >> We want to avoid people dropping code and then leaving a maintenance >> burden to others. Ideally, each architecture should have a maintainer >> (listed in the MAINTAINERS file) responsible for reviewing patches about >> that architecture, and to whom we can reach out for questions about that >> architecture. Would you be ready to accept such a role? > > Yes, I'm ready. Thanks. >> Shouldn't there be two different target descriptions for Z80 and eZ80? > > I have not completely supported the eZ80 in ADL mode, so I do not provide > a description for it. In legacy Z80 mode eZ80 has the same registers and > flags as Z80/Z180. So, do we need to mention eZ80 at all? Can't we just talk about Z80 and never talk about eZ80? > >>> diff --git a/gdb/stubs/z80-stub.c b/gdb/stubs/z80-stub.c >>> new file mode 100644 >>> index 0000000000..36316b95ba >>> --- /dev/null >>> +++ b/gdb/stubs/z80-stub.c >> >> What is this file for? How do I use it? > > This file is used as an embedded debugger monitor. Currently GDB is > tested with it. Ok, it would be good to give an example of how to use it, otherwise it's pretty much dead code (like all other files in this directories are). Simon