From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 93016C433E2 for ; Mon, 20 Jul 2020 16:53:52 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 5C36F206F2 for ; Mon, 20 Jul 2020 16:53:52 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="0FXtDCvL" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 5C36F206F2 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=arm.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=/Cy6eYH2PRouz7CtFhBLChrYo7P0QlRyVOCTdha7DAI=; b=0FXtDCvLRvPlWHtY1SUPDdLfS nKYI62FPtJ8nbnCoDQy4yD7HiPm3mVw4PSVlRqpA4NJnZw4rNsx/8xNQyh6zUnk76cj4ozbSjAsB3 oPSeAYK3gN/ycYk1O+rY9i3h9LnWDNfvBf5Gu9WfByQl1R99PartG929T5KPM24Q2oJ4/nwnBwCZB WRMAG5sO5Rd/8idIQMtyVcwhNGcikXxKSUJfo8vcr70pnt2G3yW4EN0BTSEDYm9ZDITHTM39J8Vcs 6KxMiKu54SW7JkPK4lOzkWnZaEJBaOXwi/RIv6S8HgQSv13J6hdIWb2NCHEhC6bnnJHyL3VRWF4Wp hAGulew8w==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1jxZ1X-0007rd-B6; Mon, 20 Jul 2020 16:52:19 +0000 Received: from foss.arm.com ([217.140.110.172]) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1jxZ1U-0007r6-IY for linux-arm-kernel@lists.infradead.org; Mon, 20 Jul 2020 16:52:17 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id CD674106F; Mon, 20 Jul 2020 09:52:10 -0700 (PDT) Received: from arm.com (usa-sjc-imap-foss1.foss.arm.com [10.121.207.14]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id B27703F66E; Mon, 20 Jul 2020 09:52:09 -0700 (PDT) Date: Mon, 20 Jul 2020 17:52:07 +0100 From: Dave Martin To: "Michael Kerrisk (man-pages)" Subject: Re: [PATCH v3 0/2] prctl.2 man page updates for Linux 5.6 Message-ID: <20200720165205.GI30452@arm.com> References: <1593020162-9365-1-git-send-email-Dave.Martin@arm.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200720_125216_685488_8B43367B X-CRM114-Status: GOOD ( 22.22 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-arch@vger.kernel.org, linux-man@vger.kernel.org, Catalin Marinas , Mark Rutland , Vincenzo Frascino , Will Deacon , linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, Jun 29, 2020 at 01:52:24PM +0200, Michael Kerrisk (man-pages) wrote: > Hi Dave, > > On 6/24/20 7:36 PM, Dave Martin wrote: > > A bunch of updates to the prctl(2) man page to fill in missing > > prctls (mostly) up to Linux 5.6 (along with a few other tweaks and > > fixes). > > > > Patches from the v2 series [1] that have been applied or rejected > > already have been dropped. > > > > All that remain here now are the SVE and tagged address ABI controls > > for arm64. > > > > > > > > [1] https://lore.kernel.org/linux-man/1590614258-24728-1-git-send-email-Dave.Martin@arm.com/ > > > > > > Dave Martin (2): > > prctl.2: Add SVE prctls (arm64) > > prctl.2: Add tagged address ABI control prctls (arm64) > > > > man2/prctl.2 | 331 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ > > 1 file changed, 331 insertions(+) > Thanks. I've pushed these changes to master now. Thanks -- btw I finally got around to reviewing master, and noted a few editorial changes that man-pages(7) does not make any statement about: "arg1, arg2, and arg3" Do you strictly prefer the command before "and" here? Conventionally, the final comma would typically be omitted in prose, except where the list members are complex enough that the command is required to assist parsing. However, lists of formal arguments are not quite vanilla prose. "Providing that" -> "Provided that" Any particular rationale here? "error EFOO" -> "the error EFOO" Is this a rule, in general? .IP \(bu 2 I assumed that specifying an explicit indentation amount would be fragile. Going with the default behaviour also tends to result in a more consistent appearance. Do you have any recommandations in this area? Do you have rules about the order to use bullet symbols? I tend to avoid \(bu if possible, since while it's "correct", nroff can render it nastily as an unadorned letter "o" (e.g., with -Tascii or LC_CTYPE=C). This is particlarly annoying if the indent is <= 2, since then the "o" tends to be visually swallowed by the following text (i.e., to a casual glance it looks like a word, particlarly if the following text is not capitalised). Perhaps this is a bad glyph substitution decision in nroff rather than something that should be fixed in the man-pages source, but the man-pages source may be easier to fix... There is already inconsistency here: there are may top-level lists using ".IP *" in prctl.2, and plenty of places where the default indentation is used. Should any of these be written up in man-pages(7), or is there a checker than can detect them? I wan't to minimise the amount of tweaking you have to do when merging patches. Cheers ---Dave _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel