RE: [PATCH 3/6] net: thunderx: Increase transmit queue length

From	Pavel Fedin <p.fedin@samsung.com>
Newsgroups	linux.kernel
Subject	RE: [PATCH 3/6] net: thunderx: Increase transmit queue length
Date	2015-12-02 13:30 +0100
Message-ID	<qBeHM-76w-17@gated-at.bofh.it> (permalink)
References	(2 earlier) <qAUpH-2rO-9@gated-at.bofh.it> <qAVc6-30Q-13@gated-at.bofh.it> <qAW8a-3BH-11@gated-at.bofh.it> <qBbAd-59b-5@gated-at.bofh.it> <qBcZj-5WX-1@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

 Hello!

> >  So, i see several possible ways to solve this:
> >
> > 1. Introduce some mechanism which would allow the driver to tell the kernel that it needs
> > coherent pool of large size. Can be problematic because the driver can be a module, and pool
> > allocation happens early.
> > 2. Can we use some other method for allocating queues, which would not require such a huge
> > coherent pool?
> > 3. The driver could check value of atomic_pool_size and adjust own memory requirements
> > accordingly. This indeed looks like a quick hack, but would at least make things running
> > quickly.
> 
>  I have also noticed that CONFIG_DMA_CMA is turned off in my kernel. I guess it was a leftover
> from old defconfig, because i carry over my .config from version to version. I enabled it and
> rebuilt the kernel, but in order to get the driver working with this patch i had to also add
> cma=32M option to kernel arguments. With default of 16M the allocation still fails.
>  Should we add Kconfig dependencies?

 After getting it working in guest i tried to apply it to host. With total of 128 virtual functions (= 128 interfaces) it does not work at all. Even after bumping cma region size to insane value of 2GB more than half of interfaces still failed to allocate queues. And after setting cma=3G i could not mount my rootfs.
 So, absolute NAK, unfortunately.

Kind regards,
Pavel Fedin
Expert Engineer
Samsung Electronics Research center Russia


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

[PATCH 3/6] net: thunderx: Increase transmit queue length Sunil Goutham <sunil.kovvuri@gmail.com> - 2015-12-01 10:20 +0100
  RE: [PATCH 3/6] net: thunderx: Increase transmit queue length Pavel Fedin <p.fedin@samsung.com> - 2015-12-01 15:50 +0100
    Re: [PATCH 3/6] net: thunderx: Increase transmit queue length Eric Dumazet <eric.dumazet@gmail.com> - 2015-12-01 16:40 +0100
      Re: [PATCH 3/6] net: thunderx: Increase transmit queue length Sunil Kovvuri <sunil.kovvuri@gmail.com> - 2015-12-01 17:40 +0100
        Re: [PATCH 3/6] net: thunderx: Increase transmit queue length David Miller <davem@davemloft.net> - 2015-12-01 20:40 +0100
          Re: [PATCH 3/6] net: thunderx: Increase transmit queue length Sunil Kovvuri <sunil.kovvuri@gmail.com> - 2015-12-02 06:50 +0100
            Re: [PATCH 3/6] net: thunderx: Increase transmit queue length Eric Dumazet <eric.dumazet@gmail.com> - 2015-12-02 14:30 +0100
              Re: [PATCH 3/6] net: thunderx: Increase transmit queue length Eric Dumazet <eric.dumazet@gmail.com> - 2015-12-02 18:00 +0100
              Re: [PATCH 3/6] net: thunderx: Increase transmit queue length Sunil Kovvuri <sunil.kovvuri@gmail.com> - 2015-12-02 18:00 +0100
            Re: [PATCH 3/6] net: thunderx: Increase transmit queue length David Miller <davem@davemloft.net> - 2015-12-02 18:40 +0100
        RE: [PATCH 3/6] net: thunderx: Increase transmit queue length Pavel Fedin <p.fedin@samsung.com> - 2015-12-02 10:10 +0100
          RE: [PATCH 3/6] net: thunderx: Increase transmit queue length Pavel Fedin <p.fedin@samsung.com> - 2015-12-02 11:40 +0100
            RE: [PATCH 3/6] net: thunderx: Increase transmit queue length Pavel Fedin <p.fedin@samsung.com> - 2015-12-02 13:30 +0100
              Re: [PATCH 3/6] net: thunderx: Increase transmit queue length Sunil Kovvuri <sunil.kovvuri@gmail.com> - 2015-12-02 14:00 +0100
                RE: [PATCH 3/6] net: thunderx: Increase transmit queue length Pavel Fedin <p.fedin@samsung.com> - 2015-12-02 14:30 +0100
      RE: [PATCH 3/6] net: thunderx: Increase transmit queue length Pavel Fedin <p.fedin@samsung.com> - 2015-12-02 09:10 +0100

csiph-web