From owner-svn-src-all@freebsd.org Mon Mar 27 07:04:53 2017 Return-Path: Delivered-To: svn-src-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 533F9D1F1C7; Mon, 27 Mar 2017 07:04:53 +0000 (UTC) (envelope-from andriyvos@gmail.com) Received: from mail-wr0-f170.google.com (mail-wr0-f170.google.com [209.85.128.170]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id EA0CF1DB0; Mon, 27 Mar 2017 07:04:52 +0000 (UTC) (envelope-from andriyvos@gmail.com) Received: by mail-wr0-f170.google.com with SMTP id w43so29909635wrb.0; Mon, 27 Mar 2017 00:04:52 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:to:cc:subject:references:date:mime-version :content-transfer-encoding:from:message-id:in-reply-to:user-agent; bh=Ntm/ZadwFPdKP49oCG5gO252cWPJi5AXKr57YZkv0DE=; b=Ln6RokY7vPXXPzXifg5j39WpVbVVEZSCFaxnUoIJibSI7op/8BKx8ZPWMNv7nOspcf /nZQ2KPjMZt7FgEMINvjHh+Lb0WgzhG19oJjBDOF/GYcVJXOAlEAkG71dITTBwDoTdf0 onLex91EdfwigCmMiwjuwv7kRtK1IfRnRKWpqFyDH5NzdAVBOgSHzEExGjLgRK4eSbdl jp6JeLLL7pFwkn1PWM61CuOUzLNj5yWIwQG1+ZEoUBRuqnwoRWhtmtg0RCL3lH21jxdv AWS1A+CLRfnoPN75aNjLO4S1t+VWtDEn3g6lh2yS82ubUG9wyecOEp2Mc2Mrd11dM3np g3Rg== X-Gm-Message-State: AFeK/H3lNYjFOVZz8zTj9N1cm6EBbCMOS50GuIca6BMkPq0khcyeOULCPzZzGydctVuqfQ== X-Received: by 10.28.156.69 with SMTP id f66mr7878714wme.56.1490598284821; Mon, 27 Mar 2017 00:04:44 -0700 (PDT) Received: from thinkpad-x220 (host-176-37-109-22.la.net.ua. [176.37.109.22]) by smtp.gmail.com with ESMTPSA id b91sm13600061wrd.29.2017.03.27.00.04.44 (version=TLS1 cipher=AES128-SHA bits=128/128); Mon, 27 Mar 2017 00:04:44 -0700 (PDT) Content-Type: text/plain; charset=utf-8; format=flowed; delsp=yes To: "Hartmann, O." Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r315981 - head/sys/dev/iwn References: <201703260910.v2Q9A26Z081833@repo.freebsd.org> <20170326164309.759fc5b9@hermann> Date: Mon, 27 Mar 2017 10:04:39 +0300 MIME-Version: 1.0 Content-Transfer-Encoding: Quoted-Printable From: "Andriy Voskoboinyk" Message-ID: In-Reply-To: <20170326164309.759fc5b9@hermann> User-Agent: Opera Mail/12.15 (FreeBSD) X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.23 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: Mon, 27 Mar 2017 07:04:53 -0000 Sun, 26 Mar 2017 17:43:18 +0300 =D0=B1=D1=83=D0=BB=D0=BE =D0=BD=D0=B0=D0= =BF=D0=B8=D1=81=D0=B0=D0=BD=D0=BE Hartmann, O. = : > On Sun, 26 Mar 2017 09:10:02 +0000 (UTC) > Andriy Voskoboinyk wrote: > >> Author: avos >> Date: Sun Mar 26 09:10:01 2017 >> New Revision: 315981 >> URL: https://svnweb.freebsd.org/changeset/base/315981 >> >> Log: >> iwn: omit unneeded bus_dmamap_sync() calls when compiled without >> 'options IWN_DEBUG' >> >> Modified: >> head/sys/dev/iwn/if_iwn.c >> >> Modified: head/sys/dev/iwn/if_iwn.c >> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D >> --- head/sys/dev/iwn/if_iwn.c Sun Mar 26 08:54:08 2017 >> (r315980) +++ head/sys/dev/iwn/if_iwn.c Sun Mar 26 09:10:01 >> 2017 (r315981) @@ -3944,6 +3944,7 @@ iwn_notif_intr(struct >> iwn_softc *sc) sc->errptr =3D le32toh(uc->errptr); >> break; >> } >> +#ifdef IWN_DEBUG >> case IWN_STATE_CHANGED: >> { >> /* >> @@ -3953,27 +3954,26 @@ iwn_notif_intr(struct iwn_softc *sc) >> */ >> bus_dmamap_sync(sc->rxq.data_dmat, data->map, >> BUS_DMASYNC_POSTREAD); >> -#ifdef IWN_DEBUG >> + >> uint32_t *status =3D (uint32_t *)(desc + 1); >> DPRINTF(sc, IWN_DEBUG_INTR | IWN_DEBUG_STATE, >> "state changed to %x\n", >> le32toh(*status)); >> -#endif >> break; >> } >> case IWN_START_SCAN: >> { >> bus_dmamap_sync(sc->rxq.data_dmat, data->map, >> BUS_DMASYNC_POSTREAD); >> -#ifdef IWN_DEBUG >> + >> struct iwn_start_scan *scan =3D >> (struct iwn_start_scan *)(desc + 1); >> DPRINTF(sc, IWN_DEBUG_ANY, >> "%s: scanning channel %d status %x\n", >> __func__, scan->chan, >> le32toh(scan->status)); -#endif >> break; >> } >> +#endif >> case IWN_STOP_SCAN: >> { >> bus_dmamap_sync(sc->rxq.data_dmat, data->map, >> _______________________________________________ >> 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@freebsd.org" > > It seems when iwm is configured in a kernel without debugging enabled,= > builkernel failsas shown below: > > > > > [...] > -------------------------------------------------------------- >>>> stage 3.1: building everything > -------------------------------------------------------------- > cd /usr/obj/usr/src/sys/HERMANN; COMPILER_VERSION=3D40000 > COMPILER_TYPE=3Dclang COMPILER_FREEBSD_VERSION=3D1200006 > MAKEOBJDIRPREFIX=3D/usr/obj MACHINE_ARCH=3Damd64 MACHINE=3Damd64 > CPUTYPE=3Dnative BUILD_TOOLS_META=3D.NOMETA_CMP > GROFF_BIN_PATH=3D/usr/obj/usr/src/tmp/legacy/usr/bin > GROFF_FONT_PATH=3D/usr/obj/usr/src/tmp/legacy/usr/share/groff_font > GROFF_TMAC_PATH=3D/usr/obj/usr/src/tmp/legacy/usr/share/tmac CC=3D"cc > -target x86_64-unknown-freebsd12.0 --sysroot=3D/usr/obj/usr/src/tmp > -B/usr/obj/usr/src/tmp/usr/bin" CXX=3D"c++ -target > x86_64-unknown-freebsd12.0 --sysroot=3D/usr/obj/usr/src/tmp > -B/usr/obj/usr/src/tmp/usr/bin" CPP=3D"cpp -target > x86_64-unknown-freebsd12.0 --sysroot=3D/usr/obj/usr/src/tmp > -B/usr/obj/usr/src/tmp/usr/bin" AS=3D"as" AR=3D"ar" LD=3D"ld" LLVM_LI= NK=3D"" > NM=3Dnm OBJCOPY=3D"objcopy" RANLIB=3Dranlib STRINGS=3D SIZE=3D"size"= > INSTALL=3D"sh /usr/src/tools/install.sh" > PATH=3D/usr/obj/usr/src/tmp/legacy/usr/sbin:/usr/obj/usr/src/tmp/legac= y/usr/bin:/usr/obj/usr/src/tmp/legacy/bin:/usr/obj/usr/src/tmp/usr/sbin:= /usr/obj/usr/src/tmp/usr/bin:/sbin:/bin:/usr/sbin:/usr/bin > make -m /usr/src/share/mk KERNEL=3Dkernel all -DNO_MODULES_OBJ > Building /usr/obj/usr/src/sys/HERMANN/if_iwm_scan.o = > /usr/src/sys/dev/iwm/if_iwm_scan.c:224:12: > error: unused variable 'aborted' [-Werror,-Wunused-variable] boolean_t= > aborted =3D (scan_notif->status =3D=3D IWM_SCAN_OFFLOAD_ABORTED); > ^ /usr/src/sys/dev/iwm/if_iwm_scan.c:251:11: error: unused variable > 'uid' [-Werror,-Wunused-variable] uint32_t uid =3D le32toh(notif->uid)= ; Fixed in r316010.