Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753410AbdLNPcj (ORCPT ); Thu, 14 Dec 2017 10:32:39 -0500 Received: from alexa-out.qualcomm.com ([129.46.98.28]:36049 "EHLO alexa-out.qualcomm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753071AbdLNPch (ORCPT ); Thu, 14 Dec 2017 10:32:37 -0500 X-IronPort-AV: E=McAfee;i="5900,7806,8744"; a="1502938927" X-MGA-submission: =?us-ascii?q?MDEYGZx8BF12V5vgvqPd7yFGA7R43EqSIfbuR+?= =?us-ascii?q?7/L86A9oaTC3oFCMw4MzghCscYqXax7iWuLQzZn3OjhtpxF8N0x61L8E?= =?us-ascii?q?MWRyKC6j0kJsG5AHZyjGj23l68gaZDqwMZupAmCIV0IQ10sC4B4JznnJ?= =?us-ascii?q?tc?= Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: Re: wcn36xx: Reduce spinlock in indication handler From: Kalle Valo In-Reply-To: <20171209003508.3507-1-bjorn.andersson@linaro.org> References: <20171209003508.3507-1-bjorn.andersson@linaro.org> To: Bjorn Andersson CC: Eugene Krasnikov , Kalle Valo , Loic Poulain , , , , User-Agent: pwcli/0.0.0-git (https://github.com/kvalo/pwcli/) Python/2.7.12 Message-ID: Date: Thu, 14 Dec 2017 16:32:23 +0100 X-Originating-IP: [10.80.80.8] X-ClientProxiedBy: NASANEXM01F.na.qualcomm.com (10.85.0.32) To euamsexm01a.eu.qualcomm.com (10.251.127.40) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 808 Lines: 22 Bjorn Andersson wrote: > The purpose of pushing indication on a list and handle these in a > separate worker is to allow the handlers to sleep. It does therefor not > make much sense to hold the queue spinlock through the entire indication > worker function. > > By removing items from the queue early we don't need to hold the lock > throughout the indication worker, allowing the individual handlers to > sleep. > > Signed-off-by: Bjorn Andersson > Signed-off-by: Kalle Valo Patch applied to ath-next branch of ath.git, thanks. 6d1f37323f5b wcn36xx: Reduce spinlock in indication handler -- https://patchwork.kernel.org/patch/10103469/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches