Re: [Wireshark-commits] rev 51169: / /trunk/epan/: app_mem_usage.c

[Evan Huus <eapache () gmail com>]

On Tue, Aug 6, 2013 at 2:17 PM, Evan Huus <eapache () gmail com> wrote:
> On Tue, Aug 6, 2013 at 2:14 PM, Jakub Zawadzki
> <darkjames-ws () darkjames pl> wrote:
> > On Tue, Aug 06, 2013 at 10:59:42AM -0700, Guy Harris wrote:
> > > On Aug 6, 2013, at 9:38 AM, darkjames () wireshark org wrote:
> > >
> > > > http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=51169
> > > >
> > > > User: darkjames
> > > > Date: 2013/08/06 09:38 AM
> > > >
> > > > Log:
> > > > app_mem_usage fixes:
> > > >
> > > > - Add missing feature test macro for pread()
> > >
> > > Do all versions of the Linux kernel and of glibc have pread()?
> >
> > Probably, from man:
> >
> > The  pread()  and  pwrite() system calls were added to Linux in version 2.1.60
> > (...) C library support (including emulation using lseek(2) on older
> > kernels without the system calls) was added in glibc 2.1
> >
> > glibc 2.1 was released about 1999
>
> Hmm, we have conflicting man pages.
> http://linux.die.net/man/2/pread

Or not quite. The question is why the feature test macro is for glibc
2.12 and 2008, when the function existed long before that...

> > > If not, would an lseek() followed by a read() do just as well?
> >
> > Sure.
> > ___________________________________________________________________________
> > Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
> > Archives:    http://www.wireshark.org/lists/wireshark-dev
> > Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
> >              mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request () wireshark org?subject=unsubscribe