DPDK patches and discussions
 help / color / mirror / Atom feed
From: "Gavin Hu (Arm Technology China)" <Gavin.Hu@arm.com>
To: "Gavin Hu (Arm Technology China)" <Gavin.Hu@arm.com>,
	"jerinj@marvell.com" <jerinj@marvell.com>,
	"dev@dpdk.org" <dev@dpdk.org>
Cc: Olivier Matz <olivier.matz@6wind.com>,
	Andrew Rybchenko <arybchenko@solarflare.com>,
	David Christensen <drc@linux.vnet.ibm.com>,
	"bruce.richardson@intel.com" <bruce.richardson@intel.com>,
	"konstantin.ananyev@intel.com" <konstantin.ananyev@intel.com>,
	"hemant.agrawal@nxp.com" <hemant.agrawal@nxp.com>,
	Shahaf Shuler <shahafs@mellanox.com>,
	Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>,
	"viktorin@rehivetech.com" <viktorin@rehivetech.com>,
	"anatoly.burakov@intel.com" <anatoly.burakov@intel.com>,
	Steve Capper <Steve.Capper@arm.com>,
	Ola Liljedahl <Ola.Liljedahl@arm.com>, nd <nd@arm.com>,
	nd <nd@arm.com>
Subject: Re: [dpdk-dev] Mbuf memory alignment constraints for	(micro)architectures
Date: Tue, 12 Nov 2019 02:36:45 +0000	[thread overview]
Message-ID: <VI1PR08MB53764C6BF6E43861532C4A368F770@VI1PR08MB5376.eurprd08.prod.outlook.com> (raw)
In-Reply-To: <VI1PR08MB5376F7027BAF2C98483397798F740@VI1PR08MB5376.eurprd08.prod.outlook.com>

Hi Jerin,

> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> On Behalf Of Gavin Hu (Arm Technology
> China)
> Sent: Monday, November 11, 2019 10:01 PM
> To: jerinj@marvell.com; dev@dpdk.org
> Cc: Olivier Matz <olivier.matz@6wind.com>; Andrew Rybchenko
> <arybchenko@solarflare.com>; David Christensen <drc@linux.vnet.ibm.com>;
> bruce.richardson@intel.com; konstantin.ananyev@intel.com;
> hemant.agrawal@nxp.com; Shahaf Shuler <shahafs@mellanox.com>;
> Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>;
> viktorin@rehivetech.com; anatoly.burakov@intel.com; Steve Capper
> <Steve.Capper@arm.com>; Ola Liljedahl <Ola.Liljedahl@arm.com>; nd
> <nd@arm.com>
> Subject: Re: [dpdk-dev] Mbuf memory alignment constraints for
> (micro)architectures
> 
> Hi Jerin,
> 
> > -----Original Message-----
> > From: Jerin Jacob Kollanukkaran <jerinj@marvell.com>
> > Sent: Thursday, October 31, 2019 2:02 AM
> > To: dev@dpdk.org
> > Cc: Olivier Matz <olivier.matz@6wind.com>; Andrew Rybchenko
> > <arybchenko@solarflare.com>; David Christensen
> <drc@linux.vnet.ibm.com>;
> > bruce.richardson@intel.com; konstantin.ananyev@intel.com;
> > hemant.agrawal@nxp.com; Shahaf Shuler <shahafs@mellanox.com>;
> > Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>; Gavin Hu (Arm
> > Technology China) <Gavin.Hu@arm.com>; viktorin@rehivetech.com;
> > anatoly.burakov@intel.com
> > Subject: Mbuf memory alignment constraints for (micro)architectures
> >
> > CC:  Arch and platform maintainers
> >
> > While reviewing the mempool objection allocation requirements in the code,
> >
> > A) it's found that in the default case, mempool objects have padding
> > in the object trailer to have start addresses of objects among the different
> > channels,
> > to enable equally load on the DRAM channel to have better performance
> >
> > # More documentation is here
> > https://doc.dpdk.org/guides/prog_guide/mempool_lib.html
> > in section 8.3. Memory Alignment Constraints
> >
> > B) The optimize_object_size() does the channel distribution requirement
> > by the following formula
> >
> >         new_obj_size = (obj_size + RTE_MEMPOOL_ALIGN_MASK) /
> > RTE_MEMPOOL_ALIGN;
> >         while (get_gcd(new_obj_size, nrank * nchan) != 1)
> >                new_obj_size++;
> >
> >
> > C) The formula mentioned in the (B) is NOT generic. At least of the octeontx2
> > SoC
> > The memory/DDR controller works in different way. Where by:
> > # It does XOR operation of some  of physical address lines(not the user space
> > VA address)
> > to compute the hash and that the function defines the actual channel.
> >
> > The XOR(kind of CRC) scheme is useful because there is natural  channel
> > distribution
> > based on the address i.e No need to have padding to waste memory
> >
> > So, in short the padding scheme does not need for some SoC. I trying to send
> > the patch
> > to fix it. So the questions is,
> >
> > # Is PPC and other ARM SoC has formula (B)  to compute DRAM channel
> > distribution ? or
> > Is it specific to x86? That would define where the hooks needs to added to
> have
> > proper fix.
> Reading through some documents, both x86 and arm, and having internal
> discussion,
> it looks like this is specific to x86, x86 spreads adjacent virtual addresses within
> a page across multiple memory devices,
> the interleaving was done per one or two cache lines.
> https://software.intel.com/en-us/articles/how-memory-is-accessed
> 
> Arm leaves flexibility to implementations, no fixed pattern for interleaving and
> thus it can hardly be generalized.
Same conclusion, but more words for this topic(from Arm internally):
"Interleaving (or stripping) happens at the interconnect/memory controller level, so on Arm-based systems it's going to be highly dependent on the given SoC's integration and probably the system configuration too. Arm own interconnect and DMC IPs generally offer various options to support stripping, but even then it's the integrator's choice how to use them, and obviously there are multitudes of alternative third-party IPs too.
In summary, this really depends on the system's interconnect and memory controller capabilities and how it has been configured."
/Gavin
> >
> >
> >
> >
> >
> >
> >
> >


  reply	other threads:[~2019-11-12  2:37 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-10-30 18:02 Jerin Jacob Kollanukkaran
2019-11-11 14:01 ` Gavin Hu (Arm Technology China)
2019-11-12  2:36   ` Gavin Hu (Arm Technology China) [this message]
2019-11-13 23:08 ` David Christensen

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=VI1PR08MB53764C6BF6E43861532C4A368F770@VI1PR08MB5376.eurprd08.prod.outlook.com \
    --to=gavin.hu@arm.com \
    --cc=Honnappa.Nagarahalli@arm.com \
    --cc=Ola.Liljedahl@arm.com \
    --cc=Steve.Capper@arm.com \
    --cc=anatoly.burakov@intel.com \
    --cc=arybchenko@solarflare.com \
    --cc=bruce.richardson@intel.com \
    --cc=dev@dpdk.org \
    --cc=drc@linux.vnet.ibm.com \
    --cc=hemant.agrawal@nxp.com \
    --cc=jerinj@marvell.com \
    --cc=konstantin.ananyev@intel.com \
    --cc=nd@arm.com \
    --cc=olivier.matz@6wind.com \
    --cc=shahafs@mellanox.com \
    --cc=viktorin@rehivetech.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).