Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp1846012imu; Sat, 5 Jan 2019 08:14:36 -0800 (PST) X-Google-Smtp-Source: ALg8bN7/ITJuaKIiVCg0NStRcHeGFIJeS2QoB529fE02N/Jxcfxi759Uh1PiFVfWLdcMQ2p2tA+1 X-Received: by 2002:a63:7154:: with SMTP id b20mr5207594pgn.342.1546704876597; Sat, 05 Jan 2019 08:14:36 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1546704876; cv=none; d=google.com; s=arc-20160816; b=MwXZVTqFrGUQb6zSn9Oz6kB7H4kxj7bJzqrW0VaQjI6dIw5XhsJCJ0jbmXUEejfWKq 5lFPnRhdIbUD+6yLWMGyCXHFtrqq63EUWuSKUEV00D6XS/wt8cJvVmbk/8/DjA3VdIQq 7XHCM90d5Ye/e0F4YpP8Qb+4B5q0f47mVK1N8R/JxxflqX6mTyngFYvh4dSRD8bAjAIC Ojj3Pyh6FOWcYIX++JwU2qeHYVRdP5Rn7U3pJi0Ze7zE+FM8VBhSD8N5VrJ8vbKUNF+v ZFjg9nhVkQw2mVj2KpefWZHBs9HbiRrc9zZgp8izbcrVKtUKEYcgbldGMU7a21HxGb7g wmAg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:to:subject:dkim-signature; bh=DEUAb1t8Umvmxupcvpas8QpKculGCGn3a+HixUXZcYE=; b=Pnzb4Z3Qb9YDivFUfLRyi+S13ykGtvDUWXnOmpSfGySIITo9npIp5ldJTkTlGuE7lG Vlf6xLDEJZayZfOkJyJnmwMUA5NzKwrvCv/d/kw3vJgUmOJcUFmZ+vKbqDw1ROoKqaf/ +gJtn6wmZp6/PDCD36nYmwsNNwU3+3+dBRdFxoPYdWNgzSzriW+OKWhLu9ximJGkg0oT 3QSh0bRwT+dsOdtV1i1BNuA8LlPIYPIfYh9qg8CFDT6xil9uG4+bScJFiMgwzQGvPR3x hDT2+PzHSEPXJLKa9bQgu2WDjhytSbVI5K3uCpIE6s2rHJPVv3ZaJgNAB++xtp6L49EP ahlw== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@gmail.com header.s=20161025 header.b=pqiuBnnb; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id v189si12603148pgb.398.2019.01.05.08.14.21; Sat, 05 Jan 2019 08:14:36 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=fail header.i=@gmail.com header.s=20161025 header.b=pqiuBnnb; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726361AbfAEQNP (ORCPT + 99 others); Sat, 5 Jan 2019 11:13:15 -0500 Received: from mail-ot1-f66.google.com ([209.85.210.66]:46599 "EHLO mail-ot1-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726234AbfAEQNP (ORCPT ); Sat, 5 Jan 2019 11:13:15 -0500 Received: by mail-ot1-f66.google.com with SMTP id w25so34481635otm.13; Sat, 05 Jan 2019 08:13:14 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:subject:to:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=DEUAb1t8Umvmxupcvpas8QpKculGCGn3a+HixUXZcYE=; b=pqiuBnnbMgvd6Gwa4RhyYs78YFcN/4RqiA0yzjx4UhsrL8mWfTGrxBK3OldaVThmBK WRUDlnGublVZH4TyCw0kahmfH9u6BswAFD+I9YwFF9d8O6X8T4XNvqHyZDSFccuFmy+B k0WKn4xe1aGJihgUrqTYGrvo55zdlxOqhpkUvCAMdsj+PwY/AX3Gp51ezMgtQ3K348FG 4xg6529ZphPNztTJQmcbIF95asE79f11LwcCaaQWYZNHZ98mCoJ+8Md08oOXamA8nJYx elRPSoev22A4p4e321ubZ7l0pO39Xgc0Yhrp6Y9Dg7JZdr8M4jSyJe1gqUokPi8/6aBb SGew== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:subject:to:references:from:message-id :date:user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=DEUAb1t8Umvmxupcvpas8QpKculGCGn3a+HixUXZcYE=; b=JL3znTbt/Ud4Kb4S7w72zMN6csnEEMTedn/w+c3wkDfkhWvzhFArb3WIwfpyPc8x17 cKfqcZvTtH/JSC6lD8YtzMLPjqH4qU0UT9cQQgPP0IQNwbYKZy1SQdLmvGiFAh7r8UQe 3bcLL/YULLXwXehcPKrs3ZYWtRk6rwGsI88e3FqBHP17UfxhtZ6W/LqodtZDyUpv/idb U6CAPjHS7Ka6sr/aXOnyEZ0krMtXw3d9DeC7qaS2Hip0f+HAqz3zJxwYLSdeUnqogGaa HLe9liKqGOOzPq2Dv7wOg9QYskFqAbHHVQAHNO5G9PWoibDOYudn0DG0KSRBmbItkJ1C IMcA== X-Gm-Message-State: AJcUukcoOV6G1XL6muO7pS4wraBGa2un9O2/9T5GQ/sfrkQmX1NeFeyh DXZdlHtmBK5niJZUej/FEZMgM+b+ X-Received: by 2002:a9d:1e86:: with SMTP id n6mr41226515otn.9.1546704793638; Sat, 05 Jan 2019 08:13:13 -0800 (PST) Received: from [192.168.1.102] (cpe-24-31-245-230.kc.res.rr.com. [24.31.245.230]) by smtp.gmail.com with ESMTPSA id b18sm35362076oii.51.2019.01.05.08.13.12 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sat, 05 Jan 2019 08:13:13 -0800 (PST) Subject: Re: [PATCH 1/4] rtlwifi: rtl8723ae: Take the FW LPS mode handling out To: Bernd Edlinger , Ping-Ke Shih , Kalle Valo , "David S. Miller" , "linux-wireless@vger.kernel.org" , "netdev@vger.kernel.org" , "linux-kernel@vger.kernel.org" References: From: Larry Finger Message-ID: <7c1f25bc-dfc6-56e2-745c-4607f0f41159@lwfinger.net> Date: Sat, 5 Jan 2019 10:13:12 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.3.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 1/5/19 5:31 AM, Bernd Edlinger wrote: > On 1/5/19 3:44 AM, Larry Finger wrote: >> On 1/4/19 6:48 AM, Bernd Edlinger wrote: >>> This appears to trigger a firmware bug and causes severe >>> problems with rtl8723ae PCI devices. >>> >>> When the power save mode is activated for longer periods >>> of time the firmware stops to receive any packets. >>> >>> This problem was exposed by commit 873ffe154ae0 ("rtlwifi: >>> Fix logic error in enter/exit power-save mode"). >>> >>> Previously the power save mode was only active rarely and >>> only for a short time so that the problem was not noticeable. >>> >>> Signed-off-by: Bernd Edlinger >>> --- >> >> While the Realtek firmware group has a chance to look for a bug, I would like you to perform a couple of tests on the original code. >> >> The driver has three module parameters that affect power save. The 'modinfo rtl8723ae' command lists them as >> >> parm:   ips:Set to 0 to not use link power save (default 1) (bool) >> parm:   swlps:Set to 1 to use SW control power save (default 0) (bool) >> parm:   fwlps:Set to 1 to use FW control power save (default 1) (bool) >> >> If you were to load rtl8723ae with 'ips=0', does it still fail? >> If you were to load the driver with 'swlps=1 fwlps=0', does it still fail? >> > > this does not work: > > modprobe rtl8723ae debug_mask=0xFFFFFFFF debug_level=5 ips=0 > > tail -f /var/log/syslog|grep "AP off" > Jan 5 11:42:06 w-ed kernel: [ 7267.229713] rtlwifi: :<0> AP off for 2 s > Jan 5 11:42:08 w-ed kernel: [ 7269.276761] rtlwifi: :<0> AP off for 4 s > Jan 5 11:42:10 w-ed kernel: [ 7271.323758] rtlwifi: :<0> AP off for 6 s > Jan 5 11:42:12 w-ed kernel: [ 7273.370759] rtlwifi: :<0> AP off for 8 s > Jan 5 11:42:14 w-ed kernel: [ 7275.417753] rtlwifi: :<0> AP off for 10 s > Jan 5 11:42:14 w-ed kernel: [ 7275.417754] rtlwifi: AP off, try to reconnect now > Jan 5 11:42:28 w-ed kernel: [ 7289.746676] rtlwifi: :<0> AP off for 2 s > Jan 5 11:42:40 w-ed kernel: [ 7302.028327] rtlwifi: :<0> AP off for 2 s > Jan 5 11:42:43 w-ed kernel: [ 7304.075327] rtlwifi: :<0> AP off for 4 s > Jan 5 11:42:45 w-ed kernel: [ 7306.122330] rtlwifi: :<0> AP off for 6 s > Jan 5 11:42:47 w-ed kernel: [ 7308.169292] rtlwifi: :<0> AP off for 8 s > Jan 5 11:42:49 w-ed kernel: [ 7310.216236] rtlwifi: :<0> AP off for 10 s > Jan 5 11:42:49 w-ed kernel: [ 7310.216238] rtlwifi: AP off, try to reconnect now > Jan 5 11:43:05 w-ed kernel: [ 7326.592222] rtlwifi: :<0> AP off for 2 s > Jan 5 11:43:07 w-ed kernel: [ 7328.639076] rtlwifi: :<0> AP off for 4 s > Jan 5 11:43:09 w-ed kernel: [ 7330.686220] rtlwifi: :<0> AP off for 6 s > Jan 5 11:43:11 w-ed kernel: [ 7332.733078] rtlwifi: :<0> AP off for 8 s > Jan 5 11:43:13 w-ed kernel: [ 7334.779988] rtlwifi: :<0> AP off for 10 s > Jan 5 11:43:13 w-ed kernel: [ 7334.779989] rtlwifi: AP off, try to reconnect now > Jan 5 11:43:28 w-ed kernel: [ 7349.108839] rtlwifi: :<0> AP off for 2 s > Jan 5 11:43:30 w-ed kernel: [ 7351.155837] rtlwifi: :<0> AP off for 4 s > Jan 5 11:43:32 w-ed kernel: [ 7353.202838] rtlwifi: :<0> AP off for 6 s > Jan 5 11:43:42 w-ed kernel: [ 7363.437779] rtlwifi: :<0> AP off for 2 s > Jan 5 11:43:46 w-ed kernel: [ 7367.531622] rtlwifi: :<0> AP off for 2 s > Jan 5 11:43:48 w-ed kernel: [ 7369.578597] rtlwifi: :<0> AP off for 4 s > Jan 5 11:43:50 w-ed kernel: [ 7371.625694] rtlwifi: :<0> AP off for 6 s > Jan 5 11:43:52 w-ed kernel: [ 7373.672691] rtlwifi: :<0> AP off for 8 s > Jan 5 11:43:54 w-ed kernel: [ 7375.719690] rtlwifi: :<0> AP off for 10 s > Jan 5 11:43:54 w-ed kernel: [ 7375.719691] rtlwifi: AP off, try to reconnect now > Jan 5 11:44:09 w-ed kernel: [ 7390.048406] rtlwifi: :<0> AP off for 2 s > Jan 5 11:44:11 w-ed kernel: [ 7392.095678] rtlwifi: :<0> AP off for 4 s > Jan 5 11:44:13 w-ed kernel: [ 7394.142349] rtlwifi: :<0> AP off for 6 s > Jan 5 11:44:15 w-ed kernel: [ 7396.189352] rtlwifi: :<0> AP off for 8 s > Jan 5 11:44:17 w-ed kernel: [ 7398.236352] rtlwifi: :<0> AP off for 10 s > Jan 5 11:44:17 w-ed kernel: [ 7398.236353] rtlwifi: AP off, try to reconnect now > Jan 5 11:44:31 w-ed kernel: [ 7412.565079] rtlwifi: :<0> AP off for 2 s > Jan 5 11:44:33 w-ed kernel: [ 7414.612167] rtlwifi: :<0> AP off for 4 s > Jan 5 11:44:35 w-ed kernel: [ 7416.659101] rtlwifi: :<0> AP off for 6 s > Jan 5 11:44:37 w-ed kernel: [ 7418.706035] rtlwifi: :<0> AP off for 8 s > Jan 5 11:44:39 w-ed kernel: [ 7420.753100] rtlwifi: :<0> AP off for 10 s > Jan 5 11:44:39 w-ed kernel: [ 7420.753101] rtlwifi: AP off, try to reconnect now > Jan 5 11:44:54 w-ed kernel: [ 7435.081860] rtlwifi: :<0> AP off for 2 s > Jan 5 11:44:56 w-ed kernel: [ 7437.128857] rtlwifi: :<0> AP off for 4 s > Jan 5 11:45:08 w-ed kernel: [ 7449.410653] rtlwifi: :<0> AP off for 2 s > Jan 5 11:45:10 w-ed kernel: [ 7451.457650] rtlwifi: :<0> AP off for 4 s > Jan 5 11:45:12 w-ed kernel: [ 7453.504647] rtlwifi: :<0> AP off for 6 s > Jan 5 11:45:14 w-ed kernel: [ 7455.551607] rtlwifi: :<0> AP off for 8 s > Jan 5 11:45:16 w-ed kernel: [ 7457.598645] rtlwifi: :<0> AP off for 10 s > Jan 5 11:45:16 w-ed kernel: [ 7457.598646] rtlwifi: AP off, try to reconnect now > > > but this works: > > modprobe rtl8723ae debug_mask=0xFFFFFFFF debug_level=5 swlps=1 fwlps=0 Yes, I think that is a better thing to do now. If and when Realtek finds a firmware bug, and when the new firmware is readily available, then there will not be a lot of code to reinstall. Thanks for testing, Larry