From patchwork Mon Mar 15 14:48:05 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Karsten Keil X-Patchwork-Id: 47762 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 43B08B7D85 for ; Tue, 16 Mar 2010 01:57:35 +1100 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965140Ab0COO46 (ORCPT ); Mon, 15 Mar 2010 10:56:58 -0400 Received: from moutng.kundenserver.de ([212.227.17.10]:62949 "EHLO moutng.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S964955Ab0COO4z (ORCPT ); Mon, 15 Mar 2010 10:56:55 -0400 Received: from pingi (p5497DF17.dip.t-dialin.net [84.151.223.23]) by mrelayeu.kundenserver.de (node=mrbap1) with ESMTP (Nemesis) id 0MDW85-1NsbxF49Ph-00H2HC; Mon, 15 Mar 2010 15:56:49 +0100 MIME-Version: 1.0 Subject: Re: [3/3] gigaset: prune use of tty_buffer_request_room Date: Mon, 15 Mar 2010 14:48:05 -0000 To: Tilman Schmidt From: Karsten Keil Cc: David Miller , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org, isdn4linux , Hansjoerg Lipp , i4ldeveloper , Alan Cox Message-Id: <1268665009.25656@pingi> X-Provags-ID: V01U2FsdGVkX1+le6qrdV5s2Y8sEKLp63xzyXEuACHw8TmaO47 BkIo4dHpIWzq5XdZhQTChlzGF0IaZeU4/l3ggbyO6Cvpi5leVN zakWQuHOihcCn2kDpaIJ8geMXK4t+QB Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org Calling tty_buffer_request_room() before tty_insert_flip_string() is unnecessary, costs CPU and for big buffers can mess up the multi-page allocation avoidance. Signed-off-by: Tilman Schmidt Acked-by: Karsten Keil --- Note to -stable: applies correctly to 2.6.33 with fuzz 2. drivers/isdn/gigaset/interface.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/drivers/isdn/gigaset/interface.c b/drivers/isdn/gigaset/interface.c index a1bcbc2..f0dc6c9 100644 --- a/drivers/isdn/gigaset/interface.c +++ b/drivers/isdn/gigaset/interface.c @@ -628,7 +628,6 @@ void gigaset_if_receive(struct cardstate *cs, if (tty == NULL) gig_dbg(DEBUG_IF, "receive on closed device"); else { - tty_buffer_request_room(tty, len); tty_insert_flip_string(tty, buffer, len); tty_flip_buffer_push(tty); }