Re: [PATCH v4 2nd 1/4] ramoops: Add __ramoops_init_prz() as generic function

From: Mark Salyzyn
Date: Thu Feb 02 2017 - 17:13:24 EST


On 01/30/2017 05:58 PM, Nobuhiro Iwamatsu wrote:
+
+ if (zap && *paddr + sz - cxt->phys_addr > cxt->size) {
+ dev_err(dev, "no room for %s mem region (0x%zx@0x%llx) in (0x%lx@0x%llx)\n",
+ name, sz, (unsigned long long)*paddr,
+ cxt->size, (unsigned long long)cxt->phys_addr);
+ return -ENOMEM;
+ }
+
Why not allow this limit check for ramoops_init_prsz call?

-- Mark