Re: [PATCH 3/6]: Migrate to new naming scheme, using macro to declare cache

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Quoting Ian McDonald:
|  On 6/17/07, Gerrit Renker <gerrit@xxxxxxxxxxxxxx> wrote:
|  > Ian McDonald wrote:
|  > |  > [TFRC]: Migrate to new naming scheme, using macro to declare cache
|  > |  >
|  > |  I agree totally with the intent. I reworked li_hist as well but
|  > |  Arnaldo decided to do in a better way and I think he'll want the same
|  > |  for this also.
|  > Is this an Ack?
|  
|  Not yet - it's saying look at his patch and see if you want to match
|  his style...
Difficult to see what is meant. I think that my set mirrors in a way what Arnaldo
does in the spirit, but not in detail. Since in the end the approach is different.
But the principles are similar: 
 * also use explicit allocation/deallocation routines 
 * also put the slab allocation into the module, as Arnaldos patch 
 * effectively the same functions are removed in my changeset
 * as discussed earlier, modularity of calc_first_li is achieved by using a function pointer.
 
==> wonder whether I should add `__read_mostly' to the slab declaration, but that is 
    a smaller issue. I will think about this.

The main difference is that in my changeset further functionality is implemented which
intersects with the work Arnaldo has been doing. 

|  >
|  > |  See his patch (which is in Dave M's tree now) at:
|  > |  http://www.mail-archive.com/dccp@xxxxxxxxxxxxxxx/msg01879.html
|  > |
-
To unsubscribe from this list: send the line "unsubscribe dccp" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel]     [IETF DCCP]     [Linux Networking]     [Git]     [Security]     [Linux Assembly]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]

  Powered by Linux