From owner-freebsd-virtualization@freebsd.org Mon Feb 15 01:58:22 2016 Return-Path: Delivered-To: freebsd-virtualization@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 5510BAA9A3C for ; Mon, 15 Feb 2016 01:58:22 +0000 (UTC) (envelope-from daemon-user@freebsd.org) Received: from phabric-backend.rbsd.freebsd.org (unknown [IPv6:2607:fc50:2000:101::1bb:73]) by mx1.freebsd.org (Postfix) with ESMTP id 3475296D for ; Mon, 15 Feb 2016 01:58:22 +0000 (UTC) (envelope-from daemon-user@freebsd.org) Received: by phabric-backend.rbsd.freebsd.org (Postfix, from userid 1346) id 342416FBA; Mon, 15 Feb 2016 01:58:22 +0000 (UTC) Date: Mon, 15 Feb 2016 01:58:22 +0000 To: freebsd-virtualization@freebsd.org From: "sepherosa_gmail.com (Sepherosa Ziehau)" Reply-to: D5282+333+bde49f8f80ca0f1f@reviews.freebsd.org Subject: [Differential] [Request, 4 lines] D5282: hyperv/hn: Use non-fast taskqueue for transmission Message-ID: X-Priority: 3 X-Phabricator-Sent-This-Message: Yes X-Mail-Transport-Agent: MetaMTA X-Auto-Response-Suppress: All X-Phabricator-Mail-Tags: , , , Thread-Topic: D5282: hyperv/hn: Use non-fast taskqueue for transmission X-Herald-Rules: none X-Phabricator-To: X-Phabricator-To: X-Phabricator-To: X-Phabricator-To: X-Phabricator-To: X-Phabricator-To: X-Phabricator-To: X-Phabricator-To: X-Phabricator-Cc: X-Phabricator-Cc: Precedence: bulk Thread-Index: MTA0OWYzZmY5Yjk5MjNmYjE3MzdmZmI1ZDFj MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="b1_0295902bf080b87423a1fe647f5acc2b" X-BeenThere: freebsd-virtualization@freebsd.org X-Mailman-Version: 2.1.20 List-Id: "Discussion of various virtualization techniques FreeBSD supports." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 15 Feb 2016 01:58:22 -0000 --b1_0295902bf080b87423a1fe647f5acc2b Content-Type: text/plain; charset = "utf-8" Content-Transfer-Encoding: 8bit sepherosa_gmail.com created this revision. sepherosa_gmail.com added reviewers: network, adrian, delphij, royger, decui_microsoft.com, honzhan_microsoft.com, howard0su_gmail.com. sepherosa_gmail.com added subscribers: freebsd-net-list, freebsd-virtualization-list. REVISION SUMMARY Performance stays same; so no need to use fast taskqueue here. REVISION DETAIL https://reviews.freebsd.org/D5282 AFFECTED FILES sys/dev/hyperv/netvsc/hv_netvsc_drv_freebsd.c CHANGE DETAILS diff --git a/sys/dev/hyperv/netvsc/hv_netvsc_drv_freebsd.c b/sys/dev/hyperv/netvsc/hv_netvsc_drv_freebsd.c --- a/sys/dev/hyperv/netvsc/hv_netvsc_drv_freebsd.c +++ b/sys/dev/hyperv/netvsc/hv_netvsc_drv_freebsd.c @@ -368,7 +368,7 @@ sc->hn_direct_tx_size = hn_direct_tx_size; if (hn_tx_taskq == NULL) { - sc->hn_tx_taskq = taskqueue_create_fast("hn_tx", M_WAITOK, + sc->hn_tx_taskq = taskqueue_create("hn_tx", M_WAITOK, taskqueue_thread_enqueue, &sc->hn_tx_taskq); taskqueue_start_threads(&sc->hn_tx_taskq, 1, PI_NET, "%s tx", device_get_nameunit(dev)); @@ -2178,7 +2178,7 @@ if (!hn_share_tx_taskq) return; - hn_tx_taskq = taskqueue_create_fast("hn_tx", M_WAITOK, + hn_tx_taskq = taskqueue_create("hn_tx", M_WAITOK, taskqueue_thread_enqueue, &hn_tx_taskq); taskqueue_start_threads(&hn_tx_taskq, 1, PI_NET, "hn tx"); } EMAIL PREFERENCES https://reviews.freebsd.org/settings/panel/emailpreferences/ To: sepherosa_gmail.com, network, adrian, delphij, royger, decui_microsoft.com, honzhan_microsoft.com, howard0su_gmail.com Cc: freebsd-virtualization-list, freebsd-net-list --b1_0295902bf080b87423a1fe647f5acc2b Content-Type: text/x-patch; charset=utf-8; name="D5282.13307.patch" Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="D5282.13307.patch" ZGlmZiAtLWdpdCBhL3N5cy9kZXYvaHlwZXJ2L25ldHZzYy9odl9uZXR2c2NfZHJ2X2ZyZWVic2Qu YyBiL3N5cy9kZXYvaHlwZXJ2L25ldHZzYy9odl9uZXR2c2NfZHJ2X2ZyZWVic2QuYwotLS0gYS9z eXMvZGV2L2h5cGVydi9uZXR2c2MvaHZfbmV0dnNjX2Rydl9mcmVlYnNkLmMKKysrIGIvc3lzL2Rl di9oeXBlcnYvbmV0dnNjL2h2X25ldHZzY19kcnZfZnJlZWJzZC5jCkBAIC0zNjgsNyArMzY4LDcg QEAKIAlzYy0+aG5fZGlyZWN0X3R4X3NpemUgPSBobl9kaXJlY3RfdHhfc2l6ZTsKIAogCWlmICho bl90eF90YXNrcSA9PSBOVUxMKSB7Ci0JCXNjLT5obl90eF90YXNrcSA9IHRhc2txdWV1ZV9jcmVh dGVfZmFzdCgiaG5fdHgiLCBNX1dBSVRPSywKKwkJc2MtPmhuX3R4X3Rhc2txID0gdGFza3F1ZXVl X2NyZWF0ZSgiaG5fdHgiLCBNX1dBSVRPSywKIAkJICAgIHRhc2txdWV1ZV90aHJlYWRfZW5xdWV1 ZSwgJnNjLT5obl90eF90YXNrcSk7CiAJCXRhc2txdWV1ZV9zdGFydF90aHJlYWRzKCZzYy0+aG5f dHhfdGFza3EsIDEsIFBJX05FVCwgIiVzIHR4IiwKIAkJICAgIGRldmljZV9nZXRfbmFtZXVuaXQo ZGV2KSk7CkBAIC0yMTc4LDcgKzIxNzgsNyBAQAogCWlmICghaG5fc2hhcmVfdHhfdGFza3EpCiAJ CXJldHVybjsKIAotCWhuX3R4X3Rhc2txID0gdGFza3F1ZXVlX2NyZWF0ZV9mYXN0KCJobl90eCIs IE1fV0FJVE9LLAorCWhuX3R4X3Rhc2txID0gdGFza3F1ZXVlX2NyZWF0ZSgiaG5fdHgiLCBNX1dB SVRPSywKIAkgICAgdGFza3F1ZXVlX3RocmVhZF9lbnF1ZXVlLCAmaG5fdHhfdGFza3EpOwogCXRh c2txdWV1ZV9zdGFydF90aHJlYWRzKCZobl90eF90YXNrcSwgMSwgUElfTkVULCAiaG4gdHgiKTsK IH0KCg== --b1_0295902bf080b87423a1fe647f5acc2b--