• A
    arm64: mm: Set ZONE_DMA size based on early IORT scan · 7dc792bf
    Ard Biesheuvel 提交于
    mainline inclusion
    from mainline-5.11-rc1
    commit 2b865293
    category: bugfix
    bugzilla: 50423
    CVE: NA
    
    ---------------------------------------------
    
    We recently introduced a 1 GB sized ZONE_DMA to cater for platforms
    incorporating masters that can address less than 32 bits of DMA, in
    particular the Raspberry Pi 4, which has 4 or 8 GB of DRAM, but has
    peripherals that can only address up to 1 GB (and its PCIe host
    bridge can only access the bottom 3 GB)
    
    Instructing the DMA layer about these limitations is straight-forward,
    even though we had to fix some issues regarding memory limits set in
    the IORT for named components, and regarding the handling of ACPI _DMA
    methods. However, the DMA layer also needs to be able to allocate
    memory that is guaranteed to meet those DMA constraints, for bounce
    buffering as well as allocating the backing for consistent mappings.
    
    This is why the 1 GB ZONE_DMA was introduced recently. Unfortunately,
    it turns out the having a 1 GB ZONE_DMA as well as a ZONE_DMA32 causes
    problems with kdump, and potentially in other places where allocations
    cannot cross zone boundaries. Therefore, we should avoid having two
    separate DMA zones when possible.
    
    So let's do an early scan of the IORT, and only create the ZONE_DMA
    if we encounter any devices that need it. This puts the burden on
    the firmware to describe such limitations in the IORT, which may be
    redundant (and less precise) if _DMA methods are also being provided.
    However, it should be noted that this situation is highly unusual for
    arm64 ACPI machines. Also, the DMA subsystem still gives precedence to
    the _DMA method if implemented, and so we will not lose the ability to
    perform streaming DMA outside the ZONE_DMA if the _DMA method permits
    it.
    
    [nsaenz: unified implementation with DT's counterpart]
    Signed-off-by: NArd Biesheuvel <ardb@kernel.org>
    Signed-off-by: NNicolas Saenz Julienne <nsaenzjulienne@suse.de>
    Tested-by: NJeremy Linton <jeremy.linton@arm.com>
    Acked-by: NLorenzo Pieralisi <lorenzo.pieralisi@arm.com>
    Acked-by: NHanjun Guo <guohanjun@huawei.com>
    Cc: Jeremy Linton <jeremy.linton@arm.com>
    Cc: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
    Cc: Nicolas Saenz Julienne <nsaenzjulienne@suse.de>
    Cc: Rob Herring <robh+dt@kernel.org>
    Cc: Christoph Hellwig <hch@lst.de>
    Cc: Robin Murphy <robin.murphy@arm.com>
    Cc: Hanjun Guo <guohanjun@huawei.com>
    Cc: Sudeep Holla <sudeep.holla@arm.com>
    Cc: Anshuman Khandual <anshuman.khandual@arm.com>
    Link: https://lore.kernel.org/r/20201119175400.9995-7-nsaenzjulienne@suse.deSigned-off-by: NCatalin Marinas <catalin.marinas@arm.com>
    Signed-off-by: NJing Xiangfeng <jingxiangfeng@huawei.com>
    Reviewed-by: NKefeng  Wang <wangkefeng.wang@huawei.com>
    Signed-off-by: NZheng Zengkai <zhengzengkai@huawei.com>
    7dc792bf
acpi_iort.h 2.3 KB