[lkml]   [2010]   [Jul]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [RFC 3/3] mm: iommu: The Virtual Contiguous Memory Manager

    On Thu, 1 Jul 2010, Randy Dunlap wrote:

    > > + * @start_addr The starting address of the VCM region.
    > > + * @len The len of the VCM region. This must be at least
    > > + * vcm_min() bytes.
    > and missing lots of struct members here.
    > If some of them are private, you can use:
    > /* private: */
    > ...
    > /* public: */
    > comments in the struct below and then don't add the private ones to the
    > kernel-doc notation above.

    To avoid wasting space in structures, it makes sense to place fields
    smaller than the alignment width together in the structure definition.
    If one were to do this and follow your proposal, some structures may need
    multiple "private" and "public" comments, which seems undesirable. The
    alternative, wasting memory, also seems undesirable. Perhaps you might
    have a proposal for a way to resolve this?

    - Paul

     \ /
      Last update: 2010-07-01 23:03    [W:0.019 / U:8.028 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site