Re: [PATCH 5/7] pci hotplug core: add check of duplicate slot name

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

 



* Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx>:
> On Wednesday, May 28, 2008 4:04 pm Alex Chiang wrote:
> > * Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>:
> > > I assume these patches will destroy Alex's
> > > frequently-destroyed pci-hotplug-introduce-pci_slot.patch.
> > > Ho hum.
> >
> > I took a quick scan, and yeah, they'll break my patch series,
> > but I should be able to adapt. Ho hum indeed.
> >
> > More importantly though, I can't seem to get any help on the
> > pseries front. Benjamin Herrenschmidt is too busy, and I
> > haven't heard a single peep from the IBM fellow (Mike Mason).
> >
> > Any suggestions on how to move forward? My fixup patch did
> > fix Badari's original oops in a syntactically sane manner; it
> > was the semantics that were a bit trickier.
> >
> > Perhaps we could roll that last fixup patch into the series,
> > aim for the next merge window, and see who complains?
> > Obviously I'd work any issues that would pop up, and we could
> > always revert it as a last resort.
> >
> > Or is that idea too sloppy for mainline?
> 
> Assuming Kenji-san & Kristen are ok with your slot patches, we
> can push them in during the 2.6.27 merge window and see who
> screams.  If things look like they'll take too long to fix, we
> can always revert, but I think it's more likely that we can fix
> anything that comes up...
> 
> Which means I'll need a new patch set from you, Alex, against
> the linux-next branch.  I'll stuff them in once I get acks from
> Kenji & Kristen.

Sounds good. I'll work on refreshing the series and send them
over to you. In the meantime, if BenH sends review comments, I'll
incorporate them as well.

Thanks.

/ac

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

[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux