From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm1-x32c.google.com (mail-wm1-x32c.google.com [IPv6:2a00:1450:4864:20::32c]) by sourceware.org (Postfix) with ESMTPS id 6CA9B3858C27 for ; Mon, 9 Jan 2023 18:03:28 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 6CA9B3858C27 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=harmstone.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=gmail.com Received: by mail-wm1-x32c.google.com with SMTP id k26-20020a05600c1c9a00b003d972646a7dso9747388wms.5 for ; Mon, 09 Jan 2023 10:03:28 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :sender:from:to:cc:subject:date:message-id:reply-to; bh=/XY5DOZWrT9eUB5o+jHCVIwWFt5PzpGbhTGPnXP/SPk=; b=ZF7O7j9CWKlNIwuf+M44QvHjV1uu3/KVDPKVZeSQfApkO3yIfUIB3d1SDdYL12Ocvk Og0TR8pYddieQ03ZO0ULujOTAkzFrN5KGDWtd41gG4qYOxt25HVsxTkvGKMhEw/oXCAN IXocRzwYwoJ0jEf10SQ0+8G5Lo38VFMe1FfkbSAjSLdKF5+mVZBsgll/9LwPQKKkX3Y2 Fo710BQEAJ4nCQKZ91K5isTKLcqfKIIS15TBNcJp3kVqHQMsNZxq+fvH53qbXcz+XGWp sPQx9J6/vovyAR7o56P7LJ3a+qaC+K3RbWo36GJLAnuaIP2/i433Ja8uYVstiZDuuPi7 OUtQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :sender:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=/XY5DOZWrT9eUB5o+jHCVIwWFt5PzpGbhTGPnXP/SPk=; b=LbgKg4iiOvDDCC9u5Jlr75sCnZRtkUCUegXy4/W3NLyi/1DulX9o4BJ0XVSZ5joREU mwvec5H4b3MUwNEBxUaPj90XyDedwytK+82y3IqAQz8SwxMkEHnYgoClDJHXeA8+wz9I DUi01vUGJMVO7Scmn/gSTWkOX3h1Qfvp0aUNJ2Qnq/yO1shdjlxQWyGAI+C2Vo/Ynf42 tpdMaD3D2j4VzdYx93mZPEd6d4GHjFlx3RgnMQ9trUHSSRD/lvq79wZjZYVslaOaAD4c 18KVjaHDZeppHrerkpXyKiDvQoT4jN1ktB3Ir1AB/a2gS5S7cLPk+onoFqX/3FrjL8+I WDSQ== X-Gm-Message-State: AFqh2kqCR/0G0hnOa0DT/d6zUTLFGsCyxeL2x9T3MAVkutVnOAPF5B1C C52BYDl0K4g+DL7qX5QbdGY= X-Google-Smtp-Source: AMrXdXvRz7e51MJ8xEmqBsbZwismFt+cBYoaJz6ANmT7sDFGKrJ1l6XHsUskUI3XD7sszHmUwlGfcA== X-Received: by 2002:a05:600c:4e13:b0:3c6:e63d:fcfc with SMTP id b19-20020a05600c4e1300b003c6e63dfcfcmr46950538wmq.35.1673287407082; Mon, 09 Jan 2023 10:03:27 -0800 (PST) Received: from ?IPV6:2a02:8010:64ea:0:8eb8:7eff:fe53:9d5f? ([2a02:8010:64ea:0:8eb8:7eff:fe53:9d5f]) by smtp.googlemail.com with ESMTPSA id i3-20020a1c5403000000b003d35c09d4b9sm16656393wmb.40.2023.01.09.10.03.25 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 09 Jan 2023 10:03:26 -0800 (PST) Sender: Mark Harmstone Message-ID: <9af6c55d-c926-bcde-35c3-cb4a92aa3067@harmstone.com> Date: Mon, 9 Jan 2023 18:03:24 +0000 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.5.1 Subject: Re: [PATCH 1/7] Fix size of external_reloc for pe-aarch64 To: Christophe Lyon , Jan Beulich Cc: binutils@sourceware.org, Tamar.Christina@arm.com, nickc@redhat.com, pinskia@gmail.com, Richard.Earnshaw@foss.arm.com, nightstrike@gmail.com, wej22007@outlook.com, zac.walker@linaro.org References: <20230106012509.7918-1-mark@harmstone.com> <42d1673b-bca4-a4ce-daf2-ae4c24ca923e@arm.com> <3fb4611d-46ee-fe35-748c-238b4d71fff8@suse.com> <5cacce21-6eb9-fd6e-9497-006b076fb171@arm.com> Content-Language: en-US From: Mark Harmstone In-Reply-To: <5cacce21-6eb9-fd6e-9497-006b076fb171@arm.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-4.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_EF,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM,HEADER_FROM_DIFFERENT_DOMAINS,KAM_NUMSUBJECT,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS,TXREP autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: On 9/1/23 09:22, Christophe Lyon wrote: > > > On 1/9/23 09:11, Jan Beulich wrote: >> On 06.01.2023 18:51, Mark Harmstone wrote: >>> On 6/1/23 09:47, Christophe Lyon wrote: >>>> Hi! >>>> >>>> I am not a maintainer, but would you mind adding proper commit messages describing what each patch does (or intends to)? >>>> >>>> Thanks, >>>> >>>> Christophe >>> >>> Hi Christophe, >>> >>> This is a resubmission of a patch set from a few days ago, because of a change that Tamar wanted. If you're interested in the discussion behind each patch, it's available in the mailing list archives. >> >> I'm afraid pointing to list archives for explanations on patches isn't a good >> approach. Once committed, such links will not be easy to (re-)establish. >> Other projects are quite a bit more demanding towards the content of commit >> messages, but I guess some minimal level of explanation wants to be in the >> average binutils patch as well. >> > > That's what I meant I think, thanks for rephrasing :-) > > I can't find guidelines on how to contribute patches for binutils, but they are similar to GCC's and GDB's. If you just run 'git log' in a binutils clone, you'll see what we mean: in addition to short a summary (title), commit messages include a description of what the commit does and why this is the right change. > > It's great if all your changes are obvious for Nick, but they are not for others like me ;-) > > For instance this patch 1/7 only says "Fix size of external_reloc for pe-aarch64", so why is the removal of SWAP_IN_RELOC_OFFSET/SWAP_OUT_RELOC_OFFSET and r_offset needed? > > I did check the list archives, so if I'm not mistaken this is third iteration of this patch series? (I saw 1/5 and 1/8 in December, then this one in January). The first iteration had an introduction message which led to a discussion with Jan and Tamar, and I think an updated version of that message would help here, when others will have to try to understand these patches in whatever future time ;-) > > Also as Jan mentioned on your testsuite patches, can you describe why we have to skip so many of them? Jan seems to think that they could be adjusted to cope with both formats intead. > > Thanks, > > Christophe > >> Jan You both raise good points. Sorry, I still find mailing list culture very confusing, compared to something like GitHub... it's not at all clear what you're expected to do, who has what authority, etc. I'm not at all certain that I was even expected to resubmit the patches the third time, given it was more or less a trivial rebase. Jan, I suggest that I push patches 1 and 4-7 with the original messages, and save the two test patches for a later date. Would that be acceptable? Patches 2 and 3 probably should have been submitted separately anyway, as the later patches don't depend on them. Mark