From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from simark.ca by simark.ca with LMTP id +0TQGXBr3GKfURkAWB0awg (envelope-from ) for ; Sat, 23 Jul 2022 17:43:12 -0400 Received: by simark.ca (Postfix, from userid 112) id 337D01E5EA; Sat, 23 Jul 2022 17:43:12 -0400 (EDT) Authentication-Results: simark.ca; dkim=pass (1024-bit key; secure) header.d=sourceware.org header.i=@sourceware.org header.a=rsa-sha256 header.s=default header.b=HBtd0KiD; dkim-atps=neutral X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on simark.ca X-Spam-Level: X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,RDNS_DYNAMIC,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 Received: from sourceware.org (ip-8-43-85-97.sourceware.org [8.43.85.97]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by simark.ca (Postfix) with ESMTPS id D26F91E222 for ; Sat, 23 Jul 2022 17:43:11 -0400 (EDT) Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 12AC93858027 for ; Sat, 23 Jul 2022 21:43:11 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 12AC93858027 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1658612591; bh=o1Dv87VquED39A/Ml5blk0/qAP2piYtb2RGavHVVNkw=; h=Date:To:Subject:In-Reply-To:References:List-Id:List-Unsubscribe: List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:Cc: From; b=HBtd0KiDKQjuZJXgXPNMgyRpJNignEGxwhRwvY72bWeTs19ETtVH6ndnrK/jfp+EX 9U+6JGXVpz9jAMu966qWjvxOKilKlfRWfPUvfpIfo4KL6amdFN/Rn5zdsju4XTLQy/ Q7GFmmpguQn1ibM3Egnmze3ZOJ45gi3OdnzoGIFc= Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by sourceware.org (Postfix) with ESMTPS id 89FAC385840A for ; Sat, 23 Jul 2022 21:42:51 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 89FAC385840A Received: from mimecast-mx02.redhat.com (mx3-rdu2.redhat.com [66.187.233.73]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-447-ilfjJQWRM9WRWWJCtiNS6w-1; Sat, 23 Jul 2022 17:42:47 -0400 X-MC-Unique: ilfjJQWRM9WRWWJCtiNS6w-1 Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.rdu2.redhat.com [10.11.54.5]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 357DC3806706; Sat, 23 Jul 2022 21:42:47 +0000 (UTC) Received: from f35-zws-1 (unknown [10.2.16.60]) by smtp.corp.redhat.com (Postfix) with ESMTPS id D44A290A04; Sat, 23 Jul 2022 21:42:46 +0000 (UTC) Date: Sat, 23 Jul 2022 14:42:45 -0700 To: Jiangshuai Li via Gdb-patches Subject: Re: [PATCH] gdb/csky support .reg2 for kernel 4.x and later Message-ID: <20220723144245.07768aa7@f35-zws-1> In-Reply-To: <20220722055446.2836-1-jiangshuai_li@linux.alibaba.com> References: <20220722055446.2836-1-jiangshuai_li@linux.alibaba.com> Organization: Red Hat MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.11.54.5 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit 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: , From: Kevin Buettner via Gdb-patches Reply-To: Kevin Buettner Cc: Jiangshuai Li Errors-To: gdb-patches-bounces+public-inbox=simark.ca@sourceware.org Sender: "Gdb-patches" On Fri, 22 Jul 2022 13:54:46 +0800 Jiangshuai Li via Gdb-patches wrote: > static void > csky_supply_gregset (const struct regset *regset, > - struct regcache *regcache, int regnum, > - const void *regs, size_t len) > + struct regcache *regcache, int regnum, > + const void *regs, size_t len) > { > int i, gregset_num; > const gdb_byte *gregs = (const gdb_byte *) regs ; > @@ -77,8 +79,8 @@ csky_supply_gregset (const struct regset *regset, > for (i = 0; i < gregset_num; i++) > { > if ((regnum == csky_gregset_offset[i] || regnum == -1) > - && csky_gregset_offset[i] != -1) > - regcache->raw_supply (csky_gregset_offset[i], gregs + 4 * i); > + && csky_gregset_offset[i] != -1) > + regcache->raw_supply (csky_gregset_offset[i], gregs + 4 * i); > } > } > So... in the above two hunks of this patch, plus in many other places in the patch, it seems that the only change was to convert tabs to spaces. Could you please resend your patch without these whitespace changes? Not only will it be easier to review, but I think we still prefer the use of tabs in place of leading spaces in GDB's source code. Even if such a change were desirable, we'd prefer to see a separate patch for whitespace changes. Thanks, Kevin