From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754684AbXGGShN (ORCPT ); Sat, 7 Jul 2007 14:37:13 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752692AbXGGSgw (ORCPT ); Sat, 7 Jul 2007 14:36:52 -0400 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 Message-ID: <468FDC91.2080300@bx.jp.nec.com> Date: Sun, 08 Jul 2007 03:33:53 +0900 From: KII Keiichi User-Agent: Thunderbird 2.0.0.4 (Windows/20070604) MIME-Version: 1.0 To: Satyam Sharma CC: Linux Kernel Mailing List , Netdev , Joel Becker , Matt Mackall , Andrew Morton , David Miller Subject: Re: [PATCH -mm 8/9] netconsole: Update documentation for dynamic reconfigurability 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> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@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