From owner-svn-src-head@freebsd.org Mon Aug 10 12:40:18 2015 Return-Path: Delivered-To: svn-src-head@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 D562799D217; Mon, 10 Aug 2015 12:40:18 +0000 (UTC) (envelope-from glebius@FreeBSD.org) Received: from cell.glebius.int.ru (glebius.int.ru [81.19.69.10]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "cell.glebius.int.ru", Issuer "cell.glebius.int.ru" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id 62F9CBB2; Mon, 10 Aug 2015 12:40:17 +0000 (UTC) (envelope-from glebius@FreeBSD.org) Received: from cell.glebius.int.ru (localhost [127.0.0.1]) by cell.glebius.int.ru (8.15.2/8.15.2) with ESMTPS id t7ACe8Rw076955 (version=TLSv1.2 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO); Mon, 10 Aug 2015 15:40:08 +0300 (MSK) (envelope-from glebius@FreeBSD.org) Received: (from glebius@localhost) by cell.glebius.int.ru (8.15.2/8.15.2/Submit) id t7ACe89f076954; Mon, 10 Aug 2015 15:40:08 +0300 (MSK) (envelope-from glebius@FreeBSD.org) X-Authentication-Warning: cell.glebius.int.ru: glebius set sender to glebius@FreeBSD.org using -f Date: Mon, 10 Aug 2015 15:40:08 +0300 From: Gleb Smirnoff To: Rui Paulo Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r286441 - in head: share/man/man4 sys/contrib/dev/iwm sys/dev/iwm sys/modules/iwm sys/modules/iwmfw sys/modules/iwmfw/iwm3160fw sys/modules/iwmfw/iwm7260fw sys/modules/iwmfw/iwm7265fw Message-ID: <20150810124008.GB75813@FreeBSD.org> References: <201508080606.t7866nw0072494@repo.freebsd.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <201508080606.t7866nw0072494@repo.freebsd.org> User-Agent: Mutt/1.5.23 (2014-03-12) X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.20 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: Mon, 10 Aug 2015 12:40:18 -0000 On Sat, Aug 08, 2015 at 06:06:49AM +0000, Rui Paulo wrote: R> Author: rpaulo R> Date: Sat Aug 8 06:06:48 2015 R> New Revision: 286441 R> URL: https://svnweb.freebsd.org/changeset/base/286441 R> R> Log: R> Import OpenBSD's iwm WiFi driver for Intel 3160/7260/7265. R> R> There are still several bugs, but I've been using it for a while now. R> Thanks to all the testers and to Adrian for his help with this R> driver. R> R> This driver isn't connected to the build yet, but it will be soon. R> R> There's no MFC planned because the driver isn't very stable yet. R> R> Reviewed by: adrian R> Obtained from: https://github.com/rpaulo/iwm R> Tested by: adrian, gjb, dumbbell (others that I forgot). Converting the driver to new KPI, I noticed that iwm_media_change() expects the "parent ifnet" to be its argument: static int iwm_media_change(struct ifnet *ifp) { struct iwm_softc *sc = ifp->if_softc; I believe this is wrong, its argument is "vap ifnet", so the code should be: struct ieee80211vap *vap = ifp->if_softc; struct ieee80211com *ic = vap->iv_ic; struct wi_softc *sc = ic->ic_softc; The fact that it didn't panic is probably explained by the fact that this function was never executed. P.S. And this is an illustration on how convoluted the current KPI is :) P.P.S. I plan to be ready with retry of KPI patch today. Including iwm, and all ath debug fixes. -- Totus tuus, Glebius.