Hi Alex-san,
Hi Kenji-san,
* Kenji Kaneshige <[EMAIL PROTECTED]>:
Hi Alex-san,
On my system, hotplug slots themselves can be added, removed
and replaced with the ohter type of I/O box.
Are you talking about some sort of I/O cabinet/chassis that you
can attach to the actual compute
Hi Alex-san,
Hi Kenji-san,
* Kenji Kaneshige [EMAIL PROTECTED]:
Hi Alex-san,
On my system, hotplug slots themselves can be added, removed
and replaced with the ohter type of I/O box.
Are you talking about some sort of I/O cabinet/chassis that you
can attach to the actual computer? Can
_STA evaluation
fails on the hotplug slot which doesn't have _STA method. If the
device object doesn't have a _STA method, we need to handle it as
if it is present. I believe firmware normally doesn't provide
_STA method for PCI hotplug slots.
Thanks,
Kenji Kaneshige
--
To unsubscribe from this lis
method. If the
device object doesn't have a _STA method, we need to handle it as
if it is present. I believe firmware normally doesn't provide
_STA method for PCI hotplug slots.
Thanks,
Kenji Kaneshige
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message
st 2.6.24-rc3 with your patches applied. Could you
try it?
BTW, acpiphp also seems to have the same problem...
Thanks,
Kenji Kaneshige
---
drivers/acpi/pci_slot.c | 13 +
1 file changed, 13 insertions(+)
Index: linux-2.6.24-rc3/drivers/acpi/pci_slot.c
==
, pci_slot driver need to check _STA when scanning ACPI
Namespace.
I'm sorry for reporting this so late. I'm attaching the patch to fix the
problem. This is against 2.6.24-rc3 with your patches applied. Could you
try it?
BTW, acpiphp also seems to have the same problem...
Thanks,
Kenji Kaneshige
drivers only visiting your new
code when the pci_slot driver is loaded, although I think this
would be more difficult to implement.
I agree to Gary very much.
Thanks,
Kenji Kaneshige
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message
drivers only visiting your new
code when the pci_slot driver is loaded, although I think this
would be more difficult to implement.
I agree to Gary very much.
Thanks,
Kenji Kaneshige
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL
lt;3>pci_slot: pci_create_slot returned -22
Thanks,
Kenji Kaneshige
Alex Chiang :
Hello,
[this patch series touches a few subsystems; hopefully I got all
the right maintainers]
Recently, Matthew Wilcox sent out the following mail about PCI
slots:
http://marc.info/?l=linux-pci=1
it was added.
is slot number decided according to the shpc specification,
as you said.
Thanks,
Kenji Kaneshige
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org
it was added.
is slot number decided according to the shpc specification,
as you said.
Thanks,
Kenji Kaneshige
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo
bsp=e14101be0c20
[a00189c0] start_kernel_thread+0x20/0x40
sp=e14101befe30 bsp=e14101be0c20
Unable to register kobject 10243pci_slot: pci_create_slot returned -22
Thanks,
Kenji Kaneshige
Alex Chiang :
Hello,
[this patch series touches
Mark Lord さんは書きました:
> Kenji Kaneshige wrote:
>>
>> - When the card is inserted *after* modprobing pciehp, the card
>>is *not* automatically powered on/detected. So it is very
>>natural that the card, which had been inserted before modprobing
>>pci
r controller. So there is no
contradiction.
I think your patch has a bad effect to this kind of slots. So I
think your patch needs additional checks to see if the slot should
be enabled or not.
And you need to test your patch on PCIe Native hotplug capable
hardware:)
Thanks,
Kenji Kane
lots/XXX/*"?
(6) I think your slot is surprise removable. Is it correct?
Thanks,
Kenji Kaneshige
(repost to conform with akpm's subject line conventions)
One of three patches to fix PCIe Hotplug so that it works with ExpressCard slots
on Dell notebooks (and others?) in conjunctio
?
Thanks,
Kenji Kaneshige
(repost to conform with akpm's subject line conventions)
One of three patches to fix PCIe Hotplug so that it works with ExpressCard slots
on Dell notebooks (and others?) in conjunction with modparam of pciehp_force=1.
Fix pciehp_probe() to deal with ExpressCard cards
your patch needs additional checks to see if the slot should
be enabled or not.
And you need to test your patch on PCIe Native hotplug capable
hardware:)
Thanks,
Kenji Kaneshige
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL
Mark Lord さんは書きました:
Kenji Kaneshige wrote:
- When the card is inserted *after* modprobing pciehp, the card
is *not* automatically powered on/detected. So it is very
natural that the card, which had been inserted before modprobing
pciehp, is not automatically enabled at the pciehp
.
Thank you for reporting.
I think this problem might be fixed by the commit
6ffbc82351c62aeb9e817ddf93049353493d. Could you try that?
Thanks,
Kenji Kaneshige
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTE
for reporting.
I think this problem might be fixed by the commit
6ffbc82351c62aeb9e817ddf93049353493d. Could you try that?
Thanks,
Kenji Kaneshige
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http
vironment.
Thanks,
Kenji Kaneshige
2007-05-11 (金) の 09:25 -0700 に Stephen Hemminger さんは書きました:
> On Fri, 11 May 2007 14:40:45 +0900
> Kenji Kaneshige <[EMAIL PROTECTED]> wrote:
>
> > Hi,
> >
> > I encountered the following error when I was hot-plugging
.
Thanks,
Kenji Kaneshige
2007-05-11 (金) の 09:25 -0700 に Stephen Hemminger さんは書きました:
On Fri, 11 May 2007 14:40:45 +0900
Kenji Kaneshige [EMAIL PROTECTED] wrote:
Hi,
I encountered the following error when I was hot-plugging network card
using pci hotplug driver.
kobject_add failed
es it have no side effect? I'm worrying
about if there are somebody that depend on sysfs entry until
net_run_todo() is called.
Anyway, I'll try your patch.
Thanks,
Kenji Kaneshige
2007-05-11 (金) の 09:25 -0700 に Stephen Hemminger さんは書きました:
> On Fri, 11 May 2007 14:40:45 +0900
> Kenji Kaneshige
about if there are somebody that depend on sysfs entry until
net_run_todo() is called.
Anyway, I'll try your patch.
Thanks,
Kenji Kaneshige
2007-05-11 (金) の 09:25 -0700 に Stephen Hemminger さんは書きました:
On Fri, 11 May 2007 14:40:45 +0900
Kenji Kaneshige [EMAIL PROTECTED] wrote:
Hi,
I
and snapshot list
to find a suitable id. If some devices exists on the todo list or
snapshot list, sysfs entries corresponding to them are not deleted
yet.
Thanks,
Kenji Kaneshige
---
Fix the race condition between dev_alloc_name() and net device
unregistration.
dev_alloc_name checks `dev_base
entries corresponding to them are not deleted
yet.
Thanks,
Kenji Kaneshige
---
Fix the race condition between dev_alloc_name() and net device
unregistration.
dev_alloc_name checks `dev_base' list to find a suitable ID. On the
other hand, net devices are removed from that list on net device
fixed by it.
Thanks,
Kenji Kaneshige
This patch fixes the problem that uninitialized (NULL) 'info' member
of uart_port structure can be accessed if serial driver is accessed
through /proc filesystem before uart_open(), which initializes the
'info' member', is called.
Signed-off-by: Kenji Kaneshige
(). This
is why we fix this problem in 8250 driver. But if there is a
possibility that other drivers accesses port-info in their
-get_mctrl(), we should be fixing this in uart_read_proc(), as
you said.
How about the following patch? We've also confirmed the problem
is fixed by it.
Thanks,
Kenji Kaneshige
covery.
"Old style" is now deprecated. Don't use it. Left description in
to document existing driver behaviors.
o add section "Legacy I/O Port free driver" by Kenji Kaneshige
http://lkml.org/lkml/2006/11/22/25
(renamed to "pci_enable_device_bars() and Legacy I/O Port sp
deprecated. Don't use it. Left description in
to document existing driver behaviors.
o add section Legacy I/O Port free driver by Kenji Kaneshige
http://lkml.org/lkml/2006/11/22/25
(renamed to pci_enable_device_bars() and Legacy I/O Port space)
Thank you for updating this.
It looks much
n't use an interrupt pin.
Thanks,
Kenji Kaneshige
The following warning message should not be displayed for devices
which don't use an interrupt pin.
pcie_portdrv_probe->Dev[:] has invalid IRQ. Check vendor BIOS
Signed-off-by: Kenji Kaneshige <[EMAIL PROTECTED]>
---
driv
use an interrupt pin.
Thanks,
Kenji Kaneshige
The following warning message should not be displayed for devices
which don't use an interrupt pin.
pcie_portdrv_probe-Dev[:] has invalid IRQ. Check vendor BIOS
Signed-off-by: Kenji Kaneshige [EMAIL PROTECTED]
---
drivers/pci/pcie
The function prototype for handle_IRQ_event() in include/asm-ppc/irq.h
seems no longer needed because ppc uses GENERIC_HARDIRQ. This patch
removes it.
Signed-off-by: Kenji Kaneshige <[EMAIL PROTECTED]>
include/asm-ppc/irq.h |4
1 files changed, 4 deletions(-)
Index: linux-
The function prototype for handle_IRQ_event() in include/asm-x86_64/irq.h
seems no longer needed because x86_64 uses GENERIC_HARDIRQ. This patch
removes it.
Signed-off-by: Kenji Kaneshige <[EMAIL PROTECTED]>
include/asm-x86_64/irq.h |4
1 files changed, 4 deletions(-)
Index:
The function prototype for handle_IRQ_event() in include/asm-sh/irq.h
seems no longer needed because sh uses GENERIC_HARDIRQ. This patch
removes it.
Signed-off-by: Kenji Kaneshige <[EMAIL PROTECTED]>
include/asm-sh/irq.h |4
1 files changed, 4 deletions(-)
Index: linux-2.6.13/i
The function prototype for handle_IRQ_event() in include/asm-mips/irq.h
seems no longer needed because mips uses GENERIC_HARDIRQ. This patch
removes it.
Signed-off-by: Kenji Kaneshige <[EMAIL PROTECTED]>
include/asm-mips/irq.h |3 ---
1 files changed, 3 deletions(-)
Index: linux-
these architectures, these patches are not
tested yet. I hope these patches will be tested by someone who has
these architectures.
Thanks,
Kenji Kaneshige
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo inf
these architectures, these patches are not
tested yet. I hope these patches will be tested by someone who has
these architectures.
Thanks,
Kenji Kaneshige
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http
The function prototype for handle_IRQ_event() in include/asm-mips/irq.h
seems no longer needed because mips uses GENERIC_HARDIRQ. This patch
removes it.
Signed-off-by: Kenji Kaneshige [EMAIL PROTECTED]
include/asm-mips/irq.h |3 ---
1 files changed, 3 deletions(-)
Index: linux-2.6.13
The function prototype for handle_IRQ_event() in include/asm-sh/irq.h
seems no longer needed because sh uses GENERIC_HARDIRQ. This patch
removes it.
Signed-off-by: Kenji Kaneshige [EMAIL PROTECTED]
include/asm-sh/irq.h |4
1 files changed, 4 deletions(-)
Index: linux-2.6.13/include
The function prototype for handle_IRQ_event() in include/asm-x86_64/irq.h
seems no longer needed because x86_64 uses GENERIC_HARDIRQ. This patch
removes it.
Signed-off-by: Kenji Kaneshige [EMAIL PROTECTED]
include/asm-x86_64/irq.h |4
1 files changed, 4 deletions(-)
Index: linux
The function prototype for handle_IRQ_event() in include/asm-ppc/irq.h
seems no longer needed because ppc uses GENERIC_HARDIRQ. This patch
removes it.
Signed-off-by: Kenji Kaneshige [EMAIL PROTECTED]
include/asm-ppc/irq.h |4
1 files changed, 4 deletions(-)
Index: linux-2.6.13/include
Hi Bjorn,
Thank you very much for the new patch
and I'm very sorry for troubling you.
The patch looks very good to me.
Thanks,
Kenji Kaneshige
Bjorn Helgaas wrote:
On Tuesday 02 August 2005 7:05 pm, Kenji Kaneshige wrote:
This breaks the following patch that is already included into -mm
Hi Bjorn,
Thank you very much for the new patch
and I'm very sorry for troubling you.
The patch looks very good to me.
Thanks,
Kenji Kaneshige
Bjorn Helgaas wrote:
On Tuesday 02 August 2005 7:05 pm, Kenji Kaneshige wrote:
This breaks the following patch that is already included into -mm
irq_resource[i].start = irq;
+ res->irq_resource[i].end = irq;
}
}
This breaks the following patch that is already included into -mm
tree.
http://sourceforge.net/mailarchive/forum.php?thread_id=7844247_id=6102
I think we need to check if acpi_register_gsi() succeeded or
;
+ res-irq_resource[i].end = irq;
}
}
This breaks the following patch that is already included into -mm
tree.
http://sourceforge.net/mailarchive/forum.php?thread_id=7844247forum_id=6102
I think we need to check if acpi_register_gsi() succeeded or not.
Thanks,
Kenji Kaneshige
This patch adds the error check of acpi_register_gsi() into pnpacpi
driver.
Signed-off-by: Kenji Kaneshige <[EMAIL PROTECTED]>
---
linux-2.6.13-rc3-kanesige/drivers/pnp/pnpacpi/rsparser.c |2 +-
1 files changed, 1 insertion(+), 1 deletion(-)
diff -puN drivers/pnp/pnpacpi/rspa
Change iosapic_register_intr(), which called by acpi_register_gsi(),
to return negative value on error instead of panic.
Signed-off-by: Kenji Kaneshige <[EMAIL PROTECTED]>
---
linux-2.6.13-rc3-kanesige/arch/ia64/kernel/iosapic.c | 20 ---
1 files changed, 13 inse
the case that acpi_register_gsi() fails. I think failure
of acpi_register_gsi() should be handled properly.
This series of patches changes acpi_register_gsi() to return negative
value on error, and also changes callers of acpi_register_gsi() to
handle failure of acpi_register_gsi().
Thanks,
Kenji
This patch adds the error check of acpi_register_gsi() into ACPI based
8250 serial driver.
Signed-off-by: Kenji Kaneshige <[EMAIL PROTECTED]>
---
linux-2.6.13-rc3-kanesige/drivers/serial/8250_acpi.c | 20 +++
1 files changed, 16 insertions(+), 4 deletions(-)
dif
This patch adds the error check of acpi_register_gsi() into
acpi_pci_enable_irq().
Signed-off-by: Kenji Kaneshige <[EMAIL PROTECTED]>
---
linux-2.6.13-rc3-kanesige/drivers/acpi/pci_irq.c |9 -
1 files changed, 8 insertions(+), 1 deletion(-)
diff -puN
drivers/acpi/pci
This patch adds the error check of acpi_register_gsi() into hpet
driver.
Signed-off-by: Kenji Kaneshige <[EMAIL PROTECTED]>
---
linux-2.6.13-rc3-kanesige/drivers/char/hpet.c |8 ++--
1 files changed, 6 insertions(+), 2 deletions(-)
diff -puN drivers/char/hpet.c~handle
This patch changes the type of return value of acpi_register_gsi()
from "unsigned int" to "int" to indicate an error. If
acpi_register_gsi() fails to register gsi, it returns negative value.
Signed-off-by: Kenji Kaneshige <[EMAIL PROTECTED]>
---
linux-2.6.13-rc3
This patch changes the type of return value of acpi_register_gsi()
from unsigned int to int to indicate an error. If
acpi_register_gsi() fails to register gsi, it returns negative value.
Signed-off-by: Kenji Kaneshige [EMAIL PROTECTED]
---
linux-2.6.13-rc3-kanesige/arch/i386/kernel/acpi
This patch adds the error check of acpi_register_gsi() into hpet
driver.
Signed-off-by: Kenji Kaneshige [EMAIL PROTECTED]
---
linux-2.6.13-rc3-kanesige/drivers/char/hpet.c |8 ++--
1 files changed, 6 insertions(+), 2 deletions(-)
diff -puN drivers/char/hpet.c~handle-error
This patch adds the error check of acpi_register_gsi() into
acpi_pci_enable_irq().
Signed-off-by: Kenji Kaneshige [EMAIL PROTECTED]
---
linux-2.6.13-rc3-kanesige/drivers/acpi/pci_irq.c |9 -
1 files changed, 8 insertions(+), 1 deletion(-)
diff -puN
drivers/acpi/pci_irq.c~handle
This patch adds the error check of acpi_register_gsi() into ACPI based
8250 serial driver.
Signed-off-by: Kenji Kaneshige [EMAIL PROTECTED]
---
linux-2.6.13-rc3-kanesige/drivers/serial/8250_acpi.c | 20 +++
1 files changed, 16 insertions(+), 4 deletions(-)
diff -puN
the case that acpi_register_gsi() fails. I think failure
of acpi_register_gsi() should be handled properly.
This series of patches changes acpi_register_gsi() to return negative
value on error, and also changes callers of acpi_register_gsi() to
handle failure of acpi_register_gsi().
Thanks,
Kenji
Change iosapic_register_intr(), which called by acpi_register_gsi(),
to return negative value on error instead of panic.
Signed-off-by: Kenji Kaneshige [EMAIL PROTECTED]
---
linux-2.6.13-rc3-kanesige/arch/ia64/kernel/iosapic.c | 20 ---
1 files changed, 13 insertions(+), 7
This patch adds the error check of acpi_register_gsi() into pnpacpi
driver.
Signed-off-by: Kenji Kaneshige [EMAIL PROTECTED]
---
linux-2.6.13-rc3-kanesige/drivers/pnp/pnpacpi/rsparser.c |2 +-
1 files changed, 1 insertion(+), 1 deletion(-)
diff -puN drivers/pnp/pnpacpi/rsparser.c~handle
be removed
from -mm tree.
Thanks,
Kenji Kaneshige
Luck, Tony wrote:
I started on my OLS homework from Andrew ... and began looking
into what is going on here.
The story so far: Pavel added calls to device_suspend() to three of
the cases in the sys_reboot() path. This stopped ia64 from being
be removed
from -mm tree.
Thanks,
Kenji Kaneshige
Luck, Tony wrote:
I started on my OLS homework from Andrew ... and began looking
into what is going on here.
The story so far: Pavel added calls to device_suspend() to three of
the cases in the sys_reboot() path. This stopped ia64 from being
. Even when we use per node approach,
we need to consider interrupt migration between nodes
because all CPUs on the node could be hot-removed.
Thanks,
Kenji Kaneshige
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTE
. Even when we use per node approach,
we need to consider interrupt migration between nodes
because all CPUs on the node could be hot-removed.
Thanks,
Kenji Kaneshige
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo
through pci_enable_device()/pci_disable_device().
(B
(BSigned-off-by: Kenji Kaneshige <[EMAIL PROTECTED]>
(B
(B
(B---
(B
(B linux-2.6.12-rc2-kanesige/drivers/pci/pci.c |7 +++
(B 1 files changed, 3 insertions(+), 4 deletions(-)
(B
(Bdiff -puN drivers/pci/pci.c~fix_update_is_e
through pci_enable_device()/pci_disable_device().
(B
(BSigned-off-by: Kenji Kaneshige [EMAIL PROTECTED]
(B
(B
(B---
(B
(B linux-2.6.12-rc2-kanesige/drivers/pci/pci.c |7 +++
(B 1 files changed, 3 insertions(+), 4 deletions(-)
(B
(Bdiff -puN drivers/pci/pci.c~fix_update_is_enabled
66 matches
Mail list logo