lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-Id: <20071017170106.5d9e0a9f.kristen.c.accardi@intel.com> Date: Wed, 17 Oct 2007 17:01:06 -0700 From: Kristen Carlson Accardi <kristen.c.accardi@...el.com> To: Mark Lord <lkml@....ca> Cc: Linux Kernel <linux-kernel@...r.kernel.org>, Greg KH <greg@...ah.com>, pcihpd-discuss@...ts.sourceforge.net, Andrew Morton <akpm@...ux-foundation.org>, Theodore Tso <tytso@....edu> Subject: Re: [PATCH 4/4] Fix PCIe hotplug for non-ACPI ExpressCard slots (version 2) On Tue, 16 Oct 2007 21:55:30 -0400 Mark Lord <lkml@....ca> wrote: > Make use of the previously split out pcie_init_enable_events() function > to reinitialize the hotplug hardware on resume from suspend, > but only when pciehp_force==1. Otherwise behaviour is unmodified. OK - definitely in this case the right thing to do is not use this code unless you are forcing pciehp, thanks. I think I'd be careful when you rename this patch - non-ACPI ExpressCard slots is not what you want to say, as this fix is very specific for your machine. We have no idea if this will fix any other problems that occur when people force pciehp - and in many cases this either may not be needed, or may not be enough. I only care about the name because I don't want people mislead when they are reading git commits. > > Signed-off-by: Mark Lord <mlord@...ox.com> > --- > --- old/drivers/pci/hotplug/pciehp_ctrl.c 2007-10-16 21:17:27.000000000 -0400 > +++ linux/drivers/pci/hotplug/pciehp_ctrl.c 2007-10-16 21:28:36.000000000 -0400 > @@ -37,7 +37,6 @@ > #include "pciehp.h" > > static void interrupt_event_handler(struct work_struct *work); > -static int pciehp_disable_slot(struct slot *p_slot); > > static int queue_interrupt_event(struct slot *p_slot, u32 event_type) > { > --- old/drivers/pci/hotplug/pciehp.h 2007-10-16 21:17:27.000000000 -0400 > +++ linux/drivers/pci/hotplug/pciehp.h 2007-10-16 21:28:42.000000000 -0400 > @@ -162,6 +162,8 @@ > extern void pciehp_queue_pushbutton_work(struct work_struct *work); > int pcie_init(struct controller *ctrl, struct pcie_device *dev); > int pciehp_enable_slot(struct slot *p_slot); > +int pciehp_disable_slot(struct slot *p_slot); > +int pcie_init_enable_events(struct controller *ctrl, struct pcie_device *dev); > > static inline struct slot *pciehp_find_slot(struct controller *ctrl, u8 device) > { > --- old/drivers/pci/hotplug/pciehp_core.c 2007-10-16 21:17:27.000000000 -0400 > +++ linux/drivers/pci/hotplug/pciehp_core.c 2007-10-16 21:30:19.000000000 -0400 > @@ -512,6 +512,24 @@ > static int pciehp_resume (struct pcie_device *dev) > { > printk("%s ENTRY\n", __FUNCTION__); > + if (pciehp_force) { > + struct pci_dev *pdev = dev->port; > + struct controller *ctrl = pci_get_drvdata(pdev); > + struct slot *t_slot; > + u8 status; > + > + /* reinitialize the chipset's event detection logic */ > + pcie_init_enable_events(ctrl, dev); > + > + t_slot = pciehp_find_slot(ctrl, ctrl->slot_device_offset); > + > + /* Check if slot is occupied */ > + t_slot->hpc_ops->get_adapter_status(t_slot, &status); > + if (status) > + pciehp_enable_slot(t_slot); > + else > + pciehp_disable_slot(t_slot); > + } > return 0; > } > #endif > - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists