Hi Konstantin,
>
>
> Hi Akhil,
>
> > > > > > Added my comments inline with your draft.
> > > > > > [snip]..
> > > > > >
> > > > > > >
> > > > > > > Ok, then my suggestion:
> > > > > > > Let's at least write down all points about crypto-dev approach
> > > > > > > where
> we
> > > > > > > disagr
Hi Akhil,
> > > > > Added my comments inline with your draft.
> > > > > [snip]..
> > > > >
> > > > > >
> > > > > > Ok, then my suggestion:
> > > > > > Let's at least write down all points about crypto-dev approach
> > > > > > where we
> > > > > > disagree and then probably try to resolve them o
Hi Konstantin,
>
> Hi Akhil,
>
>
> > > > Added my comments inline with your draft.
> > > > [snip]..
> > > >
> > > > >
> > > > > Ok, then my suggestion:
> > > > > Let's at least write down all points about crypto-dev approach where
> > > > > we
> > > > > disagree and then probably try to resol
> > > > Added my comments inline with your draft.
> > > > [snip]..
> > > >
> > > > >
> > > > > Ok, then my suggestion:
> > > > > Let's at least write down all points about crypto-dev approach where
> > > > > we
> > > > > disagree and then probably try to resolve them one by one
> > > > > If we
Hi Akhil,
> > > Added my comments inline with your draft.
> > > [snip]..
> > >
> > > >
> > > > Ok, then my suggestion:
> > > > Let's at least write down all points about crypto-dev approach where we
> > > > disagree and then probably try to resolve them one by one
> > > > If we fail to make
Hi Konstantin,
>
>
> Hi Akhil,
>
>
> > Added my comments inline with your draft.
> > [snip]..
> >
> > >
> > > Ok, then my suggestion:
> > > Let's at least write down all points about crypto-dev approach where we
> > > disagree and then probably try to resolve them one by one
> > > If we f
Hi Akhil,
> Added my comments inline with your draft.
> >
> >
> > Hi Akhil,
> >
> > > > BTW, to be honest, I don't consider current rte_cryptodev_sym_session
> > > > construct for multiple device_ids:
> > > > __extension__ struct {
> > > > void *data;
> > > > ui
Hi Konstantin,
Added my comments inline with your draft.
>
>
> Hi Akhil,
>
> > > BTW, to be honest, I don't consider current rte_cryptodev_sym_session
> > > construct for multiple device_ids:
> > > __extension__ struct {
> > > void *data;
> > > uint16_t refcnt;
>
>
> > > > User can use the same session, that is what I am also insisting, but it
> > > > may have
> > > separate
> > > > Session private data. Cryptodev session create API provide that
> > > > functionality
> > > and we can
> > > > Leverage that.
> > >
> > > rte_cryptodev_sym_session. sess_da
Hi Akhil,
> > > User can use the same session, that is what I am also insisting, but it
> > > may have
> > separate
> > > Session private data. Cryptodev session create API provide that
> > > functionality
> > and we can
> > > Leverage that.
> >
> > rte_cryptodev_sym_session. sess_data[] is in
> -Original Message-
> From: Akhil Goyal [mailto:akhil.go...@nxp.com]
> Sent: Tuesday, October 15, 2019 4:02 PM
> To: Ananyev, Konstantin ; Zhang, Roy Fan
> ; 'dev@dpdk.org' ;
> De Lara Guarch, Pablo ; 'Thomas Monjalon'
> ; Doherty, Declan
>
> Cc: 'Anoob Joseph' ; Jerin Jacob ;
> Hem
>
>
> > Hi Akhil,
> >
> > Thanks for the review and comments!
> > Knowing you are extremely busy. Here is my point in brief:
> > I think placing the CPU synchronous crypto in the rte_security make sense,
> > as
> >
> > 1. rte_security contains inline crypto and lookaside crypto action type
>
Hi Fan,
>
> Hi Akhil,
>
> Thanks for the review and comments!
> Knowing you are extremely busy. Here is my point in brief:
> I think placing the CPU synchronous crypto in the rte_security make sense, as
>
> 1. rte_security contains inline crypto and lookaside crypto action type
> already,
> ad
> Hi Akhil,
>
> Thanks for the review and comments!
> Knowing you are extremely busy. Here is my point in brief:
> I think placing the CPU synchronous crypto in the rte_security make sense, as
>
> 1. rte_security contains inline crypto and lookaside crypto action type
> already, adding cpu_cry
Hi Akhil,
Thanks for the review and comments!
Knowing you are extremely busy. Here is my point in brief:
I think placing the CPU synchronous crypto in the rte_security make sense, as
1. rte_security contains inline crypto and lookaside crypto action type
already, adding cpu_crypto action type i
Hi Konstantin,
>
> Hi Akhil,
>
..[snip]
> > > > > > OK let us assume that you have a separate structure. But I have a
> > > > > > few
> > > queries:
> > > > > > 1. how can multiple drivers use a same session
> > > > >
> > > > > As a short answer: they can't.
> > > > > It is pretty much the sam
Hi Akhil,
> > > > > > > > > > > > > > > > > This action type allows the burst of
> > > > > > > > > > > > > > > > > symmetric crypto
> > > > > > workload
> > > > > > > > using
> > > > > > > > > > > the
> > > > > > > > > > > > > > > same
> > > > > > > > > > > > > > > > > algorithm, key, and direc
Hi Konstantin,
>
>
> Hi Akhil,
>
> > > > > > > > > > > > > > > > This action type allows the burst of symmetric
> > > > > > > > > > > > > > > > crypto
> > > > > workload
> > > > > > > using
> > > > > > > > > > the
> > > > > > > > > > > > > > same
> > > > > > > > > > > > > > > > algorithm, key,
Hi Akhil,
> > > > > > > > > > > > > > > This action type allows the burst of symmetric
> > > > > > > > > > > > > > > crypto
> > > > workload
> > > > > > using
> > > > > > > > > the
> > > > > > > > > > > > > same
> > > > > > > > > > > > > > > algorithm, key, and direction being processed by
> >
Hi Konstantin,
>
> Hi Akhil,
>
> > > > > > > > > > > > > > This action type allows the burst of symmetric
> > > > > > > > > > > > > > crypto
> > > workload
> > > > > using
> > > > > > > > the
> > > > > > > > > > > > same
> > > > > > > > > > > > > > algorithm, key, and direction being processed
Hi Akhil,
> > > > > > > > > > > > > This action type allows the burst of symmetric crypto
> > workload
> > > > using
> > > > > > > the
> > > > > > > > > > > same
> > > > > > > > > > > > > algorithm, key, and direction being processed by CPU
> > > > > > > > > > > > > cycles
> > > > > > > > > syn
Hi Konstantin,
>
> Hi Akhil,
>
> > > > > > > > > > > > This action type allows the burst of symmetric crypto
> workload
> > > using
> > > > > > the
> > > > > > > > > > same
> > > > > > > > > > > > algorithm, key, and direction being processed by CPU
> > > > > > > > > > > > cycles
> > > > > > >
Hi Akhil,
> > > > > > > > > > > This action type allows the burst of symmetric crypto
> > > > > > > > > > > workload
> > using
> > > > > the
> > > > > > > > > same
> > > > > > > > > > > algorithm, key, and direction being processed by CPU
> > > > > > > > > > > cycles
> > > > > > > synchronously.
Hi Konstantin,
> -Original Message-
> From: Ananyev, Konstantin
> Sent: Wednesday, September 25, 2019 11:54 PM
> To: Akhil Goyal ; 'dev@dpdk.org' ; De
> Lara Guarch, Pablo ; 'Thomas Monjalon'
>
> Cc: Zhang, Roy Fan ; Doherty, Declan
> ; 'Anoob Joseph'
> Subject: RE: [RFC PATCH 1/9] secu
> > > > > > > > > This action type allows the burst of symmetric crypto
> > > > > > > > > workload using
> > > the
> > > > > > > same
> > > > > > > > > algorithm, key, and direction being processed by CPU cycles
> > > > > synchronously.
> > > > > > > > > This flexible action type does not requir
Hi Akhil,
> > > > > > > > This action type allows the burst of symmetric crypto workload
> > > > > > > > using
> > the
> > > > > > same
> > > > > > > > algorithm, key, and direction being processed by CPU cycles
> > > > synchronously.
> > > > > > > > This flexible action type does not require e
Hi Konstantin,
>
> Hi Akhil,
>
> > > > > > > This action type allows the burst of symmetric crypto workload
> > > > > > > using
> the
> > > > > same
> > > > > > > algorithm, key, and direction being processed by CPU cycles
> > > synchronously.
> > > > > > > This flexible action type does not r
> Hi Akhil,
>
> > > > > > > This action type allows the burst of symmetric crypto workload
> > > > > > > using the
> > > > > same
> > > > > > > algorithm, key, and direction being processed by CPU cycles
> > > synchronously.
> > > > > > > This flexible action type does not require external hard
Hi Akhil,
> > > > > > This action type allows the burst of symmetric crypto workload
> > > > > > using the
> > > > same
> > > > > > algorithm, key, and direction being processed by CPU cycles
> > synchronously.
> > > > > > This flexible action type does not require external hardware
> > > > > >
Hi Konstantin,
> Hi Akhil,
> > >
> > > > > This action type allows the burst of symmetric crypto workload using
> > > > > the
> > > same
> > > > > algorithm, key, and direction being processed by CPU cycles
> synchronously.
> > > > > This flexible action type does not require external hardware
>
Hi lads,
> >
> > You are right, the new API will process the crypto workload, no heavy
> > enqueue
> > Dequeue operations required.
> >
> > Cryptodev tends to support multiple crypto devices, including HW and SW.
> > The 3-cache line access, iova address computation and assignment, simulation
>
Hi Akhil,
> >
> > > > This action type allows the burst of symmetric crypto workload using the
> > same
> > > > algorithm, key, and direction being processed by CPU cycles
> > > > synchronously.
> > > > This flexible action type does not require external hardware
> > > > involvement,
> > > > h
Hi Fan,
>
> Hi Akhil,
>
> You are right, the new API will process the crypto workload, no heavy enqueue
> Dequeue operations required.
>
> Cryptodev tends to support multiple crypto devices, including HW and SW.
> The 3-cache line access, iova address computation and assignment, simulation
> of
Hi Konstantin,
>
> Hi Akhil,
>
> > > This action type allows the burst of symmetric crypto workload using the
> same
> > > algorithm, key, and direction being processed by CPU cycles synchronously.
> > > This flexible action type does not require external hardware involvement,
> > > having the
Hi Akhil,
> > This action type allows the burst of symmetric crypto workload using the
> > same
> > algorithm, key, and direction being processed by CPU cycles synchronously.
> > This flexible action type does not require external hardware involvement,
> > having the crypto workload processed syn
Hi Akhil,
You are right, the new API will process the crypto workload, no heavy enqueue
Dequeue operations required.
Cryptodev tends to support multiple crypto devices, including HW and SW.
The 3-cache line access, iova address computation and assignment, simulation
of async enqueue/dequeue ope
Hi Fan,
>
> Hi Akhil,
>
> This action type allows the burst of symmetric crypto workload using the same
> algorithm, key, and direction being processed by CPU cycles synchronously.
> This flexible action type does not require external hardware involvement,
> having the crypto workload processed
Hi Akhil,
This action type allows the burst of symmetric crypto workload using the same
algorithm, key, and direction being processed by CPU cycles synchronously.
This flexible action type does not require external hardware involvement,
having the crypto workload processed synchronously, and is m
Hi Fan,
>
> This patch introduce new RTE_SECURITY_ACTION_TYPE_CPU_CRYPTO action
> type to
> security library. The type represents performing crypto operation with CPU
> cycles. The patch also includes a new API to process crypto operations in
> bulk and the function pointers for PMDs.
>
I am not
This patch introduce new RTE_SECURITY_ACTION_TYPE_CPU_CRYPTO action type to
security library. The type represents performing crypto operation with CPU
cycles. The patch also includes a new API to process crypto operations in
bulk and the function pointers for PMDs.
Signed-off-by: Fan Zhang
---
l
40 matches
Mail list logo