dev.dpdk.org archive mirror
 help / color / mirror / Atom feed
From: Rajesh Ravi <rajesh.ravi@broadcom.com>
To: David Marchand <david.marchand@redhat.com>
Cc: Ajit Khaparde <ajit.khaparde@broadcom.com>, dev <dev@dpdk.org>,
	 Jonathan Richardson <jonathan.richardson@broadcom.com>,
	 Scott Branden <scott.branden@broadcom.com>,
	 Vikram Mysore Prakash <vikram.prakash@broadcom.com>,
	Srinath Mannam <srinath.mannam@broadcom.com>,
	 "Burakov, Anatoly" <anatoly.burakov@intel.com>
Subject: Re: [dpdk-dev] [PATCH] eal: add option --iso-cmem for external custom memory
Date: Thu, 7 Nov 2019 21:46:23 +0530	[thread overview]
Message-ID: <CAAr5zNe5UrWx6nXF3=Hav9VVc7vuytfimuZ_fVtxiXtWOvxArg@mail.gmail.com> (raw)
In-Reply-To: <CAJFAV8wjFG-vsJ3G-WjuHHUs8QD4A9EYcwYT4z2jdoHvgU7bLg@mail.gmail.com>

Thanks David.

With Anatoly's patch applied my patch is not needed as the purpose is
solved.

Regards,
Rajesh


On Thu, Nov 7, 2019 at 9:21 PM David Marchand <david.marchand@redhat.com>
wrote:

> On Wed, Nov 6, 2019 at 2:55 PM David Marchand <david.marchand@redhat.com>
> wrote:
> >
> > On Tue, Nov 5, 2019 at 6:14 PM Burakov, Anatoly
> > <anatoly.burakov@intel.com> wrote:
> > >
> > > On 05-Nov-19 3:18 PM, Burakov, Anatoly wrote:
> > > > On 05-Nov-19 2:10 PM, Rajesh Ravi wrote:
> > > >> Thanks a lot Anatoly.
> > > >> Will the same solution work with  DPDK 19.02 as well? We 're
> actually
> > > >> using DPDK 19.02 for memory allocations for SPDK 19.07.
> > > >> DPDK 19.11 may not be supported by SPDK 19.07 we 're currently
> using.
> > > >>
> > > >> I 'll definitely test if the patch works with DPDK 19.02
> > > >>
> > > >> Thanks again for your time and help.
> > > >>
> > > >> Regards,
> > > >> Rajesh
> > > >
> > > > Hi Rajesh,
> > > >
> > > > Please test the patch [1] and see if it works for you. The patch is
> not
> > > > intended for 19.02 (in fact, 19.02 is not even supported any more),
> but
> > > > it may still apply. In the meantime, i'm working on a different patch
> > > > for 18.11, that will achieve the same thing using different means.
> > > >
> > > > [1] http://patches.dpdk.org/patch/62468/
> > > >
> > >
> > > The following is an alternative implementation that is based off 18.11:
> > >
> > > http://patches.dpdk.org/patch/62486/
> >
>
> Thanks Rajesh, following your test report, I applied Anatoly patch.
> A backport will go to 18.11.
>
> I will mark this current proposal as rejected.
> If you think it should still be considered, then please set this patch
> state back to new in patchwork and explain what is missing in this
> thread.
>
>
> Thanks.
> --
> David Marchand
>
>

-- 
Regards,
Rajesh

      reply	other threads:[~2019-11-08 16:17 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-10-15  5:30 [dpdk-dev] [PATCH] eal: add option --iso-cmem for external custom memory Ajit Khaparde
2019-10-17 15:45 ` Burakov, Anatoly
2019-10-18 10:54   ` Rajesh Ravi
2019-10-18 16:53     ` Burakov, Anatoly
2019-10-21 15:46       ` Rajesh Ravi
2019-10-22  7:56         ` Rajesh Ravi
2019-10-24 11:43           ` Burakov, Anatoly
2019-10-25 12:53             ` Rajesh Ravi
2019-10-25 15:02               ` Burakov, Anatoly
2019-10-30 19:50                 ` Rajesh Ravi
2019-11-04 10:25                   ` Burakov, Anatoly
2019-11-05 11:41                     ` Burakov, Anatoly
2019-11-05 14:10                       ` Rajesh Ravi
2019-11-05 15:18                         ` Burakov, Anatoly
2019-11-05 17:13                           ` Burakov, Anatoly
2019-11-06 13:55                             ` David Marchand
2019-11-07 15:51                               ` David Marchand
2019-11-07 16:16                                 ` Rajesh Ravi [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAAr5zNe5UrWx6nXF3=Hav9VVc7vuytfimuZ_fVtxiXtWOvxArg@mail.gmail.com' \
    --to=rajesh.ravi@broadcom.com \
    --cc=ajit.khaparde@broadcom.com \
    --cc=anatoly.burakov@intel.com \
    --cc=david.marchand@redhat.com \
    --cc=dev@dpdk.org \
    --cc=jonathan.richardson@broadcom.com \
    --cc=scott.branden@broadcom.com \
    --cc=srinath.mannam@broadcom.com \
    --cc=vikram.prakash@broadcom.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).