[lkml]   [2002]   [Jul]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [BK PATCH 2.5] fs/binfmt_aout.c: Use PAGE_ALIGN_LL() on 64-bit values
    At 18:59 28/07/02, Eric W. Biederman wrote:
    >Anton Altaparmakov <> writes:
    > > Following from previous patch which introduced PAGE_ALIGN_LL, this
    > > one fixes a bug in fs/binfmt_aout.c which was using PAGE_ALIGN
    > > on 64-bit values... It now uses PAGE_ALIGN_LL.
    > >
    > > Patch together with the other two patches available from:
    > >
    > > bk pull
    >All virtual addresses on 32bit platforms are 32bit, as are all lengths
    >of address space.

    I thought (intel) CPUs did 48-bit addressing? How do we support 32GiB of
    RAM? With pure 32-bit addressing it would be limited to 4GiB only... No?
    (Of course I am probably confusing varius types of addresses...)

    >Unless you are running a 32bit kernel with a 64bit user space,
    >which is simply crazy, unless you are stuck doing it that way.

    The code is still broken. The values ARE 64-bit (check the struct
    definitions if you don't believe me). This is just a matter of correctness.
    It is incorrect as it is right now and it will do Bad Things(tm) if the
    supplied values are > 32-bit. (Whether it is possible for a malicious user
    to supply them is beyond my knowledge.)

    Best regards,


    "I've not lost my mind. It's backed up on tape somewhere." - Unknown
    Anton Altaparmakov <aia21 at> (replace at with @)
    Linux NTFS Maintainer / IRC: #ntfs on
    WWW: &

    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2005-03-22 13:27    [W:0.022 / U:9.324 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site