Message ID | alpine.LRH.2.02.2003300729320.9938@file01.intranet.prod.int.rdu2.redhat.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | [v2] memcpy_flushcache: use cache flusing for larger lengths | expand |
> -----Original Message----- > From: Mikulas Patocka <mpatocka@redhat.com> > Sent: Monday, March 30, 2020 6:32 AM > To: Dan Williams <dan.j.williams@intel.com>; Vishal Verma > <vishal.l.verma@intel.com>; Dave Jiang <dave.jiang@intel.com>; Ira > Weiny <ira.weiny@intel.com>; Mike Snitzer <msnitzer@redhat.com> > Cc: linux-nvdimm@lists.01.org; dm-devel@redhat.com > Subject: [PATCH v2] memcpy_flushcache: use cache flusing for larger > lengths > > I tested dm-writecache performance on a machine with Optane nvdimm > and it turned out that for larger writes, cached stores + cache > flushing perform better than non-temporal stores. This is the > throughput of dm- writecache measured with this command: > dd if=/dev/zero of=/dev/mapper/wc bs=64 oflag=direct > > block size 512 1024 2048 4096 > movnti 496 MB/s 642 MB/s 725 MB/s 744 MB/s > clflushopt 373 MB/s 688 MB/s 1.1 GB/s 1.2 GB/s > > We can see that for smaller block, movnti performs better, but for > larger blocks, clflushopt has better performance. There are other interactions to consider... see threads from the last few years on the linux-nvdimm list. For example, software generally expects that read()s take a long time and avoids re-reading from disk; the normal pattern is to hold the data in memory and read it from there. By using normal stores, CPU caches end up holding a bunch of persistent memory data that is probably not going to be read again any time soon, bumping out more useful data. In contrast, movnti avoids filling the CPU caches. Another option is the AVX vmovntdq instruction (if available), the most recent of which does 64-byte (cache line) sized transfers to zmm registers. There's a hefty context switching overhead (e.g., 304 clocks), and the CPU often runs AVX instructions at a slower clock frequency, so it's hard to judge when it's worthwhile. In user space, glibc faces similar choices for its memcpy() functions; glibc memcpy() uses non-temporal stores for transfers > 75% of the L3 cache size divided by the number of cores. For example, with glibc-2.216-16.fc27 (August 2017), on a Broadwell system with E5-2699 36 cores 45 MiB L3 cache, non-temporal stores are used for memcpy()s over 36 MiB. It'd be nice if glibc, PMDK, and the kernel used the same algorithms.
On Tue, 31 Mar 2020, Elliott, Robert (Servers) wrote: > > > > -----Original Message----- > > From: Mikulas Patocka <mpatocka@redhat.com> > > Sent: Monday, March 30, 2020 6:32 AM > > To: Dan Williams <dan.j.williams@intel.com>; Vishal Verma > > <vishal.l.verma@intel.com>; Dave Jiang <dave.jiang@intel.com>; Ira > > Weiny <ira.weiny@intel.com>; Mike Snitzer <msnitzer@redhat.com> > > Cc: linux-nvdimm@lists.01.org; dm-devel@redhat.com > > Subject: [PATCH v2] memcpy_flushcache: use cache flusing for larger > > lengths > > > > I tested dm-writecache performance on a machine with Optane nvdimm > > and it turned out that for larger writes, cached stores + cache > > flushing perform better than non-temporal stores. This is the > > throughput of dm- writecache measured with this command: > > dd if=/dev/zero of=/dev/mapper/wc bs=64 oflag=direct > > > > block size 512 1024 2048 4096 > > movnti 496 MB/s 642 MB/s 725 MB/s 744 MB/s > > clflushopt 373 MB/s 688 MB/s 1.1 GB/s 1.2 GB/s > > > > We can see that for smaller block, movnti performs better, but for > > larger blocks, clflushopt has better performance. > > There are other interactions to consider... see threads from the last > few years on the linux-nvdimm list. dm-writecache is the only linux driver that uses memcpy_flushcache on persistent memory. There is also the btt driver, it uses the "do_io" method to write to persistent memory and I don't know where this method comes from. Anyway, if patching memcpy_flushcache conflicts with something else, we should introduce memcpy_flushcache_to_pmem. > For example, software generally expects that read()s take a long time and > avoids re-reading from disk; the normal pattern is to hold the data in > memory and read it from there. By using normal stores, CPU caches end up > holding a bunch of persistent memory data that is probably not going to > be read again any time soon, bumping out more useful data. In contrast, > movnti avoids filling the CPU caches. But if I write one cacheline and flush it immediatelly, it would consume just one associative entry in the cache. > Another option is the AVX vmovntdq instruction (if available), the > most recent of which does 64-byte (cache line) sized transfers to > zmm registers. There's a hefty context switching overhead (e.g., > 304 clocks), and the CPU often runs AVX instructions at a slower > clock frequency, so it's hard to judge when it's worthwhile. The benchmark shows that 64-byte non-temporal avx512 vmovntdq is as good as 8, 16 or 32-bytes writes. ram nvdimm sequential write-nt 4 bytes 4.1 GB/s 1.3 GB/s sequential write-nt 8 bytes 4.1 GB/s 1.3 GB/s sequential write-nt 16 bytes (sse) 4.1 GB/s 1.3 GB/s sequential write-nt 32 bytes (avx) 4.2 GB/s 1.3 GB/s sequential write-nt 64 bytes (avx512) 4.1 GB/s 1.3 GB/s With cached writes (where each cache line is immediatelly followed by clwb or clflushopt), 8, 16 or 32-byte write performs better than non-temporal stores and avx512 performs worse. sequential write 8 + clwb 5.1 GB/s 1.6 GB/s sequential write 16 (sse) + clwb 5.1 GB/s 1.6 GB/s sequential write 32 (avx) + clwb 4.4 GB/s 1.5 GB/s sequential write 64 (avx512) + clwb 1.7 GB/s 0.6 GB/s > In user space, glibc faces similar choices for its memcpy() functions; > glibc memcpy() uses non-temporal stores for transfers > 75% of the > L3 cache size divided by the number of cores. For example, with > glibc-2.216-16.fc27 (August 2017), on a Broadwell system with > E5-2699 36 cores 45 MiB L3 cache, non-temporal stores are used > for memcpy()s over 36 MiB. BTW. what does glibc do with reads? Does it flush them from the cache after they are consumed? AFAIK glibc doesn't support persistent memory - i.e. there is no function that flushes data and the user has to use inline assembly for that. > It'd be nice if glibc, PMDK, and the kernel used the same algorithms. Mikulas
[ add x86 and LKML ] On Tue, Mar 31, 2020 at 5:27 AM Mikulas Patocka <mpatocka@redhat.com> wrote: > > > > On Tue, 31 Mar 2020, Elliott, Robert (Servers) wrote: > > > > > > > > -----Original Message----- > > > From: Mikulas Patocka <mpatocka@redhat.com> > > > Sent: Monday, March 30, 2020 6:32 AM > > > To: Dan Williams <dan.j.williams@intel.com>; Vishal Verma > > > <vishal.l.verma@intel.com>; Dave Jiang <dave.jiang@intel.com>; Ira > > > Weiny <ira.weiny@intel.com>; Mike Snitzer <msnitzer@redhat.com> > > > Cc: linux-nvdimm@lists.01.org; dm-devel@redhat.com > > > Subject: [PATCH v2] memcpy_flushcache: use cache flusing for larger > > > lengths > > > > > > I tested dm-writecache performance on a machine with Optane nvdimm > > > and it turned out that for larger writes, cached stores + cache > > > flushing perform better than non-temporal stores. This is the > > > throughput of dm- writecache measured with this command: > > > dd if=/dev/zero of=/dev/mapper/wc bs=64 oflag=direct > > > > > > block size 512 1024 2048 4096 > > > movnti 496 MB/s 642 MB/s 725 MB/s 744 MB/s > > > clflushopt 373 MB/s 688 MB/s 1.1 GB/s 1.2 GB/s > > > > > > We can see that for smaller block, movnti performs better, but for > > > larger blocks, clflushopt has better performance. > > > > There are other interactions to consider... see threads from the last > > few years on the linux-nvdimm list. > > dm-writecache is the only linux driver that uses memcpy_flushcache on > persistent memory. There is also the btt driver, it uses the "do_io" > method to write to persistent memory and I don't know where this method > comes from. > > Anyway, if patching memcpy_flushcache conflicts with something else, we > should introduce memcpy_flushcache_to_pmem. > > > For example, software generally expects that read()s take a long time and > > avoids re-reading from disk; the normal pattern is to hold the data in > > memory and read it from there. By using normal stores, CPU caches end up > > holding a bunch of persistent memory data that is probably not going to > > be read again any time soon, bumping out more useful data. In contrast, > > movnti avoids filling the CPU caches. > > But if I write one cacheline and flush it immediatelly, it would consume > just one associative entry in the cache. > > > Another option is the AVX vmovntdq instruction (if available), the > > most recent of which does 64-byte (cache line) sized transfers to > > zmm registers. There's a hefty context switching overhead (e.g., > > 304 clocks), and the CPU often runs AVX instructions at a slower > > clock frequency, so it's hard to judge when it's worthwhile. > > The benchmark shows that 64-byte non-temporal avx512 vmovntdq is as good > as 8, 16 or 32-bytes writes. > ram nvdimm > sequential write-nt 4 bytes 4.1 GB/s 1.3 GB/s > sequential write-nt 8 bytes 4.1 GB/s 1.3 GB/s > sequential write-nt 16 bytes (sse) 4.1 GB/s 1.3 GB/s > sequential write-nt 32 bytes (avx) 4.2 GB/s 1.3 GB/s > sequential write-nt 64 bytes (avx512) 4.1 GB/s 1.3 GB/s > > With cached writes (where each cache line is immediatelly followed by clwb > or clflushopt), 8, 16 or 32-byte write performs better than non-temporal > stores and avx512 performs worse. > > sequential write 8 + clwb 5.1 GB/s 1.6 GB/s > sequential write 16 (sse) + clwb 5.1 GB/s 1.6 GB/s > sequential write 32 (avx) + clwb 4.4 GB/s 1.5 GB/s > sequential write 64 (avx512) + clwb 1.7 GB/s 0.6 GB/s This is indeed compelling straight-line data. My concern, similar to Robert's, is what it does to the rest of the system. In addition to increasing cache pollution, which I agree is difficult to quantify, it may also increase read-for-ownership traffic. Could you collect 'perf stat' for this clwb vs nt comparison to check if any of this incidental overhead effect shows up in the numbers? Here is a 'perf stat' line that might capture that. perf stat -e L1-dcache-loads,L1-dcache-load-misses,L1-dcache-stores,L1-dcache-store-misses,L1-dcache-prefetch-misses,LLC-loads,LLC-load-misses,LLC-stores,LLC-store-misses,LLC-prefetch-misses -r 5 $benchmark In both cases nt and explicit clwb there's nothing that prevents the dirty-cacheline, or the fill buffer from being written-back / flushed before the full line is populated and maybe you are hitting that scenario differently with the two approaches? I did not immediately see a perf counter for events like this. Going forward I think this gets better with the movdir64b instruction because that can guarantee full-line-sized store-buffer writes. Maybe the perf data can help make a decision about whether we go with your patch in the near term? > > > > In user space, glibc faces similar choices for its memcpy() functions; > > glibc memcpy() uses non-temporal stores for transfers > 75% of the > > L3 cache size divided by the number of cores. For example, with > > glibc-2.216-16.fc27 (August 2017), on a Broadwell system with > > E5-2699 36 cores 45 MiB L3 cache, non-temporal stores are used > > for memcpy()s over 36 MiB. > > BTW. what does glibc do with reads? Does it flush them from the cache > after they are consumed? > > AFAIK glibc doesn't support persistent memory - i.e. there is no function > that flushes data and the user has to use inline assembly for that. Yes, and I don't know of any copy routines that try to limit the cache pollution of pulling the source data for a copy, only the destination. > > It'd be nice if glibc, PMDK, and the kernel used the same algorithms. Yes, it would. Although I think PMDK would make a different decision than the kernel when optimizing for highest bandwidth for the local application vs bandwidth efficiency across all applications.
On Tue, 31 Mar 2020, Dan Williams wrote: > > The benchmark shows that 64-byte non-temporal avx512 vmovntdq is as good > > as 8, 16 or 32-bytes writes. > > ram nvdimm > > sequential write-nt 4 bytes 4.1 GB/s 1.3 GB/s > > sequential write-nt 8 bytes 4.1 GB/s 1.3 GB/s > > sequential write-nt 16 bytes (sse) 4.1 GB/s 1.3 GB/s > > sequential write-nt 32 bytes (avx) 4.2 GB/s 1.3 GB/s > > sequential write-nt 64 bytes (avx512) 4.1 GB/s 1.3 GB/s > > > > With cached writes (where each cache line is immediatelly followed by clwb > > or clflushopt), 8, 16 or 32-byte write performs better than non-temporal > > stores and avx512 performs worse. > > > > sequential write 8 + clwb 5.1 GB/s 1.6 GB/s > > sequential write 16 (sse) + clwb 5.1 GB/s 1.6 GB/s > > sequential write 32 (avx) + clwb 4.4 GB/s 1.5 GB/s > > sequential write 64 (avx512) + clwb 1.7 GB/s 0.6 GB/s > > This is indeed compelling straight-line data. My concern, similar to > Robert's, is what it does to the rest of the system. In addition to > increasing cache pollution, which I agree is difficult to quantify, it I've made this program that measures cache pollution: http://people.redhat.com/~mpatocka/testcases/pmem/misc/l1-test.c - it fills the L1 cache with random pointers, so that memory prediction won't help, and then walks these pointers before and after the task that we want to measure. The results are: on RAM, there is not much difference - i.e. nt write is flushing cache as much as clflushopt and clwb: nt write: 8514 - 21034 clflushopt: 8516 - 21798 clwb: 8516 - 22882 But on PMEM, non-temporal stores perform much better and they perform even better than on RAM: nt write: 8514 - 11694 clflushopt: 8514 - 20816 clwb: 8514 - 21480 However, both dm-writecache and the nova filesystem perform better if we use cache flushing instead of nt writes: http://people.redhat.com/~mpatocka/testcases/pmem/benchmarks/fs-bench.txt > may also increase read-for-ownership traffic. Could you collect 'perf > stat' for this clwb vs nt comparison to check if any of this > incidental overhead effect shows up in the numbers? Here is a 'perf > stat' line that might capture that. > > perf stat -e L1-dcache-loads,L1-dcache-load-misses,L1-dcache-stores,L1-dcache-store-misses,L1-dcache-prefetch-misses,LLC-loads,LLC-load-misses,LLC-stores,LLC-store-misses,LLC-prefetch-misses > -r 5 $benchmark > > In both cases nt and explicit clwb there's nothing that prevents the > dirty-cacheline, or the fill buffer from being written-back / flushed > before the full line is populated and maybe you are hitting that > scenario differently with the two approaches? I did not immediately > see a perf counter for events like this. Going forward I think this > gets better with the movdir64b instruction because that can guarantee > full-line-sized store-buffer writes. > > Maybe the perf data can help make a decision about whether we go with > your patch in the near term? These are results for 6 tests: 1. movntiq on pmem 2. 8 writes + clflushopt on pmem 3. 8 writes + clwb on pmem 4. movntiq on ram 5. 8 writes + clflushopt on ram 6. 8 writes + clwb on ram perf stat -e L1-dcache-loads,L1-dcache-load-misses,L1-dcache-stores,L1-dcache-store-misses,L1-dcache-prefetch-misses,LLC-loads,LLC-load-misses,LLC-stores,LLC-store-misses,LLC-prefetch-misses -r 5 ./thrp-write-nt-8 /dev/dax3.0 thr: 1.280840 GB/s, lat: 6.245904 nsec thr: 1.281988 GB/s, lat: 6.240310 nsec thr: 1.281000 GB/s, lat: 6.245120 nsec thr: 1.278589 GB/s, lat: 6.256896 nsec thr: 1.280094 GB/s, lat: 6.249541 nsec Performance counter stats for './thrp-write-nt-8 /dev/dax3.0' (5 runs): 814899605 L1-dcache-loads ( +- 0.04% ) (42.86%) 8924277 L1-dcache-load-misses # 1.10% of all L1-dcache hits ( +- 0.19% ) (57.15%) 810672184 L1-dcache-stores ( +- 0.02% ) (57.15%) <not supported> L1-dcache-store-misses <not supported> L1-dcache-prefetch-misses 100254 LLC-loads ( +- 9.58% ) (57.15%) 6990 LLC-load-misses # 6.97% of all LL-cache hits ( +- 5.08% ) (57.14%) 16509 LLC-stores ( +- 1.38% ) (28.57%) 5070 LLC-store-misses ( +- 3.28% ) (28.57%) <not supported> LLC-prefetch-misses 5.62889 +- 0.00357 seconds time elapsed ( +- 0.06% ) perf stat -e L1-dcache-loads,L1-dcache-load-misses,L1-dcache-stores,L1-dcache-store-misses,L1-dcache-prefetch-misses,LLC-loads,LLC-load-misses,LLC-stores,LLC-store-misses,LLC-prefetch-misses -r 5 ./thrp-write-8-clflushopt /dev/dax3.0 thr: 1.611084 GB/s, lat: 4.965600 nsec thr: 1.598570 GB/s, lat: 5.004474 nsec thr: 1.600563 GB/s, lat: 4.998243 nsec thr: 1.596818 GB/s, lat: 5.009964 nsec thr: 1.593989 GB/s, lat: 5.018856 nsec Performance counter stats for './thrp-write-8-clflushopt /dev/dax3.0' (5 runs): 137415972 L1-dcache-loads ( +- 1.28% ) (42.84%) 136513938 L1-dcache-load-misses # 99.34% of all L1-dcache hits ( +- 1.24% ) (57.13%) 1153397051 L1-dcache-stores ( +- 1.29% ) (57.14%) <not supported> L1-dcache-store-misses <not supported> L1-dcache-prefetch-misses 168100 LLC-loads ( +- 0.84% ) (57.15%) 3975 LLC-load-misses # 2.36% of all LL-cache hits ( +- 2.41% ) (57.16%) 58441682 LLC-stores ( +- 1.38% ) (28.57%) 2493 LLC-store-misses ( +- 6.80% ) (28.56%) <not supported> LLC-prefetch-misses 5.7029 +- 0.0582 seconds time elapsed ( +- 1.02% ) perf stat -e L1-dcache-loads,L1-dcache-load-misses,L1-dcache-stores,L1-dcache-store-misses,L1-dcache-prefetch-misses,LLC-loads,LLC-load-misses,LLC-stores,LLC-store-misses,LLC-prefetch-misses -r 5 ./thrp-write-8-clwb /dev/dax3.0 thr: 1.595520 GB/s, lat: 5.014039 nsec thr: 1.598659 GB/s, lat: 5.004194 nsec thr: 1.599901 GB/s, lat: 5.000309 nsec thr: 1.603323 GB/s, lat: 4.989636 nsec thr: 1.608657 GB/s, lat: 4.973093 nsec Performance counter stats for './thrp-write-8-clwb /dev/dax3.0' (5 runs): 135421993 L1-dcache-loads ( +- 0.06% ) (42.85%) 134869685 L1-dcache-load-misses # 99.59% of all L1-dcache hits ( +- 0.02% ) (57.14%) 1138042172 L1-dcache-stores ( +- 0.02% ) (57.14%) <not supported> L1-dcache-store-misses <not supported> L1-dcache-prefetch-misses 184600 LLC-loads ( +- 0.79% ) (57.15%) 5756 LLC-load-misses # 3.12% of all LL-cache hits ( +- 5.23% ) (57.15%) 55755196 LLC-stores ( +- 0.04% ) (28.57%) 4928 LLC-store-misses ( +- 4.19% ) (28.56%) <not supported> LLC-prefetch-misses 5.63954 +- 0.00987 seconds time elapsed ( +- 0.18% ) perf stat -e L1-dcache-loads,L1-dcache-load-misses,L1-dcache-stores,L1-dcache-store-misses,L1-dcache-prefetch-misses,LLC-loads,LLC-load-misses,LLC-stores,LLC-store-misses,LLC-prefetch-misses -r 5 ./thrp-write-nt-8 /dev/ram0 thr: 4.156424 GB/s, lat: 1.924732 nsec thr: 4.156363 GB/s, lat: 1.924760 nsec thr: 4.159350 GB/s, lat: 1.923377 nsec thr: 4.162535 GB/s, lat: 1.921906 nsec thr: 4.158470 GB/s, lat: 1.923784 nsec Performance counter stats for './thrp-write-nt-8 /dev/ram0' (5 runs): 3077534777 L1-dcache-loads ( +- 0.14% ) (42.85%) 49870893 L1-dcache-load-misses # 1.62% of all L1-dcache hits ( +- 0.82% ) (57.14%) 2854270644 L1-dcache-stores ( +- 0.01% ) (57.14%) <not supported> L1-dcache-store-misses <not supported> L1-dcache-prefetch-misses 5391862 LLC-loads ( +- 0.29% ) (57.15%) 5190166 LLC-load-misses # 96.26% of all LL-cache hits ( +- 0.23% ) (57.15%) 5694448 LLC-stores ( +- 0.39% ) (28.57%) 5544968 LLC-store-misses ( +- 0.37% ) (28.56%) <not supported> LLC-prefetch-misses 5.61044 +- 0.00145 seconds time elapsed ( +- 0.03% ) perf stat -e L1-dcache-loads,L1-dcache-load-misses,L1-dcache-stores,L1-dcache-store-misses,L1-dcache-prefetch-misses,LLC-loads,LLC-load-misses,LLC-stores,LLC-store-misses,LLC-prefetch-misses -r 5 ./thrp-write-8-clflushopt /dev/ram0 thr: 5.923164 GB/s, lat: 1.350629 nsec thr: 5.922262 GB/s, lat: 1.350835 nsec thr: 5.921674 GB/s, lat: 1.350969 nsec thr: 5.922305 GB/s, lat: 1.350825 nsec thr: 5.921393 GB/s, lat: 1.351033 nsec Performance counter stats for './thrp-write-8-clflushopt /dev/ram0' (5 runs): 935965584 L1-dcache-loads ( +- 0.34% ) (42.85%) 521443969 L1-dcache-load-misses # 55.71% of all L1-dcache hits ( +- 0.05% ) (57.15%) 4460590261 L1-dcache-stores ( +- 0.01% ) (57.15%) <not supported> L1-dcache-store-misses <not supported> L1-dcache-prefetch-misses 6242393 LLC-loads ( +- 0.32% ) (57.15%) 5727982 LLC-load-misses # 91.76% of all LL-cache hits ( +- 0.27% ) (57.15%) 54576336 LLC-stores ( +- 0.05% ) (28.57%) 54056225 LLC-store-misses ( +- 0.04% ) (28.57%) <not supported> LLC-prefetch-misses 5.79196 +- 0.00105 seconds time elapsed ( +- 0.02% ) perf stat -e L1-dcache-loads,L1-dcache-load-misses,L1-dcache-stores,L1-dcache-store-misses,L1-dcache-prefetch-misses,LLC-loads,LLC-load-misses,LLC-stores,LLC-store-misses,LLC-prefetch-misses -r 5 ./thrp-write-8-clwb /dev/ram0 thr: 5.821923 GB/s, lat: 1.374116 nsec thr: 5.818980 GB/s, lat: 1.374811 nsec thr: 5.821207 GB/s, lat: 1.374285 nsec thr: 5.818583 GB/s, lat: 1.374905 nsec thr: 5.820813 GB/s, lat: 1.374379 nsec Performance counter stats for './thrp-write-8-clwb /dev/ram0' (5 runs): 951910720 L1-dcache-loads ( +- 0.31% ) (42.84%) 512771268 L1-dcache-load-misses # 53.87% of all L1-dcache hits ( +- 0.03% ) (57.13%) 4390478387 L1-dcache-stores ( +- 0.02% ) (57.15%) <not supported> L1-dcache-store-misses <not supported> L1-dcache-prefetch-misses 5614628 LLC-loads ( +- 0.24% ) (57.16%) 5200663 LLC-load-misses # 92.63% of all LL-cache hits ( +- 0.09% ) (57.16%) 52627554 LLC-stores ( +- 0.10% ) (28.56%) 52108200 LLC-store-misses ( +- 0.16% ) (28.55%) <not supported> LLC-prefetch-misses 5.646728 +- 0.000438 seconds time elapsed ( +- 0.01% ) > > > In user space, glibc faces similar choices for its memcpy() functions; > > > glibc memcpy() uses non-temporal stores for transfers > 75% of the > > > L3 cache size divided by the number of cores. For example, with > > > glibc-2.216-16.fc27 (August 2017), on a Broadwell system with > > > E5-2699 36 cores 45 MiB L3 cache, non-temporal stores are used > > > for memcpy()s over 36 MiB. > > > > BTW. what does glibc do with reads? Does it flush them from the cache > > after they are consumed? > > > > AFAIK glibc doesn't support persistent memory - i.e. there is no function > > that flushes data and the user has to use inline assembly for that. > > Yes, and I don't know of any copy routines that try to limit the cache > pollution of pulling the source data for a copy, only the destination. > > > > It'd be nice if glibc, PMDK, and the kernel used the same algorithms. > > Yes, it would. Although I think PMDK would make a different decision > than the kernel when optimizing for highest bandwidth for the local > application vs bandwidth efficiency across all applications. Mikulas
Index: linux-2.6/arch/x86/lib/usercopy_64.c =================================================================== --- linux-2.6.orig/arch/x86/lib/usercopy_64.c 2020-03-24 15:15:36.644945091 -0400 +++ linux-2.6/arch/x86/lib/usercopy_64.c 2020-03-30 07:17:51.450290007 -0400 @@ -152,6 +152,42 @@ void __memcpy_flushcache(void *_dst, con return; } + if (static_cpu_has(X86_FEATURE_CLFLUSHOPT) && size >= 768 && likely(boot_cpu_data.x86_clflush_size == 64)) { + while (!IS_ALIGNED(dest, 64)) { + asm("movq (%0), %%r8\n" + "movnti %%r8, (%1)\n" + :: "r" (source), "r" (dest) + : "memory", "r8"); + dest += 8; + source += 8; + size -= 8; + } + do { + asm("movq (%0), %%r8\n" + "movq 8(%0), %%r9\n" + "movq 16(%0), %%r10\n" + "movq 24(%0), %%r11\n" + "movq %%r8, (%1)\n" + "movq %%r9, 8(%1)\n" + "movq %%r10, 16(%1)\n" + "movq %%r11, 24(%1)\n" + "movq 32(%0), %%r8\n" + "movq 40(%0), %%r9\n" + "movq 48(%0), %%r10\n" + "movq 56(%0), %%r11\n" + "movq %%r8, 32(%1)\n" + "movq %%r9, 40(%1)\n" + "movq %%r10, 48(%1)\n" + "movq %%r11, 56(%1)\n" + :: "r" (source), "r" (dest) + : "memory", "r8", "r9", "r10", "r11"); + clflushopt((void *)dest); + dest += 64; + source += 64; + size -= 64; + } while (size >= 64); + } + /* 4x8 movnti loop */ while (size >= 32) { asm("movq (%0), %%r8\n"