[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[PATCH v4 08/10] xen/arm: smmuv3: Remove unnecessary oom message



From: Zhen Lei <thunder.leizhen@xxxxxxxxxx>

Backport Linux commit affa909571b0. This is the clean backport without
any changes.

Fixes scripts/checkpatch.pl warning:
WARNING: Possible unnecessary 'out of memory' message

Remove it can help us save a bit of memory.

Signed-off-by: Zhen Lei <thunder.leizhen@xxxxxxxxxx>
Link: 
https://lore.kernel.org/r/20210609125438.14369-1-thunder.leizhen@xxxxxxxxxx
Signed-off-by: Will Deacon <will@xxxxxxxxxx>
Origin: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 
affa909571b0
Signed-off-by: Rahul Singh <rahul.singh@xxxxxxx>
Acked-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
---
Changes in v4:
- Move Stefano Acked-by after Signed-off
Changes in v3:
 - Added Stefano Acked-by
Changes in v2:
 - fix commit msg
---
 xen/drivers/passthrough/arm/smmu-v3.c | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/xen/drivers/passthrough/arm/smmu-v3.c 
b/xen/drivers/passthrough/arm/smmu-v3.c
index c53d4c96a4..c47e7dea04 100644
--- a/xen/drivers/passthrough/arm/smmu-v3.c
+++ b/xen/drivers/passthrough/arm/smmu-v3.c
@@ -1647,10 +1647,8 @@ static int arm_smmu_init_l1_strtab(struct 
arm_smmu_device *smmu)
        void *strtab = smmu->strtab_cfg.strtab;
 
        cfg->l1_desc = _xzalloc(size, sizeof(void *));
-       if (!cfg->l1_desc) {
-               dev_err(smmu->dev, "failed to allocate l1 stream table desc\n");
+       if (!cfg->l1_desc)
                return -ENOMEM;
-       }
 
        for (i = 0; i < cfg->num_l1_ents; ++i) {
                arm_smmu_write_strtab_l1_desc(strtab, &cfg->l1_desc[i]);
@@ -2435,10 +2433,8 @@ static int arm_smmu_device_probe(struct platform_device 
*pdev)
        struct arm_smmu_device *smmu;
 
        smmu = xzalloc(struct arm_smmu_device);
-       if (!smmu) {
-               dev_err(pdev, "failed to allocate arm_smmu_device\n");
+       if (!smmu)
                return -ENOMEM;
-       }
        smmu->dev = pdev;
 
        if (pdev->of_node) {
-- 
2.25.1




 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.