[lkml]   [2005]   [Oct]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [Patch 2/2] Special Memory (mspec) driver.
    On Thu, Oct 13, 2005 at 10:12:05PM -0700, Dave Hansen wrote:
    > On Wed, 2005-10-12 at 14:42 -0500, Robin Holt wrote:
    > ...
    > Looks like you could un-indent almost the entire function of you just
    > did this instead:
    > if (!atomic_dec_and_test(&vdata->refcnt))
    > return;


    > This looks pretty similar to get_one_pte_map(). Is there enough
    > commonality to use it?

    Added an extra patch to export get_one_pte_map and used that instead.

    > How about:
    > vdata = vmalloc(sizeof(struct vma_data) + pages * sizeof(long));
    > if (!vdata)
    > return -ENOMEM;


    > This whole thing really is a driver for a piece of arch-specific
    > hardware, right? Does it really belong in /proc? You already have a
    > misc device, so you already have some area in sysfs. Would that make a
    > better place for it?

    Most of the useful information for this was removed when the kernel
    uncached allocator (and gen_alloc) were moved out of the earliest mspec.c.
    Removed entirely.

    > Isn't the general kernel style for these to keep the action out of the
    > if() condition?
    > ret = misc_register(&cached_miscdev);
    > if (ret) {
    > ...


    Robin Holt
    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-10-14 20:13    [W:0.022 / U:133.684 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site