linux-kernel - Re: [PATCH v4 0/8] Allow Link state changes for Hot-Plug

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: <20140211011816.GA15412@google.com>
Date:	Mon, 10 Feb 2014 18:18:16 -0700
From:	Bjorn Helgaas <bhelgaas@...gle.com>
To:	Rajat Jain <rajatjain@...iper.net>
Cc:	Rajat Jain <rajatxjain@...il.com>,
	"Rafael J. Wysocki" <rafael.j.wysocki@...el.com>,
	Kenji Kaneshige <kaneshige.kenji@...fujitsu.com>,
	Alex Williamson <alex.williamson@...hat.com>,
	Yijing Wang <wangyijing@...wei.com>,
	"linux-pci@...r.kernel.org" <linux-pci@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Yinghai Lu <yhlu.kernel@...il.com>,
	Guenter Roeck <groeck@...iper.net>
Subject: Re: [PATCH v4 0/8] Allow Link state changes for Hot-Plug

On Wed, Feb 05, 2014 at 03:15:19AM +0000, Rajat Jain wrote:
> Hello list,
> 
> My apologies for I forgot to attach the version history information of each individual patch.
> 
> Essentially the v4 is exactly same as v3, except that it is rebased on top of 3.14-rc1.
> 
> Thanks,
> 
> Rajat
> 
> > -----Original Message-----
> > From: Rajat Jain [mailto:rajatxjain@...il.com]
> > Sent: Tuesday, February 04, 2014 6:28 PM
> > To: Bjorn Helgaas; Rafael J. Wysocki; Kenji Kaneshige; Alex Williamson;
> > Yijing Wang; linux-pci@...r.kernel.org; linux-kernel@...r.kernel.org;
> > Yinghai Lu
> > Cc: Guenter Roeck; Rajat Jain; Rajat Jain
> > Subject: [PATCH v4 0/8] Allow Link state changes for Hot-Plug
> > 
> > Hello,
> > 
> > This patch set enables the use of PCI Express link up and link down
> > events for Hotplug or Unplug. This is the pretty much the same patchset
> > as v3, only resending after rebasing on top of 3.14-rc1.
> > 
> > (As a side effect, this patch also fixes the bug
> > https://bugzilla.kernel.org/show_bug.cgi?id=65521 )
> > 
> > I'd appreciate if you could please review and provide me with any
> > comments.
> > 
> > Thanks,
> > 
> > Rajat
> > 
> > Rajat Jain (8):
> >   pciehp: Make check_link_active() non-static
> >   pciehp: Use link change notifications for hot-plug and removal
> >   pciehp: Enable link state change notifications
> >   pciehp: Don't disable the link permanently, during removal
> >   pciehp: Don't check for adapter or latch status while disabling
> >   pciehp: Disabling the link notification across slot reset
> >   pciehp: Ensure very fast hotplug events are also processed.
> >   pciehp: Introduce hotplug_lock to serialize HP events
> > 
> >  drivers/pci/hotplug/pciehp.h      |    5 ++
> >  drivers/pci/hotplug/pciehp_core.c |    7 +-
> >  drivers/pci/hotplug/pciehp_ctrl.c |  149
> > ++++++++++++++++++++++++++++++-------
> >  drivers/pci/hotplug/pciehp_hpc.c  |   74 +++++++++---------
> >  4 files changed, 169 insertions(+), 66 deletions(-)

I applied these to pci/pciehp for v3.15, thanks!  I dropped the "ret" decl
in 8/8, which I assume was the cause of the unused variable warning.

Yinghai, if "Don't disable the link permanently, during removal" causes
trouble on your platforms, let us know so we can figure out how to deal
with it.

Bjorn
--
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/