[{"id":1529,"web_url":"http://patchwork.ozlabs.org/comment/1529/","msgid":"<20080919172431.7be41b16.akpm@linux-foundation.org>","list_archive_url":null,"date":"2008-09-20T00:24:31","subject":"Re: [PATCH] gianfar: Free/iounmap memory after an error in mii bus\n\tinitialization","submitter":{"id":107,"url":"http://patchwork.ozlabs.org/api/people/107/","name":"Andrew Morton","email":"akpm@linux-foundation.org"},"content":"On Thu, 18 Sep 2008 16:37:54 -0500\nAndy Fleming <afleming@freescale.com> wrote:\n\n> Recent changes to MII bus initialization code added exit points which\n> didn't free or iounmap the bus before returning.\n> \n> This was reported as bug #11372\n> \n> Signed-off-by: Andy Fleming <afleming@freescale.com>\n\nadministrivia:\n\n- Adding a\n\n\tReported-by: Daniel Marjamki <danielm77@spray.se>\n\n  would be nice.\n\n- When referencing a bugzilla report in the changelog I like to use\n  the full URL.  Because there are people who periodically troll the\n  mainline commits closing off bugzilla reports.  Using the full\n  URL minimises the chance that they will miss something.","headers":{"Return-Path":"<netdev-owner@vger.kernel.org>","X-Original-To":"patchwork-incoming@ozlabs.org","Delivered-To":"patchwork-incoming@ozlabs.org","Received":["from vger.kernel.org (vger.kernel.org [209.132.176.167])\n\tby ozlabs.org (Postfix) with ESMTP id 01499DDF17\n\tfor <patchwork-incoming@ozlabs.org>;\n\tSat, 20 Sep 2008 10:25:22 +1000 (EST)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1752695AbYITAZS (ORCPT <rfc822;patchwork-incoming@ozlabs.org>);\n\tFri, 19 Sep 2008 20:25:18 -0400","(majordomo@vger.kernel.org) by vger.kernel.org id S1752642AbYITAZS\n\t(ORCPT <rfc822; netdev-outgoing>); Fri, 19 Sep 2008 20:25:18 -0400","from smtp1.linux-foundation.org ([140.211.169.13]:57188 \"EHLO\n\tsmtp1.linux-foundation.org\" rhost-flags-OK-OK-OK-OK)\n\tby vger.kernel.org with ESMTP id S1752011AbYITAZR (ORCPT\n\t<rfc822;netdev@vger.kernel.org>); Fri, 19 Sep 2008 20:25:17 -0400","from imap1.linux-foundation.org (imap1.linux-foundation.org\n\t[140.211.169.55])\n\tby smtp1.linux-foundation.org (8.14.2/8.13.5/Debian-3ubuntu1.1) with\n\tESMTP id m8K0OWxn024641\n\t(version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO);\n\tFri, 19 Sep 2008 17:24:33 -0700","from akpm.corp.google.com (localhost [127.0.0.1])\n\tby imap1.linux-foundation.org\n\t(8.13.5.20060308/8.13.5/Debian-3ubuntu1.1) with SMTP id\n\tm8K0OVqR014803; Fri, 19 Sep 2008 17:24:32 -0700"],"Date":"Fri, 19 Sep 2008 17:24:31 -0700","From":"Andrew Morton <akpm@linux-foundation.org>","To":"Andy Fleming <afleming@freescale.com>","Cc":"jeff@garzik.org, netdev@vger.kernel.org, danielm77@spray.se,\n\tafleming@freescale.com","Subject":"Re: [PATCH] gianfar: Free/iounmap memory after an error in mii bus\n\tinitialization","Message-Id":"<20080919172431.7be41b16.akpm@linux-foundation.org>","In-Reply-To":"<1221773874-3273-1-git-send-email-afleming@freescale.com>","References":"<1221773874-3273-1-git-send-email-afleming@freescale.com>","X-Mailer":"Sylpheed version 2.2.4 (GTK+ 2.8.20; i486-pc-linux-gnu)","Mime-Version":"1.0","Content-Type":"text/plain; charset=US-ASCII","Content-Transfer-Encoding":"7bit","X-Spam-Status":"No, hits=-4.856 required=5 tests=AWL, BAYES_00,\n\tOSDL_HEADER_SUBJECT_BRACKETED, PATCH_SUBJECT_OSDL","X-Spam-Checker-Version":"SpamAssassin 3.2.4-osdl_revision__1.47__","X-MIMEDefang-Filter":"lf$Revision: 1.188 $","X-Scanned-By":"MIMEDefang 2.63 on 140.211.169.13","Sender":"netdev-owner@vger.kernel.org","Precedence":"bulk","List-ID":"<netdev.vger.kernel.org>","X-Mailing-List":"netdev@vger.kernel.org"}}]