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]

 



On Fri, May 30, 2008 at 09:13:02AM +1000, Benjamin Herrenschmidt wrote:
> > > In fact, you don't necessarily have the bridge either.
> > > 
> > > When you do hotplug operations on pSeries, the hypervisor feeds us with
> > > a whole bunch of new OF nodes & properties -at the time of the
> > > insertion-. This can include new top level host bridges, and layers of
> > > p2p bridges finally leading to the device that was inserted.
> > 
> > But surely you get a series of events?
> > 
> > "Hi, I've just plugged in a new cabinet"
> > "Hi, I've just plugged in a new PCI root bridge"
> > "Hi, I've just plugged in a new PCI-PCI bridge"
> > "Hi, I've just plugged in a new PCI card"
> 
> Yes. We get those events along with bits of device-tree (ie, new nodes
> and properties to add to the OF tree). But we don't know necessarily
> in advance what we'll get and what devfn it will be. Maybe I'm
> misunderstanding the problem here, I though you guys needed to know that
> in advance, before the device gets plugged...

We need to know the slot number when the slot appears in your partition.
So if you can't control the slot, you don't need to register it.

> > > That is at least when you move things in and out of partitions. I have
> > > to verify what happens if you add to a partition an empty slot (if you
> > > can) and then just perform insert/remove operations on it. I'm not the
> > > expert in IBM PCI hotplug stuff, and when I tried to sort that out 3
> > > weeks ago, I just didn't manage to get the bloody thing to work at all
> > > on one of my test machines (the daemon talking to the management console
> > > is ... fragile).
> > 
> > Surely you must be able to, or we wouldn't encounter a problem.
> 
> Oh yes sure, it's just that whatever is installed on my machine here is
> stuffed and I need to fix it up / re-install.

Sorry, I misquoted.  I meant "Surely you must be able to add empty slots
to partitions or we wouldn't see a problem".

-- 
Intel are signing my paycheques ... these opinions are still mine
"Bill, look, we understand that you're interested in selling us this
operating system, but compare it to ours.  We can't possibly take such
a retrograde step."
--
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