[lkml]   [2005]   [Sep]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 2/25] NTFS: Allow highmem kmalloc() in ntfs_malloc_nofs() and add _nofail() version.
    On Fri, 2005-09-09 at 15:02 +0300, Pekka J Enberg wrote:
    > On Fri, 9 Sep 2005, Anton Altaparmakov wrote:
    > > > I completely disagree with you given that this is not "inventing [...]
    > > > own memory allocators", it is just a convenient short hand. I am sure a
    > > > lot of people would agree with you though. It is just a matter of
    > > > personal preference.
    > >
    > > I should add that this is not ntfs only, the idea is from another file
    > > system which uses it, too. Can't remember which one it was, though (xfs
    > > maybe?).
    > Indeed. It is not just a matter of personal preference but also a matter
    > of subsystems introducing duplicate code like this. Quick grepping shows
    > UDF doing same thing and XFS doing slightly differently but I am pretty
    > sure I've seen it elsewhere too.

    Yes, that is usually a good indication that a generic function should be
    provided. However having a generic function with complicated and long
    arguments is no use as everyone will want their own shorter one anyway.
    And given the function is static inline it actually makes no difference
    to the generated code size. Also calling it __vmalloc_fast makes no
    sense as it doesn't always use vmalloc... Given we have kmalloc and
    vmalloc maybe it should be just malloc?

    Obviously if there were a suitable generic function I would use it but I
    and I imagine all the other users would still wrap it with the old name.

    Best regards,

    Anton Altaparmakov <aia21 at> (replace at with @)
    Unix Support, Computing Service, University of Cambridge, CB2 3QH, UK
    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-09-09 14:11    [W:0.026 / U:9.792 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site