Re: [LTP] [PATCH] fcntl16.c patch

[chrubis@suse.cz]

Hi!
> >> The fcntl16 testcase is being compiled twice, once normal, and the other
> >> time with the _FILE_OFFSET_BITS=64 define set.
> >>
> >> When compiled with 64bit file functions (as fcntl16_64) all 
> >> file functions will automatically use their 64bit variants, e.g.
> >> "struct flock64" instead of "struct flock".
> >>
> >> So, when in the code e.g.
> >> 	fcntl(fd, F_SETLK, thislock)
> >> is called, this will automatically be translated into
> >> 	fcntl64(fd, F_SETLK64, thislock)
> >> and "thislock" needs to be of type "struct flock64".
> >>
> >> So, we need to make sure that thislock is of the correct type,
> >> with the start and len fields being of type "off_t" which then
> >> will be translated into "loff_t" during compilation when 
> >> _FILE_OFFSET_BITS=64 is set.
> >>
> >> This fixes the fcntl16_64 testcase on hppa64-linux-gnu with
> >> 32bit userspace running on 64bit kernel.
> >>
> >> Signed-off-by: Helge Deller <deller@gmx.de>
> > 
> > What about using struct flock as defined in bits/fcntl.h instead?
> 
> Sure, that's possible.
> Patch is attached.
> 
> I would be happy if you would apply one of those.

Applied, thanks.

And it would be easier for me if you had send git formatted patch with
headline and description (output from git format-patch).

-- 
Cyril Hrubis
chrubis@suse.cz

------------------------------------------------------------------------------
November Webinars for C, C++, Fortran Developers
Accelerate application performance with scalable programming models. Explore
techniques for threading, error checking, porting, and tuning. Get the most 
from the latest Intel processors and coprocessors. See abstracts and register
http://pubads.g.doubleclick.net/gampad/clk?id=60136231&iu=/4140/ostg.clktrk
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list