linux-kernel - Re: [PATCH 4/4] Fix PCIe hotplug for non-ACPI ExpressCard slots (version 2)

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

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/