From owner-svn-soc-all@freebsd.org Thu May 26 15:34:26 2016 Return-Path: Delivered-To: svn-soc-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 CCF06B4B6CA for ; Thu, 26 May 2016 15:34:26 +0000 (UTC) (envelope-from vincenzo@FreeBSD.org) Received: from socsvn.freebsd.org (socsvn.freebsd.org [IPv6:2001:1900:2254:206a::50:2]) (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 A7C0B18F0 for ; Thu, 26 May 2016 15:34:26 +0000 (UTC) (envelope-from vincenzo@FreeBSD.org) Received: from socsvn.freebsd.org ([127.0.1.124]) by socsvn.freebsd.org (8.15.2/8.15.2) with ESMTP id u4QFYQLM022503 for ; Thu, 26 May 2016 15:34:26 GMT (envelope-from vincenzo@FreeBSD.org) Received: (from www@localhost) by socsvn.freebsd.org (8.15.2/8.15.2/Submit) id u4QFYQfh022501 for svn-soc-all@FreeBSD.org; Thu, 26 May 2016 15:34:26 GMT (envelope-from vincenzo@FreeBSD.org) Date: Thu, 26 May 2016 15:34:26 GMT Message-Id: <201605261534.u4QFYQfh022501@socsvn.freebsd.org> X-Authentication-Warning: socsvn.freebsd.org: www set sender to vincenzo@FreeBSD.org using -f From: vincenzo@FreeBSD.org To: svn-soc-all@FreeBSD.org Subject: socsvn commit: r303933 - soc2016/vincenzo/head/sys/dev/netmap MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-soc-all@freebsd.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: SVN commit messages for the entire Summer of Code repository List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 26 May 2016 15:34:26 -0000 Author: vincenzo Date: Thu May 26 15:34:25 2016 New Revision: 303933 URL: http://svnweb.FreeBSD.org/socsvn/?view=rev&rev=303933 Log: ptnet: carry out features negotiation Modified: soc2016/vincenzo/head/sys/dev/netmap/if_ptnet.c Modified: soc2016/vincenzo/head/sys/dev/netmap/if_ptnet.c ============================================================================== --- soc2016/vincenzo/head/sys/dev/netmap/if_ptnet.c Thu May 26 15:33:55 2016 (r303932) +++ soc2016/vincenzo/head/sys/dev/netmap/if_ptnet.c Thu May 26 15:34:25 2016 (r303933) @@ -95,8 +95,13 @@ struct mtx core_mtx; char core_mtx_name[16]; char hwaddr[ETHER_ADDR_LEN]; + + /* PCI BARs support. */ struct resource *iomem; struct resource *msix_mem; + + /* Mirror of PTFEAT register. */ + uint32_t ptfeatures; }; #define PTNET_CORE_LOCK_INIT(_sc) do { \ @@ -163,9 +168,10 @@ static int ptnet_attach(device_t dev) { + uint32_t ptfeatures = NET_PTN_FEATURES_BASE; struct ptnet_softc *sc; struct ifnet *ifp; - int rid; + int err, rid; device_printf(dev, "%s\n", __func__); @@ -183,11 +189,24 @@ return (ENXIO); } + /* Check if we are supported by the hypervisor. If not, + * bail out immediately. */ + bus_write_4(sc->iomem, PTNET_IO_PTFEAT, ptfeatures); /* wanted */ + ptfeatures = bus_read_4(sc->iomem, PTNET_IO_PTFEAT); /* acked */ + if (!(ptfeatures & NET_PTN_FEATURES_BASE)) { + device_printf(dev, "Hypervisor does not support netmap " + "passthorugh"); + err = ENXIO; + goto err_path; + } + sc->ptfeatures = ptfeatures; + /* Setup Ethernet interface. */ sc->ifp = ifp = if_alloc(IFT_ETHER); if (ifp == NULL) { device_printf(dev, "Failed to allocate ifnet\n"); - return (ENOMEM); + err = ENOMEM; + goto err_path; } if_initname(ifp, device_get_name(dev), device_get_unit(dev)); @@ -217,6 +236,10 @@ PTNET_CORE_LOCK_INIT(sc); return (0); + +err_path: + ptnet_detach(dev); + return err; } static int @@ -224,23 +247,22 @@ { struct ptnet_softc *sc = device_get_softc(dev); - printf("%s\n", __func__); - - if (device_is_attached(dev)) { - ether_ifdetach(sc->ifp); - } - - ifmedia_removeall(&sc->media); + device_printf(dev, "%s\n", __func__); if (sc->ifp) { + ether_ifdetach(sc->ifp); + ifmedia_removeall(&sc->media); if_free(sc->ifp); sc->ifp = NULL; } PTNET_CORE_LOCK_FINI(sc); - bus_release_resource(dev, SYS_RES_IOPORT, - PCIR_BAR(PTNETMAP_IO_PCI_BAR), sc->iomem); + if (sc->iomem) { + bus_release_resource(dev, SYS_RES_IOPORT, + PCIR_BAR(PTNETMAP_IO_PCI_BAR), sc->iomem); + sc->iomem = NULL; + } return (0); }