From owner-dev-commits-src-all@freebsd.org Fri Jan 8 13:46:44 2021 Return-Path: Delivered-To: dev-commits-src-all@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id 688424CA2FC; Fri, 8 Jan 2021 13:46:44 +0000 (UTC) (envelope-from kevans@freebsd.org) Received: from smtp.freebsd.org (smtp.freebsd.org [96.47.72.83]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "smtp.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4DC4C82Kg2z3tyF; Fri, 8 Jan 2021 13:46:44 +0000 (UTC) (envelope-from kevans@freebsd.org) Received: from mail-qk1-f175.google.com (mail-qk1-f175.google.com [209.85.222.175]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "GTS CA 1O1" (verified OK)) (Authenticated sender: kevans) by smtp.freebsd.org (Postfix) with ESMTPSA id 3D803D54; Fri, 8 Jan 2021 13:46:44 +0000 (UTC) (envelope-from kevans@freebsd.org) Received: by mail-qk1-f175.google.com with SMTP id b64so8371932qkc.12; Fri, 08 Jan 2021 05:46:44 -0800 (PST) X-Gm-Message-State: AOAM531KWp/lbYkMqA4f14NBa35+mNBr1pczSR/fBBCpoltKLO9eeNHA aPCPSbc47Baqwdyy7gngpn46aBZWQc0fup7Z6h8= X-Google-Smtp-Source: ABdhPJzlQfKhxUQqHTr/UvrrZMPzP+eIvdHn1N6UNn4vCVxh1muFEDcArzo8M+OTjLF6xL4H6GOCqIvqPAuvrE2kA6E= X-Received: by 2002:a37:6713:: with SMTP id b19mr3727268qkc.493.1610113603775; Fri, 08 Jan 2021 05:46:43 -0800 (PST) MIME-Version: 1.0 References: <202101072320.107NKpMS063374@gitrepo.freebsd.org> <7a7fa0fb-51c2-1b14-6d37-2c400f69d7dd@kondratyev.su> <3EA011C9-2EEC-4693-B36D-69E74DF8473C@panasas.com> <20210108092638.x2kkvpeftey2wk62@ivaldir.net> In-Reply-To: <20210108092638.x2kkvpeftey2wk62@ivaldir.net> From: Kyle Evans Date: Fri, 8 Jan 2021 07:46:31 -0600 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: 92cf602e3809 - main - Copy strcasestr.c from libc to libkern. To: Baptiste Daroussin Cc: Vladimir Kondratyev , Ravi Pokala , Jessica Clarke , Vladimir Kondratyev , "src-committers@freebsd.org" , "dev-commits-src-all@freebsd.org" , "dev-commits-src-main@freebsd.org" Content-Type: text/plain; charset="UTF-8" X-BeenThere: dev-commits-src-all@freebsd.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Commit messages for all branches of the src repository List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 08 Jan 2021 13:46:44 -0000 On Fri, Jan 8, 2021 at 3:27 AM Baptiste Daroussin wrote: > > On Fri, Jan 08, 2021 at 02:49:31AM +0300, Vladimir Kondratyev wrote: > > On 08.01.2021 02:41, Ravi Pokala wrote: > > > -----Original Message----- > > > From: on behalf of Vladimir Kondratyev > > > Date: 2021-01-07, Thursday at 15:33 > > > To: Jessica Clarke , Vladimir Kondratyev > > > Cc: "src-committers@freebsd.org" , "dev-commits-src-all@freebsd.org" , "dev-commits-src-main@freebsd.org" > > > Subject: Re: git: 92cf602e3809 - main - Copy strcasestr.c from libc to libkern. > > > > > > On 08.01.2021 02:27, Jessica Clarke wrote: > > > > On 7 Jan 2021, at 23:20, Vladimir Kondratyev wrote: > > > >> +#include > > > >> +#include > > > >> +#include "xlocale_private.h" > > > >> + > > > >> +/* > > > >> + * Find the first occurrence of find in s, ignore case. > > > >> + */ > > > >> +char * > > > >> +strcasestr_l(const char *s, const char *find, locale_t locale) > > > > > > > > Locales in the kernel? This feels wrong. > > > > > > > > Jess > > > > > > > > > > strcasestr.c was copied from userland unmodified just to allow > > > "git log --follow" to work. Next commit removed locale support from this > > > file and connected it to build. > > > > > > Unfortunately, about two-dozen commit messages from you arrived at the same time, and some were out of order; without incrementing change numbers in the subject line, figuring out the actual ordering is non-trivial. :-p > > > > > > -Ravi (rpokala@) > > > > > > > I hope, commit hook will be improved some day. Pushing of two-dozen > > commits one by one is too boring. > > The history is in the right order and gitlog/cgit/anything that reads the log > show everything in the right order. > > mails are asynchronuous, expecting the mail to always arrive in a given order is > wrong ;) > > pushing all those commits at once was imho the right thing to do. > I've pointed this out a couple of times (with seemingly nothing coming from it), but the mails are being sent in reverse order. Fixing that would at least increase chances for some to receive them in order (and the archives show them perfectly in reverse order most of the time). Picking out the message-ids from the last series I sent, this is the correct order: 202101072237.107MbYaX002789@gitrepo.freebsd.org ("4832d2e8ae1d - main - du: tests: fix the H_flag test (primarily grep usage)") 202101072237.107MbYsM002771@gitrepo.freebsd.org ("3c5c39c7ad8f - main - du: tests: make H_flag tests more strict about output requirements") 202101072237.107MbYaX002789 -> 2789 was the last pid that assigned a message-id 202101072237.107MbYsM002771 -> 2771 was the last pid that assigned a message-id Given that this is clearly with kern.randompid=0 on the server, one can conclude that these were sent in the reverse order based on the pid sequence. If you pick a couple other series out, this holds true (at least for all the series I've looked at) Thanks, Kyle Evans