Re: [LTP] [PATCH 2/4] tst_is_cwd: Add support for ramfs

[Michal Simek <michal.simek@petalogix.com>]

Hi,

> On Mon, Aug 3, 2009 at 6:24 AM, <michal.simek@petalogix.com> wrote:
>> From: Michal Simek <monstr@monstr.eu>
>>
>> I added support for testing ramfs and add together
>> tests for nfs, tmpfs and ramfs.
> 
> [...]
> 
> Hi Michal,
> 
> I'm looking at this code to solve another interesting problem, but
> just out of curiosity -- why did you #define the constants instead of
> using the definitions in sys/vfs.h? To ensure that the manpage was
> correct :)?

Not though about. Just find value and add it like for others.
But anyway for Ramfs I don't have that value in toolchain. For tmpfs and nfs I have.
From my quick look the problem seems to be that ramfs magic is not in linux/magic.h
but in fs/ramfs/inode.c which is wrong.
What interesting problem?

Michal

> 
> Thanks,
> -Garrett


-- 
Michal Simek, Ing. (M.Eng)
PetaLogix - Linux Solutions for a Reconfigurable World
w: www.petalogix.com p: +61-7-30090663,+42-0-721842854 f: +61-7-30090663

------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with 
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list