Re: [PATCH 16/22] advansys: Eliminate prototypes

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

 



Matthew Wilcox wrote:
Rearrange a lot of the functions in the file to get rid of all the forward
declarations.

What thoughts, if any, have been given to post-patch code arrangement, besides eliminating prototypes?

This always struck me as a counterproductive exercise when taken by itself. Sure, prototypes were eliminated, but was code packed together in an efficient way afterwards? Ideally hot path code should be close together, ditto for various other things like error handling code.

Without further explanation or details about your post-patch analysis, how do we even know the new code arrangement doesn't negatively impact i-cache overall?

	Jeff


-
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux