public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* Questions about IOMMU & PCIe switch
@ 2015-01-07  4:06 Raimonds Cicans
  2015-01-07 12:09 ` Clemens Ladisch
  0 siblings, 1 reply; 7+ messages in thread
From: Raimonds Cicans @ 2015-01-07  4:06 UTC (permalink / raw)
  To: linux-kernel

Hello.

We have two kinds of PCIe cards:
A1 - based on chip B
A2 - based on same chip B but behind PCIe switch

Card A1 work flawlessly in any configuration,
but card A2 work flawlessly only if system
lack IOMMU or have disabled IOMMU

Questions:
Do we have to treat card with PCIe switch in special way?
How?
Where (PCI bridge driver? card's driver?)?
Any examples in kernel?

Thank you.

Raimonds Cicans

^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: Questions about IOMMU & PCIe switch
@ 2015-01-15 12:54 Luis Alves
  0 siblings, 0 replies; 7+ messages in thread
From: Luis Alves @ 2015-01-15 12:54 UTC (permalink / raw)
  To: linux-kernel; +Cc: Manu Abraham, ray, clemens

Hi Manu,

Do you know if your saa716x driver might have any issue in the dma code?
The code doesn't seem to be using chained scatterlists so it should be
ok with the "old" method of just increment the pointer instead of
using sg_next.
(It's still a little blurry to my this IOMMU+dma+scatterlist stuff)

Anyway, I've converted it to use "sg_next" as asked Raimonds to test
(but I think it shouldn't make any difference):
https://github.com/ljalves/linux_media/commit/54362743588bc8271e7988dd2823f818c00bf43d

I also know don't fully understand the whole driver but I say thank
you for it! :)
By the way, when can we get it upstream? If you need help with
anything just mail me. We are waiting for that driver for so long!


Regards,
Luis


On Thu, Jan 8, 2015 at 8:35 PM, Raimonds Cicans <ray@apollo.lv> wrote:
> On 08.01.2015 10:34, Clemens Ladisch wrote:
>>
>> Raimonds Cicans wrote:
>>>
>>> https://github.com/ljalves/linux_media/issues/66
>>
>> If the TBS driver works, why don't you use it?
>
> 1) driver is not stable in 24x7 setups
>
> 2) driver use old DVBAPI. This cause problems with some
>      user space programs.

It shouldn't. It is backward compatible.

>
> 3) TBS recommends to use card in MSI interrupt mode
>      but this mode on IOMMU systems do not work:
>      card is able to find transponders and tune to it
>      but can not receive any data

TBS didn't even have the courtesy to say thanks, but still.
They don't have any clue on the driver.

You shouldn't use MSI mode on most systems unless you
are sure that your machine works perfectly well with MSI
mode.

You should simply use standard interrupt mode, if you are
unsure.

In any sense, that card looks a bit odd:

1. It doesn't need a PCIe switch, nor that CPLD in there for
the fact that it is using a SAA7160, rather than an E version,
which thereby support 4 FGPI inputs (4 simultaneous TS's)
(Redundant components)

2. But in any sense, the PCIe switch is transparent, It doesn't
need any programming. All what you can do with the PCIe
switch is to reprogram the lines to it, nothing more. In any
case, if your card is detected, there's no point in playing with
the PCIe switch

3. The TS failure in a while can most likely be attributed to
bad CPLD programming. On the cards that I have worked
with, they've worked pretty well.


Regards,

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

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2015-01-15 12:54 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-01-07  4:06 Questions about IOMMU & PCIe switch Raimonds Cicans
2015-01-07 12:09 ` Clemens Ladisch
2015-01-07 14:04   ` Raimonds Cicans
2015-01-08  8:34     ` Clemens Ladisch
2015-01-08 15:05       ` Raimonds Cicans
2015-01-08 15:32         ` Manu Abraham
  -- strict thread matches above, loose matches on Subject: below --
2015-01-15 12:54 Luis Alves

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox