lkml.org 
[lkml]   [2004]   [Jul]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    SubjectRe: [PATCH] More PCI fixes for 2.6.7
    Date
    From
    ChangeSet 1.1788, 2004/07/02 13:35:57-07:00, linas@austin.ibm.com

    [PATCH] PCI Hotplug: RPAPHP structure size/performance

    Please review and apply the following patch if you find it agreeable.

    This patch does not make any functional changes, but does improve
    both performance and memory usage by rearranging structure elements.

    The need for these changes became appearent during a code review of
    the disassembly involving this structure. The memory footprint of this
    structure is made smaller by grouping the byte fields next to each other.
    The access of the list_head can be simplified by making it the first element
    of the structure, thus avoiding a needless add-immediate without negatively
    impacting any of the other accesses.

    Signed-off-by: Linas Vepstas <linas@linas.org>
    Signed-off-by: Greg Kroah-Hartman <greg@kroah.com>


    drivers/pci/hotplug/rpaphp.h | 4 ++--
    1 files changed, 2 insertions(+), 2 deletions(-)


    diff -Nru a/drivers/pci/hotplug/rpaphp.h b/drivers/pci/hotplug/rpaphp.h
    --- a/drivers/pci/hotplug/rpaphp.h 2004-07-02 14:24:19 -07:00
    +++ b/drivers/pci/hotplug/rpaphp.h 2004-07-02 14:24:19 -07:00
    @@ -85,6 +85,7 @@
    * struct slot - slot information for each *physical* slot
    */
    struct slot {
    + struct list_head rpaphp_slot_list;
    int state;
    u32 index;
    u32 type;
    @@ -92,6 +93,7 @@
    char *name;
    char *location;
    u8 removable;
    + u8 dev_type; /* VIO or PCI */
    struct device_node *dn; /* slot's device_node in OFDT */
    /* dn has phb info */
    struct pci_dev *bridge; /* slot's pci_dev in pci_devices */
    @@ -99,9 +101,7 @@
    struct list_head pci_funcs; /* pci_devs in PCI slot */
    struct vio_dev *vio_dev; /* vio_dev in VIO slot */
    } dev;
    - u8 dev_type; /* VIO or PCI */
    struct hotplug_slot *hotplug_slot;
    - struct list_head rpaphp_slot_list;
    };

    extern struct hotplug_slot_ops rpaphp_hotplug_slot_ops;
    -
    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/

    \
     
     \ /
      Last update: 2005-03-22 14:04    [W:0.025 / U:60.208 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site