All of lore.kernel.org
 help / color / mirror / Atom feed
From: "H. Peter Anvin" <hpa@zytor.com>
To: Herbert Xu <herbert@gondor.apana.org.au>
Cc: mmentovai@gmail.com, dash@vger.kernel.org
Subject: Re: Mac OS X support patch
Date: Sat, 12 Jul 2008 18:26:11 -0700	[thread overview]
Message-ID: <487959B3.1030301@zytor.com> (raw)
In-Reply-To: <E1KHq3f-0004Wb-00@gondolin.me.apana.org.au>

Herbert Xu wrote:
> H. Peter Anvin <hpa@zytor.com> wrote:
>> Mark Mentovai wrote:
>>> - open64 is not present although the stat64 family is, separate the
>>>   autoconf checks
>> This seems like the Wrong Thing anyway; the right thing should be to add 
>> whatever option (like -D_FILE_OFFSET_BITS=64 on Linux) to do 
>> *everything* using the full-sized offsets.
> 
> Actually there is a reason for this.  Not all open(2) calls in
> dash need large file support.  For example, doing it on a shell
> script or /dev/null makes no sense.  Only those dealing with user
> files (redirections) need large file support.
> 
> So that's we need to differentiate between open(2) calls on user
> files as opposed to those for dash itself.
> 

I'm not sure that that makes sense, however.  Unless you're hauling a 
*lot* of off_t's around the code, the incremental complexity is hardly 
worth it.

The *only* reason for the legacy 32-bit off_t mode is because it is an 
ABI change, it is necessary to prevent application/library interface 
breakage.

FWIW, klibc doesn't even implement the LFS64 APIs, nor does it implement 
the legacy 32-bit off_t mode in any way.

	-hpa

  reply	other threads:[~2008-07-13  1:26 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-11 21:20 Mac OS X support patch Mark Mentovai
2008-07-12  3:20 ` H. Peter Anvin
2008-07-12 14:51   ` Mark Mentovai
2008-07-13 11:24     ` Herbert Xu
2008-07-14 15:26       ` Mark Mentovai
2008-07-13  1:07   ` Herbert Xu
2008-07-13  1:26     ` H. Peter Anvin [this message]
2008-07-13  2:53       ` Herbert Xu
2009-01-13  4:19 ` Herbert Xu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=487959B3.1030301@zytor.com \
    --to=hpa@zytor.com \
    --cc=dash@vger.kernel.org \
    --cc=herbert@gondor.apana.org.au \
    --cc=mmentovai@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.