From mboxrd@z Thu Jan 1 00:00:00 1970 From: KII Keiichi Subject: Re: [PATCH -mm 8/9] netconsole: Update documentation for dynamic reconfigurability Date: Sun, 08 Jul 2007 03:33:53 +0900 Message-ID: <468FDC91.2080300@bx.jp.nec.com> References: <20070704110739.28520.5036.sendpatchset@cselinux1.cse.iitk.ac.in> <20070704110819.28520.50445.sendpatchset@cselinux1.cse.iitk.ac.in> <468CEAEC.7040102@bx.jp.nec.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Linux Kernel Mailing List , Netdev , Joel Becker , Matt Mackall , Andrew Morton , David Miller To: Satyam Sharma Return-path: Received: from TYO202.gate.nec.co.jp ([202.32.8.206]:37546 "EHLO tyo202.gate.nec.co.jp" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751868AbXGGSgv (ORCPT ); Sat, 7 Jul 2007 14:36:51 -0400 In-Reply-To: Sender: netdev-owner@vger.kernel.org List-Id: netdev.vger.kernel.org Hi Satyam, > BTW I did some testing myself, and have found another *embarrassing* bug: > if netconsole is loaded _without_ specifying any "netconsole=" parameter, > module is still kept loaded, but on unloading configfs_unregister_...() > obviously panics! :-) This should've been found by me earlier, just that > I never tested without specifying the parameter to modprobe :-( I tested your patches on the x86 architecture and report the results. It is no problem except for the above problem. I think that it is useful to use configfs(especially, mkdir and rmdir because of adding/removing netconsole_target). I am going to test the your next iteration patches on x86 and IA64 architecutures. Thanks -- Keiichi KII NEC Corporation OSS Platform Development Division E-mail: k-keiichi@bx.jp.nec.com