block/ps3vram: Check return of ps3vram_cache_init

SF Markus Elfring elfring at users.sourceforge.net
Tue Aug 8 22:46:13 AEST 2017


> @@ -741,7 +741,11 @@ static int ps3vram_probe(struct ps3_system_bus_device *dev)
>  		goto out_unmap_reports;
>  	}
>  
> -	ps3vram_cache_init(dev);
> +	error = ps3vram_cache_init(dev);
> +	if (error < 0) {
> +		goto out_unmap_reports;
> +	}
> +
>  	ps3vram_proc_init(dev);
>  
>  	queue = blk_alloc_queue(GFP_KERNEL);

I pointed a few development concerns out.
https://patchwork.ozlabs.org/patch/798575/

Will the commit subject “block/ps3vram: Check return value of ps3vram_cache_init()”
be more appropriate?

Regards,
Markus


More information about the Linuxppc-dev mailing list