26/11/2018 04:53, Varghese, Vipin:
> Hi Thomas,
>
> snipped
>
> > >
> > > Signed-off-by: Vipin Varghese
> > > Acked-by: Marko Kovacevic
> >
> > I was in the hope of seeing more reviews for this guide.
>
> Thanks for the update will wait for feedbacks
>
> > I have the feeling it is not ready
Add user guide on debug and troubleshoot for common issues and bottleneck
found in various application models running on single or multi stages.
Signed-off-by: Vipin Varghese
Acked-by: Marko Kovacevic
---
V3:
- reorder for removing warning in 'make doc-guides-html' - Thomas Monjalon
V2:
- ad
Add svg for debug and troubleshoot guide for single or multi stage
applications.
Signed-off-by: Vipin Varghese
Acked-by: Marko Kovacevic
---
V3:
- reorder for removing warning in 'make doc-guides-html' - Thomas Monjalon
V2:
- add ACK from Marko - Vipin Varghese
---
doc/guides/howto/img/dtg_
Thank you, will wait for updated documentation and release notes.
> -Original Message-
> From: Li, Xiaoyun
> Sent: Monday, November 26, 2018 10:49 AM
> To: Varghese, Vipin ; Lu, Wenzhuo
> ; dev@dpdk.org
> Cc: Lu, Wenzhuo ; Yang, Qiming
> ; Wu, Jingjing
> Subject: RE: [dpdk-dev] [PATCH 02/
Yes.
> -Original Message-
> From: Varghese, Vipin
> Sent: Monday, November 26, 2018 13:14
> To: Li, Xiaoyun ; Lu, Wenzhuo ;
> dev@dpdk.org
> Cc: Lu, Wenzhuo ; Yang, Qiming
> ; Wu, Jingjing
> Subject: RE: [dpdk-dev] [PATCH 02/19] net/ice: support device initialization
>
> Thanks for the u
Thanks for the update, so we can assume all functions are available in
secondary. If not supported it will return right error code and documentation
is updated for the limitation too.
> -Original Message-
> From: Li, Xiaoyun
> Sent: Monday, November 26, 2018 10:40 AM
> To: Varghese, Vipi
Hi
> Do we check if process is secondary and ICE PMD is already is initialized? If
> we
> do not check will we run to multi process reinitilization?
Yes. We check. It is in [PATCH 16/19] net/ice: support basic RX/TX. Please see
that.
We roughly split the ice codes in different patch. So the fil
Hi Stephen,
snipped
>
> > > > > > > >
> > > > > > > > >
> > > > > > > > > > + if ((ret) | (!is_leaf))
> > > > > > > > > > +
> > > > > > > > >
> > > > > > > > > Is the operator here should be || ?
> > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > > Check is done
Hi Qiming,
Thanks for the update, please feel free to update the limitation in release
notes which all are not supported too.
Note: do we plan to add test cases (unit) for the same?
> -Original Message-
> From: Yang, Qiming
> Sent: Monday, November 26, 2018 9:13 AM
> To: Varghese, Vipin
Hi Qiming
> -Original Message-
> From: Yang, Qiming
> Sent: Monday, November 26, 2018 9:09 AM
> To: Varghese, Vipin ; Lu, Wenzhuo
> ; dev@dpdk.org
> Cc: Lu, Wenzhuo ; Li, Xiaoyun ;
> Wu, Jingjing
> Subject: RE: [dpdk-dev] [PATCH 07/19] net/ice: support MTU setting
>
> Hi, Vipin
> Not sur
Hi Thomas,
snipped
> >
> > Signed-off-by: Vipin Varghese
> > Acked-by: Marko Kovacevic
>
> I was in the hope of seeing more reviews for this guide.
Thanks for the update will wait for feedbacks
> I have the feeling it is not ready for prime time.
ok
> We should take time to discuss what we
Hi, Vipin
Not all feature enabled for secondary process. We add secondary process check
in each ops like below,
if (rte_eal_process_type() == RTE_PROC_SECONDARY)
return -E_RTE_SECONDARY;
Qiming
-Original Message-
From: dev [mailto:dev-boun...@dpdk.org] On Behalf
Hi, Vipin
Not sure understand your question.
We have no need to configure jumbo frame, because jumbo frame offload will be
enable when mtu>1500.
Qiming
-Original Message-
From: Varghese, Vipin
Sent: Friday, November 23, 2018 5:58 PM
To: Lu, Wenzhuo ; dev@dpdk.org
Cc: Lu, Wenzhuo ; Yang,
> On Nov 21, 2018, at 1:01 AM, Amedeo Sapio wrote:
>
> Hello,
>
> I experienced a problem with the MLX5 driver running a code that is working
> fine with an Intel card. I have found that the reason of this error is an
> overflow of the uint16_t number of descriptors in the mlx driver.
>
> Here
A new DPDK release candidate is ready for testing:
https://git.dpdk.org/dpdk/tag/?id=v18.11-rc5
58 patches (docs and fixes) were integrated.
This is the last release candidate for DPDK 18.11.
If no objection, the version 18.11.0 will be out on Monday 26th.
Please do the very last checks t
23/11/2018 18:04, Maxime Coquelin:
> Hi,
>
> On 11/23/18 4:43 PM, Darek Stojaczyk wrote:
> > Even if a device failed to plug, it's still a device
> > object that references the devargs. Those devargs will
> > be freed automatically together with the device, but
> > freeing them any earlier - like
23/11/2018 20:17, Zhang, Qi Z:
> From: Stojaczyk, Dariusz
> > Device detach triggered through IPC leaked some memory.
> > It allocated a devargs objects just to use it for parsing the devargs
> > string in order
> > to retrieve the device name. Those devargs weren't passed anywhere and were
> > ne
> > Consider the following scenario:
> >
> > 1) primary process (A) starts, probes the bus
> > 2) a secondary process (B) starts, probes the bus
> > 3) yet another secondary process (C) starts
> > 4) (C) registers the pci driver and hotplugs the device
> > * an IPC attach req is sent to th
23/11/2018 22:26, Darek Stojaczyk:
> When primary process receives an IPC attach request
> of a device that's already locally-attached, it
> doesn't setup its variables properly and is prone to
> segfaulting on a subsequent rollback.
>
> `ret = local_dev_probe(req->devargs, &dev)`
>
> The above f
21/11/2018 20:05, Darek Stojaczyk:
> If a device fails to attach before it's plugged,
> the subsequent rollback will still try to detach it,
> causing a segfault. Unplugging a device that wasn't
> plugged isn't really supported, so this patch adds
> an extra error check to prevent that from happeni
23/11/2018 10:12, Burakov, Anatoly:
> On 21-Nov-18 6:41 PM, Darek Stojaczyk wrote:
> > Setting up a device that wasn't setup in the primary
> > process will possibly break the primary process. That's
> > because the IPC message to retrieve the group fd in the
> > primary will also *open* that group
23/11/2018 12:01, Maxime Coquelin:
> On 11/23/18 1:29 AM, Ferruh Yigit wrote:
> > The pci_resource_by_index called strlen() on uninitialized
> > memory which would lead to the wrong size of memory allocated
> > for the path portion of the resource map. This would either cause
> > excessively large
23/11/2018 23:29, Thomas Monjalon:
> 23/11/2018 21:07, Kevin Traynor:
> > On 11/23/2018 04:25 PM, Maxime Coquelin wrote:
> > > On 11/23/18 4:53 PM, Ilya Maximets wrote:
> > >> On 23.11.2018 18:41, Maxime Coquelin wrote:
> > >>> On 11/23/18 4:39 PM, Ilya Maximets wrote:
> > Patches primary targ
09/11/2018 16:03, Vipin Varghese:
> Add user guide on debug and troubleshoot for common issues and bottleneck
> found in various application models running on single or multi stages.
>
> Signed-off-by: Vipin Varghese
> Acked-by: Marko Kovacevic
I was in the hope of seeing more reviews for this
22/11/2018 18:59, Mcnamara, John:
> From: Wang, Yipeng1
> > This commit improves the programmer guide of the hash library to be more
> > accurate on new features introduced in 18.11.
>
> Acked-by: John McNamara
Applied, thanks
22/11/2018 18:54, Mcnamara, John:
> From: Bie, Tiwei
> > The --server-socket-path and --client-socket-path options have already
> > been removed. So also remove them from the doc to avoid confusion.
>
> Acked-by: John McNamara
Applied, thanks
26 matches
Mail list logo
