Skip to content

Commit 9b3118f

Browse files
P33Mpelwell
authored andcommitted
DTS: set default nvme Host Memory Buffer size to 32MB on BCM2711/2
In #6504 the nvme HMB allocation was found to come from CMA so was restricted to zero on the basis of a superficial amount of testing showing no performance impact. This is not the case for high-density low-capacity DRAM-less drives, where random read can suffer up to 50% degradation. With the addition of commit 6686634 ("nvme-pci: manually allocate Host Memory Buffer segments on arm64") this restriction is no longer necessary. Bump the default to 32MiB, which is a fair compromise for increasing random read performance on smaller drives that typically have less internal parallelism and request smaller HMBs. Signed-off-by: Jonathan Bell <jonathan@raspberrypi.com>
1 parent 582cb2a commit 9b3118f

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

arch/arm/boot/dts/broadcom/bcm2711-rpi-ds.dtsi

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33

44
/ {
55
chosen {
6-
bootargs = "coherent_pool=1M 8250.nr_uarts=1 snd_bcm2835.enable_headphones=0 cgroup_disable=memory numa_policy=interleave nvme.max_host_mem_size_mb=0";
6+
bootargs = "coherent_pool=1M 8250.nr_uarts=1 snd_bcm2835.enable_headphones=0 cgroup_disable=memory numa_policy=interleave nvme.max_host_mem_size_mb=32";
77
};
88

99
__overrides__ {

arch/arm64/boot/dts/broadcom/bcm2712-rpi.dtsi

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ watchdog: &pm {};
113113

114114
/ {
115115
chosen: chosen {
116-
bootargs = "reboot=w coherent_pool=1M 8250.nr_uarts=1 pci=pcie_bus_safe cgroup_disable=memory numa_policy=interleave nvme.max_host_mem_size_mb=0";
116+
bootargs = "reboot=w coherent_pool=1M 8250.nr_uarts=1 pci=pcie_bus_safe cgroup_disable=memory numa_policy=interleave nvme.max_host_mem_size_mb=32";
117117
stdout-path = "serial10:115200n8";
118118
};
119119

0 commit comments

Comments
 (0)