Re: [PATCH -mm 8/9] netconsole: Update documentation for dynamic reconfigurability

2007-07-07 Thread Satyam Sharma
Hi Keiichi, On Sun, 8 Jul 2007, KII Keiichi wrote: > [...] > 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

Re: [PATCH -mm 8/9] netconsole: Update documentation for dynamic reconfigurability

2007-07-07 Thread KII Keiichi
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, ju

Re: [PATCH -mm 8/9] netconsole: Update documentation for dynamic reconfigurability

2007-07-05 Thread Satyam Sharma
On Thu, 5 Jul 2007, Satyam Sharma wrote: > [...] > 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! :-)

Re: [PATCH -mm 8/9] netconsole: Update documentation for dynamic reconfigurability

2007-07-05 Thread Satyam Sharma
Hi Keiichi, On Thu, 5 Jul 2007, Keiichi KII wrote: > Hi Satyam, > > > + cat enabled # check if enabled is 1 > > + echo 0 > enabled # disable the target (if required) > > + echo eth2 > dev_name # set local interface > > I think t

Re: [PATCH -mm 8/9] netconsole: Update documentation for dynamic reconfigurability

2007-07-05 Thread Keiichi KII
Hi Satyam, > + cat enabled # check if enabled is 1 > + echo 0 > enabled# disable the target (if required) > + echo eth2 > dev_name# set local interface I think that the above line should change from "echo eth2 > dev_name" to "ec

Re: [PATCH -mm 8/9] netconsole: Update documentation for dynamic reconfigurability

2007-07-05 Thread Satyam Sharma
On Wed, 4 Jul 2007, Joel Becker wrote: > On Wed, Jul 04, 2007 at 04:38:19PM +0530, Satyam Sharma wrote: > > +Some examples follow (configfs is mounted at the /config mountpoint, say). > > configfs is generally expected to be mounted at > /sys/kernel/config, and it even creates that mountpoin

Re: [PATCH -mm 8/9] netconsole: Update documentation for dynamic reconfigurability

2007-07-04 Thread Joel Becker
On Wed, Jul 04, 2007 at 04:38:19PM +0530, Satyam Sharma wrote: > +Some examples follow (configfs is mounted at the /config mountpoint, say). configfs is generally expected to be mounted at /sys/kernel/config, and it even creates that mountpoint for itself. The documentation should probabl

[PATCH -mm 8/9] netconsole: Update documentation for dynamic reconfigurability

2007-07-04 Thread Satyam Sharma
From: Satyam Sharma <[EMAIL PROTECTED]> [8/9] netconsole: Update documentation for dynamic reconfigurability Add myself to parties interested in receiving bug reports, and give lots of examples. Also fix some whitespace inconsistencies I introduced earlier. Signed-off-by: Satyam Sharma <[EMAIL P