From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 4CCB1A04F0; Fri, 27 Dec 2019 12:11:50 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id B8DB21C01B; Fri, 27 Dec 2019 12:11:45 +0100 (CET) Received: from mail-wr1-f65.google.com (mail-wr1-f65.google.com [209.85.221.65]) by dpdk.org (Postfix) with ESMTP id 7279D1C00F for ; Fri, 27 Dec 2019 12:11:43 +0100 (CET) Received: by mail-wr1-f65.google.com with SMTP id c14so25780776wrn.7 for ; Fri, 27 Dec 2019 03:11:43 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind.com; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=tputKj1N2bJff24naFUwh6yuHKoUwJulBhwfEZLBt7o=; b=CS1tVHLkWq3HzFFXApoks0WBKRtbzw2GAqAlLL25z793RIkwkOIYoOhaHiYECSTEWt Qqp9f8hUchNO2DuWAf2PX4SeFp74MF3VpsdohfqNqGdcf48NkXGkkfqPqSvsN9i7EXbe RxdUMNOF57JBInH+5JQkxEzgmXBD+2MP1zVfPFSJUQMFcOa+Uj+q/A6UxJLXp49HPyPx M2T/jVm0d7KLmBSaWRAba5eSvSolDyTnZZnINvL002a8k9Yd1A8Am88ODM2uPCmhM4Gi cYfv/O44l73XB+YrEGX731oaqOySk5JRV2kC+PAmaabZtgSsU5Qcxh48xhAxXNN3ydLL 4fjg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=tputKj1N2bJff24naFUwh6yuHKoUwJulBhwfEZLBt7o=; b=ajhttvE+bOcNbRebPPxxJ6a5WHC2DUbWxas8zFqDtK4RCsMK5CAln5lPsDkFbFSycp H/3b3ySS2aG6wcfeMAwyOhXATEYJFkTqvior1/yCYwnI7O8OVTqM3mdxf9QBaIhTjWlA SndGYY1dVePyghYLTi6ZlVB4KjrW4lyXhZi5sekfRkYGERDvFqnK5RH26NDwJperjCuO vKyJtRWUS2Vi7agaGyCi18xPzckurRdvJHSrciDILGWenHz1H1QyooGwW2bPKz0zJATV 3lwstAnjM0ayfmDX66Z0k9Sq4PE5OWZvcL3xDV4mQ52n8IP2xc/h8b30c+HS0zK5Pgdz Pl/Q== X-Gm-Message-State: APjAAAW0lgvmHKULK61/yiiggNNty80b//0KEKzVwvmmSHPHRxj85jK5 qpR2YjqWxsgzRqi04wOB4Yu/lOnfkUI= X-Google-Smtp-Source: APXvYqzp60s0LjzYmlKuxhWSS3RQH6oRKuDPPiT+/xTWgrJsT6gYOzDEPuKn6njJ5WvZ7C4zEeDUgw== X-Received: by 2002:adf:d4ca:: with SMTP id w10mr47186793wrk.53.1577445102881; Fri, 27 Dec 2019 03:11:42 -0800 (PST) Received: from 6wind.com (2a01cb0c0005a600345636f7e65ed1a0.ipv6.abo.wanadoo.fr. [2a01:cb0c:5:a600:3456:36f7:e65e:d1a0]) by smtp.gmail.com with ESMTPSA id x7sm33914964wrq.41.2019.12.27.03.11.42 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 27 Dec 2019 03:11:42 -0800 (PST) Date: Fri, 27 Dec 2019 12:11:41 +0100 From: Olivier Matz To: Bao-Long Tran Cc: anatoly.burakov@intel.com, arybchenko@solarflare.com, dev@dpdk.org, users@dpdk.org, ricudis@niometrics.com Message-ID: <20191227111141.GO22738@platinum> References: <20191226154524.GG22738@platinum> <20191227081122.GL22738@platinum> <6C9B12E3-7C6C-4D0F-981B-12A49F71E467@niometrics.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <6C9B12E3-7C6C-4D0F-981B-12A49F71E467@niometrics.com> User-Agent: Mutt/1.10.1 (2018-07-13) Subject: Re: [dpdk-dev] Inconsistent behavior of mempool with regards to hugepage allocation X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi Bao-Long, On Fri, Dec 27, 2019 at 06:05:57PM +0800, Bao-Long Tran wrote: > Hi Olivier, > > > On 27 Dec 2019, at 4:11 PM, Olivier Matz wrote: > > > > On Thu, Dec 26, 2019 at 04:45:24PM +0100, Olivier Matz wrote: > >> Hi Bao-Long, > >> > >> On Mon, Dec 23, 2019 at 07:09:29PM +0800, Bao-Long Tran wrote: > >>> Hi, > >>> > >>> I'm not sure if this is a bug, but I've seen an inconsistency in the behavior > >>> of DPDK with regards to hugepage allocation for rte_mempool. Basically, for the > >>> same mempool size, the number of hugepages allocated changes from run to run. > >>> > >>> Here's how I reproduce with DPDK 19.11. IOVA=pa (default) > >>> > >>> 1. Reserve 16x1G hugepages on socket 0 > >>> 2. Replace examples/skeleton/basicfwd.c with the code below, build and run > >>> make && ./build/basicfwd > >>> 3. At the same time, watch the number of hugepages allocated > >>> "watch -n.1 ls /dev/hugepages" > >>> 4. Repeat step 2 > >>> > >>> If you can reproduce, you should see that for some runs, DPDK allocates 5 > >>> hugepages, other times it allocates 6. When it allocates 6, if you watch the > >>> output from step 3., you will see that DPDK first try to allocate 5 hugepages, > >>> then unmap all 5, retry, and got 6. > >> > >> I cannot reproduce in the same conditions than yours (with 16 hugepages > >> on socket 0), but I think I can see a similar issue: > >> > >> If I reserve at least 6 hugepages, it seems reproducible (6 hugepages > >> are used). If I reserve 5 hugepages, it takes more time, > >> taking/releasing hugepages several times, and it finally succeeds with 5 > >> hugepages. > > My apology: I just checked again, I was using DPDK 19.05, not 19.11 or master. > Let me try to see if I can repro my issue with 19.11. Sorry for the confusion. > > I also saw your patch to reduce wasted memory (eba11e). Seems like it resolves > the problem with the IOVA-contig constraint that I described in my first message. > I'll look into it to confirm. > > If I cannot repro my issue (different number of hugepages) with 19.11, from our > side we can upgrade to 19.11 and that's all we need for now. But let me also try > to repro the issue you described (multiple attempts to allocate hugepages). OK, thanks. Anyway, I think there is an issue on 19.11. And it is is even worse with 2M hugepages. Let's say we reserve 500x 2M hugepages, and try to allocate a mempool of 5G: 1/ mempool_populate tries to allocate in one virtually contiguous block, which maps all 500 hugepages, then fail, unmapping them 2/ it tries to allocate the largest zone, which returns ~2MB. 3/ this zone is added to the mempool, and for that, it allocates a mem_header struct, which triggers the mapping of a new page. 4/ Back to 1... until it fails after 3 mins The memzone allocation of "largest available area" does not have the same semantic depending on the memory model (pre-mapped hugepages or not). When using dynamic hugepage mapping, it won't map any additional hugepage. To solve the issue, we could either change it to allocate all available hugepages, or change mempool populate, by not using the "largest available area" allocation, doing the search by ourself. > > >> > >>> For our use case, it's important that DPDK allocate the same number of > >>> hugepages on every run so we can get reproducable results. > >> > >> One possibility is to use the --legacy-mem EAL option. It will try to > >> reserve all hugepages first. > > > > Passing --socket-mem=5120,0 also does the job. > > > > >>> Studying the code, this seems to be the behavior of > >>> rte_mempool_populate_default(). If I understand correctly, if the first try fail > >>> to get 5 IOVA-contiguous pages, it retries, relaxing the IOVA-contiguous > >>> condition, and eventually wound up with 6 hugepages. > >> > >> No, I think you don't have the IOVA-contiguous constraint in your > >> case. This is what I see: > >> > >> a- reserve 5 hugepages on socket 0, and start your patched basicfwd > >> b- it tries to allocate 2097151 objects of size 2304, pg_size = 1073741824 > >> c- the total element size (with header) is 2304 + 64 = 2368 > >> d- in rte_mempool_op_calc_mem_size_helper(), it calculates > >> obj_per_page = 453438 (453438 * 2368 = 1073741184) > >> mem_size = 4966058495 > >> e- it tries to allocate 4966058495 bytes, which is less than 5 x 1G, with: > >> rte_memzone_reserve_aligned(name, size=4966058495, socket=0, > >> mz_flags=RTE_MEMZONE_1GB|RTE_MEMZONE_SIZE_HINT_ONLY, > >> align=64) > >> For some reason, it fails: we can see that the number of map'd hugepages > >> increases in /dev/hugepages, the return to its original value. > >> I don't think it should fail here. > >> f- then, it will try to allocate the biggest available contiguous zone. In > >> my case, it is 1055291776 bytes (almost all the uniq map'd hugepage). > >> This is a second problem: if we call it again, it returns NULL, because > >> it won't map another hugepage. > >> g- by luck, calling rte_mempool_populate_virt() allocates a small aera > >> (mempool header), and it triggers the mapping a a new hugepage, that > >> will be used in the next loop, back at step d with a smaller mem_size. > >> > > >>> Questions: > >>> 1. Why does the API sometimes fail to get IOVA contig mem, when hugepage memory > >>> is abundant? > >> > >> In my case, it looks that we have a bit less than 1G which is free at > >> the end of the heap, than we call rte_memzone_reserve_aligned(size=5G). > >> The allocator ends up in mapping 5 pages (and fail), while only 4 is > >> needed. > >> > >> Anatoly, do you have any idea? Shouldn't we take in account the amount > >> of free space at the end of the heap when expanding? > >> > >>> 2. Why does the 2nd retry need N+1 hugepages? > >> > >> When the first alloc fails, the mempool code tries to allocate in > >> several chunks which are not virtually contiguous. This is needed in > >> case the memory is fragmented. > >> > >>> Some insights for Q1: From my experiments, seems like the IOVA of the first > >>> hugepage is not guaranteed to be at the start of the IOVA space (understandably). > >>> It could explain the retry when the IOVA of the first hugepage is near the end of > >>> the IOVA space. But I have also seen situation where the 1st hugepage is near > >>> the beginning of the IOVA space and it still failed the 1st time. > >>> > >>> Here's the code: > >>> #include > >>> #include > >>> > >>> int > >>> main(int argc, char *argv[]) > >>> { > >>> struct rte_mempool *mbuf_pool; > >>> unsigned mbuf_pool_size = 2097151; > >>> > >>> int ret = rte_eal_init(argc, argv); > >>> if (ret < 0) > >>> rte_exit(EXIT_FAILURE, "Error with EAL initialization\n"); > >>> > >>> printf("Creating mbuf pool size=%u\n", mbuf_pool_size); > >>> mbuf_pool = rte_pktmbuf_pool_create("MBUF_POOL", mbuf_pool_size, > >>> 256, 0, RTE_MBUF_DEFAULT_BUF_SIZE, 0); > >>> > >>> printf("mbuf_pool %p\n", mbuf_pool); > >>> > >>> return 0; > >>> } > >>> > >>> Best regards, > >>> BL > >> > >> Regards, > >> Olivier > > Thanks, > BL >