Re: [RFC/PATCH] Per-device parameter support (11/16)

From: Rusty Russell (rusty_at_rustcorp.com.au)
Date: 10/25/04

  • Next message: Rusty Russell: "Re: [RFC/PATCH] Per-device parameter support (13/16)"
    To: Tejun Heo <tj@home-tj.org>
    Date:	Mon, 25 Oct 2004 14:57:29 +1000
    
    

    On Sat, 2004-10-23 at 13:30 +0900, Tejun Heo wrote:
    > dp_11_module_param_arr.diff
    >
    > This is the 11st patch of 16 patches for devparam.
    >
    > The unsigned int * @nump of module_param_array is changed back to
    > unsigned int @num, and new sets of macros named module_param_arr*()
    > are added. These new macros don't take the num argument. This change
    > is made for two reasons
    >
    > 1. To be consistent with devparam macros. In devparam, we'll be
    > using field name of struct elements, so we won't be able to use
    > pointer argument.
    > 2. It's more consistent with other moduleparam macros.
    >
    > This patch only modifies moduleparam.h and doesn't modify the users
    > of the modified macros. The next patch takes care of that. This and
    > the next patch (dp_12_module_param_arr_apply.diff) are optional.

    It's finely balanced, but the module_param macro only skips the "&"
    because it turns it into a name. module_param_named() should probably
    be changed to take a ptr too. But you'll get a warning if you use the
    interface wrong, so it's not a huge issue.

    The balance is tipped here I think because the massive number of
    functions we're starting to sprout into.

    Rusty.

    -- 
    A bad analogy is like a leaky screwdriver -- Richard Braakman
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at  http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at  http://www.tux.org/lkml/
    

  • Next message: Rusty Russell: "Re: [RFC/PATCH] Per-device parameter support (13/16)"

    Relevant Pages

    • [4/7] fix loop termination condition in do_each_task_pid()/while_each_task_pid()
      ... This patch ... Now that the basic syntactic issues with the macros have been dealt ... the semantic issue of properly terminating the list iteration ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • [patch-kj] MIN/MAX removal drivers/char/rocket.c
      ... Patch removes unnecessary min/max macros and changes ... goto end; ... To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ ...
      (Linux-Kernel)
    • [PATCH 0/3] PM wakeup flags revisited
      ... This patch series fixes ... This series converts the macros to inline functions. ... All the patches in this series pass with flying colors. ... changes with a separate changelog. ...
      (Linux-Kernel)
    • Re: sendmail patch overwrites /etc/mail/sendmail.cf
      ... sendmail fixes don't only happen to the sendmail binaries. ... Sometimes fixes are made to to macros. ... patch does not contain any updated macros. ... the point is that if you look at the "overwrite" column and ...
      (comp.unix.solaris)
    • RE: [PATCH] [RESEND] Use of Performance Monitoring Counters based on Model number
      ... Sending the patch ... > 2) How about some macros for those magic numbers? ... >> Future processor implementation models may not support the ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)