Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758016AbcCUVCi (ORCPT ); Mon, 21 Mar 2016 17:02:38 -0400 Received: from iolanthe.rowland.org ([192.131.102.54]:36356 "HELO iolanthe.rowland.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1757314AbcCUVCg (ORCPT ); Mon, 21 Mar 2016 17:02:36 -0400 Date: Mon, 21 Mar 2016 17:02:30 -0400 (EDT) From: Alan Stern X-X-Sender: stern@iolanthe.rowland.org To: Woojung.Huh@microchip.com cc: oneukum@suse.com, , , , , , , , , Subject: RE: [PATCH] lan78xx: Protect runtime_auto check by #ifdef CONFIG_PM In-Reply-To: <9235D6609DB808459E95D78E17F2E43D404CC1EF@CHN-SV-EXMX02.mchp-main.com> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1088 Lines: 25 On Mon, 21 Mar 2016 Woojung.Huh@microchip.com wrote: > > > > But this leaves open the issue that querying the device too often will > > > > prevent it from going into autosuspend. It seems to me that the best > > > > way to deal with this is to make sure that the autosuspend timeout is > > > > shorter than the interal between queries, not to make the querying > > > > conditional on !runtime_auto. > > > > > > Short autosuspend timeout can affect performance. For instance our > > experiments showed that > > > shorter than 10sec timeout made Ethernet performance degrade because > > of wakeup delays. > > > So, just putting shorter timeout may have some side effects. > > > > Sure. This just means that you need a long statistics interval -- > > longer than the autosuspend timeout. That's why I suggested making the > > interval adjustable. > > What do you mean statistics interval? > Interval calling ndo_get_stats64 or another thread/timer or else getting statistics? The time between calls to ndo_get_stats64, since that's the routine which queries the device. Alan Stern