From owner-svn-src-all@freebsd.org Tue Jun 5 16:47:18 2018 Return-Path: Delivered-To: svn-src-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 1187EFDC76D for ; Tue, 5 Jun 2018 16:47:18 +0000 (UTC) (envelope-from ian@freebsd.org) Received: from pmta2.delivery6.ore.mailhop.org (pmta2.delivery6.ore.mailhop.org [54.200.129.228]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 8ECDF83C3D for ; Tue, 5 Jun 2018 16:47:17 +0000 (UTC) (envelope-from ian@freebsd.org) X-MHO-RoutePath: aGlwcGll X-MHO-User: 159b46d6-68e0-11e8-b829-b3adae557cda X-Report-Abuse-To: https://support.duocircle.com/support/solutions/articles/5000540958-duocircle-standard-smtp-abuse-information X-Originating-IP: 67.177.211.60 X-Mail-Handler: DuoCircle Outbound SMTP Received: from ilsoft.org (unknown [67.177.211.60]) by outbound2.ore.mailhop.org (Halon) with ESMTPSA id 159b46d6-68e0-11e8-b829-b3adae557cda; Tue, 05 Jun 2018 16:47:08 +0000 (UTC) Received: from rev (rev [172.22.42.240]) by ilsoft.org (8.15.2/8.15.2) with ESMTP id w55Gl7r9038253; Tue, 5 Jun 2018 10:47:07 -0600 (MDT) (envelope-from ian@freebsd.org) Message-ID: <1528217227.63685.0.camel@freebsd.org> Subject: Re: svn commit: r334656 - head/stand/libsa From: Ian Lepore To: Warner Losh , "O. Hartmann" Cc: src-committers , svn-src-all@freebsd.org, svn-src-head@freebsd.org Date: Tue, 05 Jun 2018 10:47:07 -0600 In-Reply-To: References: <201806051447.w55ElDIt013519@repo.freebsd.org> <20180605181822.661e5cf0@thor.intern.walstatt.dynvpn.de> Content-Type: text/plain; charset="ISO-8859-1" X-Mailer: Evolution 3.18.5.1 FreeBSD GNOME Team Port Mime-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.26 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 05 Jun 2018 16:47:18 -0000 On Tue, 2018-06-05 at 10:24 -0600, Warner Losh wrote: > There is no real reason for these to be different. Especially if it > causes > problems... > > Warner > I'm working on a fix. -- Ian > On Tue, Jun 5, 2018, 10:18 AM O. Hartmann > wrote: > > > > > -----BEGIN PGP SIGNED MESSAGE----- > > Hash: SHA512 > > > > Am Tue, 5 Jun 2018 14:47:13 +0000 (UTC) > > Ian Lepore schrieb: > > > > > > > > Author: ian > > > Date: Tue Jun  5 14:47:13 2018 > > > New Revision: 334656 > > > URL: https://svnweb.freebsd.org/changeset/base/334656 > > > > > > Log: > > >   Add vsnprintf() to libsa.  Alpha-sort the printf prototypes in > > > stand.h. > > > > > >   I'm not sure why the v*printf() functions in libsa return void > > > instead > > of > > > > > >   int, but this maintains that convention for the new function. > > > > > > Modified: > > >   head/stand/libsa/printf.c > > >   head/stand/libsa/stand.h > > > > > > Modified: head/stand/libsa/printf.c > > > > > =================================================================== > > =========== > > > > > > --- head/stand/libsa/printf.c Tue Jun  5 13:53:37 > > > 2018        (r334655) > > > +++ head/stand/libsa/printf.c Tue Jun  5 14:47:13 > > > 2018        (r334656) > > > @@ -141,6 +141,20 @@ snprintf(char *buf, size_t size, const char > > > *cfmt, > > ... > > > > > >  } > > > > > >  void > > > +vsnprintf(char *buf, size_t size, const char *cfmt, va_list ap) > > > +{ > > > +     struct print_buf arg; > > > + > > > +     arg.buf = buf; > > > +     arg.size = size; > > > + > > > +     kvprintf(cfmt, &snprint_func, &arg, 10, ap); > > > + > > > +     if (arg.size >= 1) > > > +             *(arg.buf)++ = 0; > > > +} > > > + > > > +void > > >  vsprintf(char *buf, const char *cfmt, va_list ap) > > >  { > > >       int     retval; > > > > > > Modified: head/stand/libsa/stand.h > > > > > =================================================================== > > =========== > > > > > > --- head/stand/libsa/stand.h  Tue Jun  5 13:53:37 > > > 2018        (r334655) > > > +++ head/stand/libsa/stand.h  Tue Jun  5 14:47:13 > > > 2018        (r334656) > > > @@ -268,10 +268,11 @@ extern void     *reallocf(void *ptr, size_t > > > size); > > >  extern void  mallocstats(void); > > > > > >  extern int   printf(const char *fmt, ...) __printflike(1, 2); > > > -extern void  vprintf(const char *fmt, __va_list); > > >  extern int   sprintf(char *buf, const char *cfmt, ...) > > > __printflike(2, > > 3); > > > > > >  extern int   snprintf(char *buf, size_t size, const char *cfmt, > > > ...) > > > __printflike(3, 4); +extern void      vprintf(const char *fmt, > > __va_list); > > > > > >  extern void  vsprintf(char *buf, const char *cfmt, __va_list); > > > +extern void  vsnprintf(char *buf, size_t size, const char *cfmt, > > __va_list); > > > > > > > > >  extern void  twiddle(u_int callerdiv); > > >  extern void  twiddle_divisor(u_int globaldiv); > > > _______________________________________________ > > > svn-src-head@freebsd.org mailing list > > > https://lists.freebsd.org/mailman/listinfo/svn-src-head > > > To unsubscribe, send any mail to "svn-src-head-unsubscribe@freebs > > > d.org" > > > > Building world/kernel fails now with the error shown below: > > > > > > [...] > > In file included from /usr/src/stand/efi/boot1/boot1.c:34: > > - --- all_subdir_share --- > > - --- all_subdir_share/i18n/esdb/KAZAKH --- > > ===> share/i18n/esdb/KAZAKH (all) > > - --- all_subdir_stand --- > > /usr/src/stand/efi/boot1/boot_module.h:109:12: error: conflicting > > types > > for 'vsnprintf' > > extern int vsnprintf(char *str, size_t sz, const char *fmt, va_list > > ap); > >            ^ > > /usr/src/stand/libsa/stand.h:275:13: note: previous declaration is > > here > > extern void     vsnprintf(char *buf, size_t size, const char *cfmt, > > __va_list); > >                 ^ > > 1 error generated. > > - --- all_subdir_rescue --- > > - --- depend_subdir_rescue/rescue/routed/rtquery --- > > ===> rescue/rescue/routed/rtquery (depend) > > - --- all_subdir_stand --- > > *** [boot1.o] Error code 1 > > > > make[5]: stopped in /usr/src/stand/efi/boot1 > > .ERROR_TARGET='boot1.o' > > > > .ERROR_META_FILE='/usr/obj/usr/src/amd64.amd64/stand/efi/boot1/boot > > 1.o.meta' > > .MAKE.LEVEL='5' > > MAKEFILE='' > > .MAKE.MODE='meta missing-filemon=yes missing-meta=yes silent=yes > > verbose' > > _ERROR_CMD='cc -target x86_64-unknown-freebsd12.0 > > - --sysroot=/usr/obj/usr/src/amd64.amd64/tmp > > -B/usr/obj/usr/src/amd64.amd64/tmp/usr/bin > > - -O2 -pipe -O3 -march=native  -Wformat -fshort-wchar -mno-red-zone > > -nostdinc > > - -I/usr/obj/usr/src/amd64.amd64/stand/libsa -I/usr/src/stand/libsa > > -D_STANDALONE > > - -I/usr/src/sys -Ddouble=jagged-little-pill -Dfloat=floaty- > > mcfloatface > > - -DLOADER_DISK_SUPPORT -ffreestanding -mno-mmx -mno-sse -mno-avx > > -mno-avx2 -msoft-float > > - -fPIC -mno-red-zone -I. -DEFI_ZFS_BOOT > > -I/usr/src/stand/efi/include > > - -I/usr/src/stand/efi/include/amd64 > > -I/usr/src/sys/contrib/dev/acpica/include > > - -DEFI_UFS_BOOT -I/usr/src/stand/common -fPIC -DNDEBUG -std=gnu99 > > -Wsystem-headers > > - -Wno-pointer-sign -Wno-empty-body -Wno-string-plus-int > > -Wno-unused-const-variable > > - -Wno-tautological-compare -Wno-unused-value -Wno-parentheses- > > equality > > - -Wno-unused-function -Wno-enum-conversion -Wno-unused-local- > > typedef > > - -Wno-address-of-packed-member -Wno-switch -Wno-switch-enum > > -Wno-knr-promoted-parameter > > - -Wno-parentheses -Wno-format -Qunused-arguments  -c > > /usr/src/stand/efi/boot1/boot1.c -o > > boot1.o; ;' > > > > ... > > > > > > Sources are at revision 334664. > > > > Kind regards, > > > > oh > > - -- > > O. Hartmann > > > > Ich widerspreche der Nutzung oder Übermittlung meiner Daten für > > Werbezwecke oder für die Markt- oder Meinungsforschung (§ 28 Abs. 4 > > BDSG). > > -----BEGIN PGP SIGNATURE----- > > > > iLUEARMKAB0WIQQZVZMzAtwC2T/86TrS528fyFhYlAUCWxa3zgAKCRDS528fyFhY > > lHBLAgCP69VupKSyww77gfghKUBS6eoR6jf6OmvMGWOnxemWBS/BAYwiGCK+ZFmk > > 3ur207rjRoqR/C/jF4ywA2ddQQ5UAgCZ7QxcQD4HClKIn3Mw+W0QvuvXPLv/TBjN > > VnvRfwQfKnXUfjNmlmXrHvDTaHS1R7RtTUvQxK5L5PcAmOuPQhRj > > =YisO > > -----END PGP SIGNATURE----- > >