Groups > linux.kernel > #1202161

Re: [PATCH v3] PCI: Only enable IO window if supported

From	Yinghai Lu <yinghai@kernel.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH v3] PCI: Only enable IO window if supported
Date	2015-08-07 01:40 +0200
Message-ID	<pUCVs-76v-5@gated-at.bofh.it> (permalink)
References	(3 earlier) <pUj6q-3nR-3@gated-at.bofh.it> <pUkYx-659-5@gated-at.bofh.it> <pUlrA-6GV-3@gated-at.bofh.it> <pUvKi-57B-29@gated-at.bofh.it> <pUw3D-5JN-1@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Thu, Aug 6, 2015 at 9:18 AM, Guenter Roeck <linux@roeck-us.net> wrote:
> On 08/06/2015 08:58 AM, Yinghai Lu wrote:

> Ok, I admit that I am lost. Earlier it seemed that you were concerned about
> this case, where no io window is available or a bus doesn't support io,
> but a non-transparent child does. Now you seem to say that the
> non-transparent
> child would not be able to support IO either.
>
> For my education, can you list the possible options, and how you suggest
> to solve them ? I can see the following situations.
>
> - root supports IO, but has no io window assigned
> - root does not support IO and has or has not an IO window assigned
> - a bridge does not support IO

should have two cases:
1. root bus does not have io window exposed.
2. pci bridge io bar is not writable.

>
> For the transparent case, each of those should result in all children
> not even trying to assign an IO window, which is what we want,
> and what my patc set tries to do.
>
> How should those cases be handled for non-transparent bridges ?

for case 2: current upstream code, no warning for the bridge itself,
but have warning for devices under the bridge.

We only need to handle case 1, aka root bus does not have io port window.

What about your setup, is it case 1 or case 2?

Yinghai
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Thread

Re: [PATCH v3] PCI: Only enable IO window if supported Yinghai Lu <yinghai@kernel.org> - 2015-08-06 03:20 +0200
  Re: [PATCH v3] PCI: Only enable IO window if supported Yinghai Lu <yinghai@kernel.org> - 2015-08-06 03:40 +0200
    Re: [PATCH v3] PCI: Only enable IO window if supported Guenter Roeck <linux@roeck-us.net> - 2015-08-06 04:30 +0200
      Re: [PATCH v3] PCI: Only enable IO window if supported Yinghai Lu <yinghai@kernel.org> - 2015-08-06 06:30 +0200
        Re: [PATCH v3] PCI: Only enable IO window if supported Guenter Roeck <linux@roeck-us.net> - 2015-08-06 07:00 +0200
          Re: [PATCH v3] PCI: Only enable IO window if supported Yinghai Lu <yinghai@kernel.org> - 2015-08-06 18:00 +0200
            Re: [PATCH v3] PCI: Only enable IO window if supported Guenter Roeck <linux@roeck-us.net> - 2015-08-06 18:20 +0200
              Re: [PATCH v3] PCI: Only enable IO window if supported Yinghai Lu <yinghai@kernel.org> - 2015-08-07 01:40 +0200
                Re: [PATCH v3] PCI: Only enable IO window if supported Guenter Roeck <linux@roeck-us.net> - 2015-08-07 02:40 +0200
                Re: [PATCH v3] PCI: Only enable IO window if supported Yinghai Lu <yinghai@kernel.org> - 2015-08-07 03:20 +0200
  Re: [PATCH v3] PCI: Only enable IO window if supported Guenter Roeck <linux@roeck-us.net> - 2015-08-06 03:50 +0200

csiph-web