From owner-svn-src-head@freebsd.org Tue Sep 22 21:40:25 2020 Return-Path: Delivered-To: svn-src-head@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 E184D3E00CD for ; Tue, 22 Sep 2020 21:40:25 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: from mail-lf1-x129.google.com (mail-lf1-x129.google.com [IPv6:2a00:1450:4864:20::129]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "smtp.gmail.com", Issuer "GTS CA 1O1" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4BwvqX5hBKz3SCd for ; Tue, 22 Sep 2020 21:40:24 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: by mail-lf1-x129.google.com with SMTP id b22so19659345lfs.13 for ; Tue, 22 Sep 2020 14:40:24 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bsdimp-com.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=wplx7mOV8skWN9Q4rkxaEdhI1qqZEsQinUksW6XZv3Y=; b=E3uejyZ8YLf7VQVI6FphdOOAqqcCCLomy228s3HuKh9smXpcN5mPfktg8aD0Kc8lkf LDsZO+HeRooHAcGsGVvboCGxiGTBi80Yq7lCbCXRfHsPuYrjO252MqEDIvvHkxk+66Oy yPE5xRvpoxdGkUSGk/wfF8ba4xhwqmZkXURPoNM1Hn0KizioOJOW4r2ENJaRivzSVJm1 8Kzh1onWqAmJVqcDYgjMaSuCEhwQ2zMjjhLOnLoP0D0UqgXaQ1bcDFO+ap9oDSke8rfp rHqV4D1x7a+rUHYJBStkz8rg97v+50TBrcjEpRWNFnqh6cD6VPjlKiSAYb+5h2vgwB+V TdcQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=wplx7mOV8skWN9Q4rkxaEdhI1qqZEsQinUksW6XZv3Y=; b=pofDkRLg+0cZyLOkI+hV33CtS7iv2Cr8+7Su1szaG4iZbwy7bGRsjMIiEXUWFrnrrF HYrlB3ECOyR6jBX3tUJebenVQT2xbJCaibCGzYd/HzbZInbQDlef+RzisBwdjCDskBmz WPF9gDjwqiVYYPlLIHEaT8BNnBbYkLvX/1YoQglUxA+P/XUasG/0kSZ1sXMvKDmdvXtd AXsh4lUeE1+gev0M9Pz4vC/d0qckqh/x7UNkxjwZQTXSvNSpr4PSNBcb995FcpHCk33B mpzlxKBy9boZazC1r/8bA6nuQ3XxO7eI6T00+p7ITr/AgWvRBNVN76jvqVCvchFpg8Zr TPBA== X-Gm-Message-State: AOAM532MRLyT2TrSeMncX9Kp9GN3p1H2WbSQmkLBR4xEp9gxFH79WUm5 wdsan871xkzakFKoYQIZqnMaGlFkPe8ds/HSdOTWkA== X-Google-Smtp-Source: ABdhPJwjfqiykgyQMB53CTbJoEXhAUv6CUZv3YiB2i31zCDl+L/m1OIOgC1bbPImrHcKexC87EFK3LnIJRnKxx3nbf8= X-Received: by 2002:a19:754:: with SMTP id 81mr2187595lfh.272.1600810822633; Tue, 22 Sep 2020 14:40:22 -0700 (PDT) MIME-Version: 1.0 References: <202009112049.08BKnavL032212@repo.freebsd.org> In-Reply-To: From: Warner Losh Date: Tue, 22 Sep 2020 15:40:11 -0600 Message-ID: Subject: Re: svn commit: r365643 - head/bin/cp To: Alan Somers Cc: Kyle Evans , Mateusz Guzik , src-committers , svn-src-all , svn-src-head X-Rspamd-Queue-Id: 4BwvqX5hBKz3SCd X-Spamd-Bar: - Authentication-Results: mx1.freebsd.org; dkim=pass header.d=bsdimp-com.20150623.gappssmtp.com header.s=20150623 header.b=E3uejyZ8; dmarc=none; spf=none (mx1.freebsd.org: domain of wlosh@bsdimp.com has no SPF policy when checking 2a00:1450:4864:20::129) smtp.mailfrom=wlosh@bsdimp.com X-Spamd-Result: default: False [-1.58 / 15.00]; RCVD_TLS_ALL(0.00)[]; MAILMAN_DEST(0.00)[svn-src-head]; R_DKIM_ALLOW(-0.20)[bsdimp-com.20150623.gappssmtp.com:s=20150623]; ARC_NA(0.00)[]; FROM_HAS_DN(0.00)[]; NEURAL_HAM_MEDIUM(-0.91)[-0.909]; NEURAL_HAM_LONG(-0.98)[-0.976]; MIME_GOOD(-0.10)[multipart/alternative,text/plain]; PREVIOUSLY_DELIVERED(0.00)[svn-src-head@freebsd.org]; DMARC_NA(0.00)[bsdimp.com]; RCPT_COUNT_FIVE(0.00)[6]; TO_MATCH_ENVRCPT_SOME(0.00)[]; TO_DN_ALL(0.00)[]; DKIM_TRACE(0.00)[bsdimp-com.20150623.gappssmtp.com:+]; RCVD_IN_DNSWL_NONE(0.00)[2a00:1450:4864:20::129:from]; NEURAL_SPAM_SHORT(0.31)[0.307]; R_SPF_NA(0.00)[no SPF record]; FORGED_SENDER(0.30)[imp@bsdimp.com,wlosh@bsdimp.com]; MIME_TRACE(0.00)[0:+,1:+,2:~]; RCVD_COUNT_TWO(0.00)[2]; FREEMAIL_CC(0.00)[freebsd.org,gmail.com]; FROM_NEQ_ENVFROM(0.00)[imp@bsdimp.com,wlosh@bsdimp.com]; ASN(0.00)[asn:15169, ipnet:2a00:1450::/32, country:US] Content-Type: text/plain; charset="UTF-8" X-Content-Filtered-By: Mailman/MimeDel 2.1.33 X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.33 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 22 Sep 2020 21:40:25 -0000 So why do we need a workaround at all? cp /dev/null has been fixed, and that's way more important to get right. I don't want to paper-over issues with this at all, though if we use the host's (now broken) cp, I suppose that might be OK in the short term. If that's the case, then maybe this is OK. Otherwise, I'd strongly prefer we fix cp. Warner On Tue, Sep 22, 2020 at 3:31 PM Alan Somers wrote: > +1. > > On Tue, Sep 22, 2020 at 3:27 PM Kyle Evans wrote: > >> I'm running a build at the suggestion of mjg to confirm there aren't >> any others hiding that can be converted, and I will commit after I've >> verified that this is it. >> >> On Tue, Sep 22, 2020 at 4:02 PM Alan Somers wrote: >> > >> > LGTM. >> > >> > On Tue, Sep 22, 2020 at 2:59 PM Kyle Evans wrote: >> >> >> >> Perhaps: >> >> >> >> diff --git a/stand/i386/zfsboot/Makefile b/stand/i386/zfsboot/Makefile >> >> index ff315abc0ef..7e362b43a39 100644 >> >> --- a/stand/i386/zfsboot/Makefile >> >> +++ b/stand/i386/zfsboot/Makefile >> >> @@ -81,7 +81,7 @@ zfsboot.ld: zfsboot.ldr zfsboot.bin ${BTXKERN} >> >> -o ${.TARGET} -P 1 zfsboot.bin >> >> >> >> zfsboot.ldr: >> >> - cp /dev/null ${.TARGET} >> >> + :> ${.TARGET} >> >> >> >> zfsboot.bin: zfsboot.out >> >> ${OBJCOPY} -S -O binary zfsboot.out ${.TARGET} >> >> diff --git a/stand/libsa/Makefile b/stand/libsa/Makefile >> >> index effece9e01b..63cd46a9c54 100644 >> >> --- a/stand/libsa/Makefile >> >> +++ b/stand/libsa/Makefile >> >> @@ -122,7 +122,7 @@ beforedepend: >> >> ln -sf ${SRCTOP}/include/arpa/inet.h arpa/inet.h; \ >> >> ln -sf ${SRCTOP}/include/arpa/tftp.h arpa/tftp.h; \ >> >> for i in _time.h _strings.h _string.h; do \ >> >> - [ -f xlocale/$$i ] || cp /dev/null xlocale/$$i; \ >> >> + [ -f xlocale/$$i ] || :> xlocale/$$i; \ >> >> done; \ >> >> for i in ${STAND_H_INC}; do \ >> >> ln -sf ${SASRC}/stand.h $$i; \ >> >> >> >> >> >> On Tue, Sep 22, 2020 at 3:58 PM Alan Somers >> wrote: >> >> > >> >> > Looks like two places in stand. Is there any reason why Mateusz's >> suggestion wouldn't work? >> >> > >> >> > > rg -g Makefile 'cp.*/dev/null' >> >> > stand/libsa/Makefile >> >> > 125: [ -f xlocale/$$i ] || cp /dev/null xlocale/$$i; \ >> >> > >> >> > stand/i386/zfsboot/Makefile >> >> > 82: cp /dev/null ${.TARGET} >> >> > >> >> > On Tue, Sep 22, 2020 at 2:54 PM Mateusz Guzik >> wrote: >> >> >> >> >> >> Can we instead add a workaround to the build tree? >> >> >> >> >> >> Where is cp /dev/null coming from anyway? Perhaps this can be >> patched >> >> >> to touch the target file. >> >> >> >> >> >> On 9/22/20, Alan Somers wrote: >> >> >> > On Tue, Sep 22, 2020 at 2:48 PM Kyle Evans >> wrote: >> >> >> > >> >> >> >> On Fri, Sep 11, 2020 at 3:49 PM Alan Somers >> wrote: >> >> >> >> > >> >> >> >> > Author: asomers >> >> >> >> > Date: Fri Sep 11 20:49:36 2020 >> >> >> >> > New Revision: 365643 >> >> >> >> > URL: https://svnweb.freebsd.org/changeset/base/365643 >> >> >> >> > >> >> >> >> > Log: >> >> >> >> > cp: fall back to read/write if copy_file_range fails >> >> >> >> > >> >> >> >> > Even though copy_file_range has a file-system agnostic >> version, it >> >> >> >> still >> >> >> >> > fails on devfs (perhaps because the file descriptor is >> non-seekable?) >> >> >> >> In >> >> >> >> > that case, fallback to old-fashioned read/write. Fixes >> >> >> >> > "cp /dev/null /tmp/null" >> >> >> >> > >> >> >> >> >> >> >> >> Hi, >> >> >> >> >> >> >> >> Any objection to adding a quick UPDATING entry for this? I'm >> seeing >> >> >> >> occasional reports of this breakage as recent as today on IRC >> from >> >> >> >> folks that were a little bit thrown off by this because it >> throws up >> >> >> >> fairly far into the build and looks like a stand build regression >> >> >> >> instead of a cp regression. >> >> >> >> >> >> >> >> Thanks, >> >> >> >> >> >> >> >> Kyle Evans >> >> >> >> >> >> >> > >> >> >> > No objection. Can you suggest the proper wording? >> >> >> > _______________________________________________ >> >> >> > svn-src-all@freebsd.org mailing list >> >> >> > https://lists.freebsd.org/mailman/listinfo/svn-src-all >> >> >> > To unsubscribe, send any mail to " >> svn-src-all-unsubscribe@freebsd.org" >> >> >> > >> >> >> >> >> >> >> >> >> -- >> >> >> Mateusz Guzik >> >