From patchwork Fri May 18 22:25:12 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Anton Vorontsov X-Patchwork-Id: 8822 Return-Path: X-Original-To: patchwork@peony.canonical.com Delivered-To: patchwork@peony.canonical.com Received: from fiordland.canonical.com (fiordland.canonical.com [91.189.94.145]) by peony.canonical.com (Postfix) with ESMTP id 08FF823EB5 for ; Fri, 18 May 2012 22:26:44 +0000 (UTC) Received: from mail-yw0-f52.google.com (mail-yw0-f52.google.com [209.85.213.52]) by fiordland.canonical.com (Postfix) with ESMTP id C50B2A186E0 for ; Fri, 18 May 2012 22:26:43 +0000 (UTC) Received: by mail-yw0-f52.google.com with SMTP id p61so4113203yhp.11 for ; Fri, 18 May 2012 15:26:43 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=x-forwarded-to:x-forwarded-for:delivered-to:received-spf:date:from :to:cc:subject:message-id:references:mime-version:content-type :content-disposition:in-reply-to:user-agent:x-gm-message-state; bh=3FS2qa4Z/CxosZWsCbOvM/ehCKOI/4pEXO2saFh72BE=; b=Um1NE0k6Vc9o086a14TXb7mPe3/gxBv4j82IcLt3vvVbBwV5d+13RFsiUdDuoSZxmI 4kWJ8A5M/CAMhTuZ+2+fCRn4mqQiZP6T8q+eBwIDKgP++kyyP9KP65QCsVzzIwwGijU1 bIG42p0XgPdxPsKx0gINqw4KJS3PYlB3GKcvDJLs2u0/R1Uk0dwtbHIfJaYXlpCZ18t0 tikTar/aNI4t4L56qiQ0H74l4OAnSn3l4NAxdtttH8PZRzg1rTuyXVdsOApZWvWqXGgN CmM/n51IUIl0MyUoR03xKtWDOEMZatU50ZQgBLKAfT5KOwR1m3VnoJ2hawAOHLE3FnHy mpvQ== Received: by 10.50.87.227 with SMTP id bb3mr1953171igb.57.1337380003380; Fri, 18 May 2012 15:26:43 -0700 (PDT) X-Forwarded-To: linaro-patchwork@canonical.com X-Forwarded-For: patch@linaro.org linaro-patchwork@canonical.com Delivered-To: patches@linaro.org Received: by 10.231.35.72 with SMTP id o8csp135278ibd; Fri, 18 May 2012 15:26:42 -0700 (PDT) Received: by 10.68.192.5 with SMTP id hc5mr2344570pbc.49.1337380002792; Fri, 18 May 2012 15:26:42 -0700 (PDT) Received: from mail-pb0-f50.google.com (mail-pb0-f50.google.com [209.85.160.50]) by mx.google.com with ESMTPS id qa10si17675425pbb.174.2012.05.18.15.26.42 (version=TLSv1/SSLv3 cipher=OTHER); Fri, 18 May 2012 15:26:42 -0700 (PDT) Received-SPF: neutral (google.com: 209.85.160.50 is neither permitted nor denied by best guess record for domain of anton.vorontsov@linaro.org) client-ip=209.85.160.50; Authentication-Results: mx.google.com; spf=neutral (google.com: 209.85.160.50 is neither permitted nor denied by best guess record for domain of anton.vorontsov@linaro.org) smtp.mail=anton.vorontsov@linaro.org Received: by mail-pb0-f50.google.com with SMTP id rr4so5393549pbb.37 for ; Fri, 18 May 2012 15:26:42 -0700 (PDT) Received: by 10.68.136.69 with SMTP id py5mr42983339pbb.115.1337380002479; Fri, 18 May 2012 15:26:42 -0700 (PDT) Received: from localhost (c-71-204-165-222.hsd1.ca.comcast.net. [71.204.165.222]) by mx.google.com with ESMTPS id vp7sm3690410pbc.35.2012.05.18.15.26.40 (version=TLSv1/SSLv3 cipher=OTHER); Fri, 18 May 2012 15:26:41 -0700 (PDT) Date: Fri, 18 May 2012 15:25:12 -0700 From: Anton Vorontsov To: Greg Kroah-Hartman , Kees Cook , Colin Cross , Tony Luck Cc: Arnd Bergmann , John Stultz , Shuah Khan , arve@android.com, Rebecca Schultz Zavin , Jesper Juhl , Randy Dunlap , Stephen Boyd , Thomas Meyer , Andrew Morton , Marco Stornelli , WANG Cong , linux-kernel@vger.kernel.org, devel@driverdev.osuosl.org, linaro-kernel@lists.linaro.org, patches@linaro.org, kernel-team@android.com Subject: [PATCH 06/14] pstore/ram: Introduce ramoops_context.max_dump_count Message-ID: <20120518222512.GF23089@lizard> References: <20120518222314.GA9425@lizard> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20120518222314.GA9425@lizard> User-Agent: Mutt/1.5.21 (2010-09-15) X-Gm-Message-State: ALoCoQmexls3sDNAmXJ5zMCJtDVsWojHyx4k1/n1OITkwAToRFMLPqs7gRoj7+5IbgiTBNQ00456 So far it is the same as max_count, but this will change when we'll add support for other message types (e.g. console, mce, tracing). Signed-off-by: Anton Vorontsov --- fs/pstore/ram.c | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/fs/pstore/ram.c b/fs/pstore/ram.c index 43d9727..408fb07 100644 --- a/fs/pstore/ram.c +++ b/fs/pstore/ram.c @@ -70,6 +70,7 @@ struct ramoops_context { bool ecc; unsigned int count; unsigned int max_count; + unsigned int max_dump_count; unsigned int read_count; struct pstore_info pstore; }; @@ -94,7 +95,7 @@ static ssize_t ramoops_pstore_read(u64 *id, enum pstore_type_id *type, struct ramoops_context *cxt = psi->data; struct persistent_ram_zone *prz; - if (cxt->read_count >= cxt->max_count) + if (cxt->read_count >= cxt->max_dump_count) return -EINVAL; *id = cxt->read_count++; @@ -172,7 +173,7 @@ static int ramoops_pstore_write(enum pstore_type_id type, size = prz->buffer_size - hlen; persistent_ram_write(prz, cxt->pstore.buf, size); - cxt->count = (cxt->count + 1) % cxt->max_count; + cxt->count = (cxt->count + 1) % cxt->max_dump_count; return 0; } @@ -182,7 +183,7 @@ static int ramoops_pstore_erase(enum pstore_type_id type, u64 id, { struct ramoops_context *cxt = psi->data; - if (id >= cxt->max_count) + if (id >= cxt->max_dump_count) return -EINVAL; persistent_ram_free_old(cxt->przs[id]); @@ -213,7 +214,7 @@ static int __init ramoops_probe(struct platform_device *pdev) /* Only a single ramoops area allowed at a time, so fail extra * probes. */ - if (cxt->max_count) + if (cxt->max_dump_count) goto fail_out; if (!pdata->mem_size || !pdata->record_size) { @@ -240,6 +241,7 @@ static int __init ramoops_probe(struct platform_device *pdev) } cxt->max_count = pdata->mem_size / pdata->record_size; + cxt->max_dump_count = cxt->max_count; cxt->count = 0; cxt->size = pdata->mem_size; cxt->phys_addr = pdata->mem_address; @@ -247,14 +249,14 @@ static int __init ramoops_probe(struct platform_device *pdev) cxt->dump_oops = pdata->dump_oops; cxt->ecc = pdata->ecc; - cxt->przs = kzalloc(sizeof(*cxt->przs) * cxt->max_count, GFP_KERNEL); + cxt->przs = kzalloc(sizeof(*cxt->przs) * cxt->max_dump_count, GFP_KERNEL); if (!cxt->przs) { err = -ENOMEM; dev_err(dev, "failed to initialize a prz array\n"); goto fail_out; } - for (i = 0; i < cxt->max_count; i++) { + for (i = 0; i < cxt->max_dump_count; i++) { size_t sz = cxt->record_size; phys_addr_t start = cxt->phys_addr + sz * i; @@ -303,6 +305,7 @@ fail_buf: fail_clear: cxt->pstore.bufsize = 0; cxt->max_count = 0; + cxt->max_dump_count = 0; fail_przs: for (i = 0; cxt->przs[i]; i++) persistent_ram_free(cxt->przs[i]);