From patchwork Thu Aug 12 19:50:34 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Alex Elder X-Patchwork-Id: 496048 Delivered-To: patch@linaro.org Received: by 2002:a05:6638:396:0:0:0:0 with SMTP id y22csp719610jap; Thu, 12 Aug 2021 12:50:57 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxSvoGWdJrYhWH/b7AT0BkzqJhCJfnLNBzNXOTEFqRhXbB1JlDzo1qHRR/Pr3W1FiPT7oyR X-Received: by 2002:a6b:b502:: with SMTP id e2mr4211383iof.152.1628797857470; Thu, 12 Aug 2021 12:50:57 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1628797857; cv=none; d=google.com; s=arc-20160816; b=I+evzkNpHOdl3K+SsTOEWIbQdOWVSXlFDS8olQL+eNLj9xBBi2lsp2arLASAwXAr// I4rx4jzrwiZgTWy6gsMRNWusHrfcNurCX9sHwlLaqVwtpVHDQMpUbZ3xgP5wxlq5CQKW hz5EUBntEYrYoQwicqrNDFTzJii3clsA7o35Xtvqzui8Ko71gBrnDNPpgWwViMn4SsrH R9PQ0xEYnbte0Nywz85M6VdDaglyf/gSTtKNl4KpkTvN/xgosQrNBK6X8VbTURv4A1mi sslH3rmy22hIHAl0M+iOLl2nG63p2B6Omi2eWeW2bvDzznK0aXnSF0vqEvlDt25zB8MH SIsQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=QSpci64/hMqy88Q22faTec2s8wp9JGPuRuA3v3rI9YQ=; b=skxW31tDCfBKdltug4YfhAxRdRZlSWuHnA0VmJuqHGj7Ap4BHo6OhYo4kZPra/uUdk qS7myaHO+kBdsOM+Zrn4nKCAdTijPgy2OZI36Fts5yRBN1OuPFdjCPzFz8J1uYver5sH go5H4XQYqq2La8iJCAmvmjU3UO0mpMxgLR3kGvUgYoE8DWBd2D7X86n1Np99ILe8YulY eOgpUNQKwd48aUI/nsVCPnqfV7jFQtLI/q3qtpokGUDFFFugZfXvBHhRyrZ78Uq/kMyn z+NviqifiZOp+FPeoQa0Lr2+wIsRc7K2wCxDvwB9nnVvUrH/G+JOeR99lkKeexx19UcH eFjA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=P3PRMZs3; spf=pass (google.com: domain of netdev-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=netdev-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id i20si4258170jaj.74.2021.08.12.12.50.57; Thu, 12 Aug 2021 12:50:57 -0700 (PDT) Received-SPF: pass (google.com: domain of netdev-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=P3PRMZs3; spf=pass (google.com: domain of netdev-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=netdev-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236856AbhHLTvT (ORCPT + 8 others); Thu, 12 Aug 2021 15:51:19 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:38378 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S236112AbhHLTvJ (ORCPT ); Thu, 12 Aug 2021 15:51:09 -0400 Received: from mail-il1-x135.google.com (mail-il1-x135.google.com [IPv6:2607:f8b0:4864:20::135]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6378CC0613A4 for ; Thu, 12 Aug 2021 12:50:43 -0700 (PDT) Received: by mail-il1-x135.google.com with SMTP id w6so3333921ill.3 for ; Thu, 12 Aug 2021 12:50:43 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=QSpci64/hMqy88Q22faTec2s8wp9JGPuRuA3v3rI9YQ=; b=P3PRMZs3/UXYGNdQf6SaCtATu/V7phGy1FxQf7KjkvhAxxvjN3hT//M8K4o4ZBQ7Fq 3dHx2dIcLC6jIsftPgWyWs4noXGATMniiRJxxFAbN+j8t+TDnsM0qUHYoiPwh2EVwWo/ ud4rYqKOEe1v1AjP4t4lIJ2aLiAcK90QyRTg5+/+EAwD/DhNv65eQwPX7XPjsJAn3Hnd HPvYsAR4a2STwLpB2581d7NsTAR83QTdTPApN1cOmTWAq78WAkIrlbQytKTiVPEY4q22 QandDbjzdrgMHQyogapKm+tDPg3OG8Ppe80H/y0rLEo/erFfc63hrdVTULUsJnEeMXXF cb+g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=QSpci64/hMqy88Q22faTec2s8wp9JGPuRuA3v3rI9YQ=; b=tLzOx5YgGIUexDTNesWNDzeEj0XJp+CN0M2MnLKOt+oqnph1cVBIpSaGcjdPajpHbN fIxk3v3Wh5xuvYEWQUT++UxPbY+uV2kp0WA167tlLo8lxWwwxOnT+OFK7cVpNHbBppdG lpfx/iqqeOIa+qm3j11fGZZhVOv5iMlEvKD/ShtfJa0d0NBCr5h9IXvJ0rj6tAsoggvk psHEs0KFIAlYun2JGC/tkqFP8+uwbJIjIUvOZC2fguO4wbPaOhTe0zjdmtQ2V1YgWgVW qCtPGz1g/ze7I/1OH7Vqoa3zn1o7rTGrUPBg2KEVdc+FZMi+9IeHEB2RGkjPASpbJlJM KqQQ== X-Gm-Message-State: AOAM532b4AV+1EzRXOkm4pFAjumkjwmwVaHpjo3daV3SvlQXu93bysmg gaKG+pt1p3h2sYsKZcX1JsRgKA== X-Received: by 2002:a92:d18c:: with SMTP id z12mr176696ilz.295.1628797842839; Thu, 12 Aug 2021 12:50:42 -0700 (PDT) Received: from presto.localdomain (c-73-185-129-58.hsd1.mn.comcast.net. [73.185.129.58]) by smtp.gmail.com with ESMTPSA id s16sm2058821iln.5.2021.08.12.12.50.42 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 12 Aug 2021 12:50:42 -0700 (PDT) From: Alex Elder To: davem@davemloft.net, kuba@kernel.org Cc: bjorn.andersson@linaro.org, evgreen@chromium.org, cpratapa@codeaurora.org, subashab@codeaurora.org, elder@kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH net-next 5/6] net: ipa: don't stop TX on suspend Date: Thu, 12 Aug 2021 14:50:34 -0500 Message-Id: <20210812195035.2816276-6-elder@linaro.org> X-Mailer: git-send-email 2.27.0 In-Reply-To: <20210812195035.2816276-1-elder@linaro.org> References: <20210812195035.2816276-1-elder@linaro.org> MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org Currently we stop the modem netdev transmit queue when suspending the hardware. For system suspend this ensured we'd never attempt to transmit while attempting to suspend the modem endpoints. For runtime suspend, the IPA hardware might get suspended while the system is operating. In that case we want an attempt to transmit a packet to cause the hardware to resume if necessary. But if we disable the queue this cannot happen. So stop disabling the queue on suspend. In case we end up disabling it in ipa_start_xmit() (see the previous commit), we still arrange to start the TX queue on resume. Signed-off-by: Alex Elder --- drivers/net/ipa/ipa_modem.c | 2 -- 1 file changed, 2 deletions(-) -- 2.27.0 diff --git a/drivers/net/ipa/ipa_modem.c b/drivers/net/ipa/ipa_modem.c index aa1b483d9f7db..b176910d72868 100644 --- a/drivers/net/ipa/ipa_modem.c +++ b/drivers/net/ipa/ipa_modem.c @@ -214,8 +214,6 @@ void ipa_modem_suspend(struct net_device *netdev) if (!(netdev->flags & IFF_UP)) return; - netif_stop_queue(netdev); - ipa_endpoint_suspend_one(ipa->name_map[IPA_ENDPOINT_AP_MODEM_RX]); ipa_endpoint_suspend_one(ipa->name_map[IPA_ENDPOINT_AP_MODEM_TX]); }