Re: [PATCH ]cyclictest: fix invalid comment of debugfs mount directory.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jun 01, 2009 at 11:06:27AM +0900, GeunSik Lim wrote:
| 
| 
| Dear Clark Williams, 
| 
| Please, Fix the "debugfs/tracing" name in cyclictest.c source file. 
| 
| P.S: Now, version is differenct between shortlog(v0.39) and tags(v0.40). 
|      Maybe, You have to check version name of shortlog table. 
|      http://git.kernel.org/?p=linux/kernel/git/clrkwllms/rt-tests.git;a=summary
| 
| Thanks,
| GeunSik Lim. 
| 
| 
| 
| Subject: [PATCH ]cyclictest: Fix invalid comment of debugfs mount directory.
| 
| The "debugfs/tracing" name is invalid.
| Example,
| #> mount -t debugfs nodev /sys/kernel/debug [enter]
| #> ln -s /sys/kernel/debug /debug [enter]
| 
| Fix directory name normally from debugfs/tracing to debug/tracing.

GeunSik,

I would suggest changing your comment in the patch to something like:

	/* finds the path to the tracing dir inside debugfs mountpoint */

Because it is not written in the stone where debugfs should be mounted.
There is a suggested place, that already changed at least twice. So, the
function locates the debugfs mountpoint, wherever it is and looks for the
tracing directory.

Luis


|          Signed-off-by: GeunSik Lim <geunsik.lim@xxxxxxxxxxx>
| ---
|  src/cyclictest/cyclictest.c |    2 +-
|  1 files changed, 1 insertions(+), 1 deletions(-)
| 
| diff --git a/src/cyclictest/cyclictest.c b/src/cyclictest/cyclictest.c
| index be9a3f9..32a120a 100644
| --- a/src/cyclictest/cyclictest.c
| +++ b/src/cyclictest/cyclictest.c
| @@ -179,7 +179,7 @@ static char functiontracer[MAX_PATH];
|  static char traceroptions[MAX_PATH];
|  
|  /*
| - * Finds the path to the debugfs/tracing
| + * Finds the path to the debug/tracing
|   */
|  static int set_debugfileprefix(void)
|  {
| -- 
| 1.6.3.1

-- 
[ Luis Claudio R. Goncalves             Red Hat  -  Realtime Team ]
[ Fingerprint: 4FDD B8C4 3C59 34BD 8BE9  2696 7203 D980 A448 C8F8 ]

--
To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [RT Stable]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]

  Powered by Linux