From patchwork Wed Apr 1 13:14:54 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Ameya Palande <2ameya@gmail.com> X-Patchwork-Id: 15646 Received: from vger.kernel.org (vger.kernel.org [209.132.176.167]) by demeter.kernel.org (8.14.2/8.14.2) with ESMTP id n31DFGkg006994 for ; Wed, 1 Apr 2009 13:15:17 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758739AbZDANPA (ORCPT ); Wed, 1 Apr 2009 09:15:00 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1756983AbZDANPA (ORCPT ); Wed, 1 Apr 2009 09:15:00 -0400 Received: from mail-bw0-f169.google.com ([209.85.218.169]:55232 "EHLO mail-bw0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757933AbZDANO7 (ORCPT ); Wed, 1 Apr 2009 09:14:59 -0400 Received: by bwz17 with SMTP id 17so22817bwz.37 for ; Wed, 01 Apr 2009 06:14:54 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:in-reply-to:references :date:message-id:subject:from:to:content-type; bh=nc+hicyYQBcZJWhXRMUI6JuIX81mOIjpnVKo04nbLSw=; b=wiRA/XIty9/tYf2t4vm5VCb5dOxt5yOS+DUExJFZveOFLg/LSeHIjCwWHChRdn+Muq ghUo9mw8UZ/xezkVstF2F3JgNAbg9Zkc3OdLVBO9JuH7HSv61cnQf12s6ED20U1JGGWK 71Ov7rxsklo306RCLvJcwjZLkN32qjPG7O/a0= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; b=GBtl8on6U3YXtCOO3iYUtN5qCU7D/iwYUV+qQjgq2DypnExJU7uiZMn41RI5aCpaEh rEmfgJJsEQgbac8yX6Hqj2Haq/qL0D3slOAp7iE4grb9x/f4AGE0l9tW8ZD9KguTex6W sSTJs5XHgwbiq/bJFXJ7yDf2/rDvNnW1VaDW8= MIME-Version: 1.0 Received: by 10.204.57.13 with SMTP id a13mr2783887bkh.205.1238591694917; Wed, 01 Apr 2009 06:14:54 -0700 (PDT) In-Reply-To: <269694c00904010613g70a5de8eo5cac92865675ad0f@mail.gmail.com> References: <1238516811-11401-1-git-send-email-ameya.palande@nokia.com> <49D3111E.2060304@yandex.ru> <49D31218.8050200@yandex.ru> <269694c00904010613g70a5de8eo5cac92865675ad0f@mail.gmail.com> Date: Wed, 1 Apr 2009 16:14:54 +0300 Message-ID: <269694c00904010614k4a2897e6p8c5c5fad120738d9@mail.gmail.com> Subject: Re: [PATCH] DSPBRIDGE: Fix module build warnings From: Ameya Palande <2ameya@gmail.com> To: "linux-omap@vger.kernel.org" Sender: linux-omap-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-omap@vger.kernel.org On Wed, Apr 1, 2009 at 4:13 PM, Ameya Palande <2ameya@gmail.com> wrote: > > Hi Artem, > > On Wed, Apr 1, 2009 at 10:04 AM, Artem Bityutskiy wrote: >> >> Artem Bityutskiy wrote: >>> >>> Ameya Palande wrote: >>>> >>>> diff --git a/drivers/dsp/bridge/wmd/tiomap3430.c b/drivers/dsp/bridge/wmd/tiomap3430.c >>>> index 1fca3e5..b489da1 100644 >>>> --- a/drivers/dsp/bridge/wmd/tiomap3430.c >>>> +++ b/drivers/dsp/bridge/wmd/tiomap3430.c >>>> @@ -2034,26 +2034,27 @@ static DSP_STATUS PteSet(struct PgTableAttrs *pt, u32 pa, u32 va, >>>>             if (pfn_valid(__phys_to_pfn(patemp))) { >>>>                 pg = phys_to_page(patemp); >>>>                 get_page(pg); >>>> +                if (page_count(pg) <= 1) { >>>> +                    printk(KERN_EMERG "DSPBRIDGE:MAP  " >>>> +                        "function: COUNT 0 FOR PA " >>>> +                        "0x%x\n", patemp); >>>> +                    printk(KERN_EMERG "Bad page state" >>>> +                        "in process '%s'\n" >>>> +                        "page:%p flags:0x%0*lx " >>>> +                        "mapping:%p mapcount:%d " >>>> +                        "count:%d\n" >>>> +                        "Trying to fix it up, but " >>>> +                        "a reboot is needed\n" >>>> +                        "Backtrace:\n", >>>> +                        current->comm, pg, >>>> +                        (int)(2*sizeof(unsigned long)), >> >> Why do you need this (int) cast? Thiere is a %zd modifyer. >>>> >>>> +                        (unsigned long)pg->flags, >> >> What for do you have to cast here? > > I have attached a new patch which should take care of the ugly code. > > Cheers, > Ameya. Sorry about the last message. This one is in plain text. Cheers, Ameya. From 5b889f8c43c0d3af47cd4ed532a10beec863e370 Mon Sep 17 00:00:00 2001 From: Ameya Palande Date: Wed, 1 Apr 2009 16:05:13 +0300 Subject: [PATCH] [PATCH] DSPBRIDGE: Fix module build warnings From: Ameya Palande Signed-off-by: Ameya Palande --- drivers/dsp/bridge/rmgr/drv_interface.c | 6 ------ drivers/dsp/bridge/rmgr/node.c | 2 +- drivers/dsp/bridge/wmd/tiomap3430.c | 30 ++++++++++++------------------ 3 files changed, 13 insertions(+), 25 deletions(-) diff --git a/drivers/dsp/bridge/rmgr/drv_interface.c b/drivers/dsp/bridge/rmgr/drv_interface.c index 350e1b5..981354a 100755 --- a/drivers/dsp/bridge/rmgr/drv_interface.c +++ b/drivers/dsp/bridge/rmgr/drv_interface.c @@ -685,12 +685,6 @@ static int bridge_release(struct inode *ip, struct file *filp) return status; } -static void bridge_free(struct device *dev) -{ - /* nothing to Free */ -} - - /* This function provides IO interface to the bridge driver. */ static int bridge_ioctl(struct inode *ip, struct file *filp, unsigned int code, unsigned long args) diff --git a/drivers/dsp/bridge/rmgr/node.c b/drivers/dsp/bridge/rmgr/node.c index 51b4b12..178b802 100644 --- a/drivers/dsp/bridge/rmgr/node.c +++ b/drivers/dsp/bridge/rmgr/node.c @@ -3471,7 +3471,7 @@ static DSP_STATUS GetRMSFxns(struct NODE_MGR *hNodeMgr) } } } -func_end: + return status; } diff --git a/drivers/dsp/bridge/wmd/tiomap3430.c b/drivers/dsp/bridge/wmd/tiomap3430.c index 1fca3e5..9a535d0 100644 --- a/drivers/dsp/bridge/wmd/tiomap3430.c +++ b/drivers/dsp/bridge/wmd/tiomap3430.c @@ -2034,24 +2034,8 @@ static DSP_STATUS PteSet(struct PgTableAttrs *pt, u32 pa, u32 va, if (pfn_valid(__phys_to_pfn(patemp))) { pg = phys_to_page(patemp); get_page(pg); - } - if (page_count(pg) <= 1) { - printk(KERN_EMERG "DSPBRIDGE:MAP function: " - "COUNT 0 FOR PA 0x%x\n", patemp); - printk(KERN_EMERG "Bad page state" - KERN_EMERG "in process '%s'\n" - KERN_EMERG "page:%p flags:0x%0*lx " - KERN_EMERG "mapping:%p mapcount:%d " - KERN_EMERG "count:%d\n" - KERN_EMERG "Trying to fix it up, but " - KERN_EMERG "a reboot is needed\n" - KERN_EMERG "Backtrace:\n", - current->comm, pg, - (int)(2*sizeof(unsigned long)), - (unsigned long)pg->flags, pg->mapping, - page_mapcount(pg), page_count(pg)); - dump_stack(); - BUG_ON(1); + if (page_count(pg) <= 1) + goto pg_cnt_err_out; } patemp += HW_PAGE_SIZE_4KB; @@ -2126,6 +2110,16 @@ static DSP_STATUS PteSet(struct PgTableAttrs *pt, u32 pa, u32 va, } DBG_Trace(DBG_ENTER, "< PteSet status %x\n", status); return status; + +pg_cnt_err_out: + pr_emerg("DSPBRIDGE:MAP function: COUNT 0 FOR PA 0x%x\n", patemp); + pr_emerg("Bad page state in process '%s'\n page:%p flags:%lx " + "mapping:%p mapcount:%d count:%d\n", + current->comm, pg, pg->flags, + pg->mapping, page_mapcount(pg), page_count(pg)); + pr_emerg("Backtrace:\n"); + dump_stack(); + BUG_ON(1); } /* Memory map kernel VA -- memory allocated with vmalloc */ -- 1.6.2.1