Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/677363/?format=api
{ "id": 677363, "url": "http://patchwork.ozlabs.org/api/patches/677363/?format=api", "web_url": "http://patchwork.ozlabs.org/project/uboot/patch/20161001141931.32354-21-paul.burton@imgtec.com/", "project": { "id": 18, "url": "http://patchwork.ozlabs.org/api/projects/18/?format=api", "name": "U-Boot", "link_name": "uboot", "list_id": "u-boot.lists.denx.de", "list_email": "u-boot@lists.denx.de", "web_url": null, "scm_url": null, "webscm_url": null, "list_archive_url": "", "list_archive_url_format": "", "commit_url_format": "" }, "msgid": "<20161001141931.32354-21-paul.burton@imgtec.com>", "list_archive_url": null, "date": "2016-10-01T14:19:23", "name": "[U-Boot,20/27] image: Use ram_top, not bi_memsize, in getenv_bootm_size", "commit_ref": null, "pull_url": null, "state": "deferred", "archived": false, "hash": "5b27fd23eb86a4b9b9113d6e0db6a2f263064a4c", "submitter": { "id": 33698, "url": "http://patchwork.ozlabs.org/api/people/33698/?format=api", "name": "Paul Burton", "email": "paul.burton@imgtec.com" }, "delegate": { "id": 3651, "url": "http://patchwork.ozlabs.org/api/users/3651/?format=api", "username": "trini", "first_name": "Tom", "last_name": "Rini", "email": "trini@ti.com" }, "mbox": "http://patchwork.ozlabs.org/project/uboot/patch/20161001141931.32354-21-paul.burton@imgtec.com/mbox/", "series": [], "comments": "http://patchwork.ozlabs.org/api/patches/677363/comments/", "check": "pending", "checks": "http://patchwork.ozlabs.org/api/patches/677363/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<u-boot-bounces@lists.denx.de>", "X-Original-To": "incoming@patchwork.ozlabs.org", "Delivered-To": "patchwork-incoming@bilbo.ozlabs.org", "Received": [ "from theia.denx.de (theia.denx.de [85.214.87.163])\n\tby ozlabs.org (Postfix) with ESMTP id 3smVvK2HrQz9s2G\n\tfor <incoming@patchwork.ozlabs.org>;\n\tSun, 2 Oct 2016 00:25:01 +1000 (AEST)", "from localhost (localhost [127.0.0.1])\n\tby theia.denx.de (Postfix) with ESMTP id EC4BCA761D;\n\tSat, 1 Oct 2016 16:24:59 +0200 (CEST)", "from theia.denx.de ([127.0.0.1])\n\tby localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024)\n\twith ESMTP id mfA8eAJYQBiS; Sat, 1 Oct 2016 16:24:59 +0200 (CEST)", "from theia.denx.de (localhost [127.0.0.1])\n\tby theia.denx.de (Postfix) with ESMTP id 7B2C5A7612;\n\tSat, 1 Oct 2016 16:24:59 +0200 (CEST)", "from localhost (localhost [127.0.0.1])\n\tby theia.denx.de (Postfix) with ESMTP id 42873A7612\n\tfor <u-boot@lists.denx.de>; Sat, 1 Oct 2016 16:24:57 +0200 (CEST)", "from theia.denx.de ([127.0.0.1])\n\tby localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024)\n\twith ESMTP id Q1WDZRdpOKEg for <u-boot@lists.denx.de>;\n\tSat, 1 Oct 2016 16:24:57 +0200 (CEST)", "from mailapp01.imgtec.com (mailapp02.imgtec.com [217.156.133.132])\n\tby theia.denx.de (Postfix) with ESMTP id 11D04A7607\n\tfor <u-boot@lists.denx.de>; Sat, 1 Oct 2016 16:24:55 +0200 (CEST)", "from HHMAIL03.hh.imgtec.org (unknown [10.44.0.21])\n\tby Forcepoint Email with ESMTPS id AA75144178ECD;\n\tSat, 1 Oct 2016 15:24:50 +0100 (IST)", "from localhost (192.168.159.74) by HHMAIL03.hh.imgtec.org\n\t(10.44.0.22) with Microsoft SMTP Server (TLS) id 14.3.294.0;\n\tSat, 1 Oct 2016 15:24:52 +0100" ], "X-policyd-weight": "NOT_IN_SBL_XBL_SPAMHAUS=-1.5 NOT_IN_SPAMCOP=-1.5\n\tNOT_IN_BL_NJABL=-1.5 (only DNSBL check requested)", "From": "Paul Burton <paul.burton@imgtec.com>", "To": "<u-boot@lists.denx.de>", "Date": "Sat, 1 Oct 2016 15:19:23 +0100", "Message-ID": "<20161001141931.32354-21-paul.burton@imgtec.com>", "X-Mailer": "git-send-email 2.10.0", "In-Reply-To": "<20161001141931.32354-1-paul.burton@imgtec.com>", "References": "<20161001141931.32354-1-paul.burton@imgtec.com>", "MIME-Version": "1.0", "X-Originating-IP": "[192.168.159.74]", "Subject": "[U-Boot] [PATCH 20/27] image: Use ram_top, not bi_memsize,\n\tin getenv_bootm_size", "X-BeenThere": "u-boot@lists.denx.de", "X-Mailman-Version": "2.1.15", "Precedence": "list", "List-Id": "U-Boot discussion <u-boot.lists.denx.de>", "List-Unsubscribe": "<http://lists.denx.de/mailman/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>", "List-Archive": "<http://lists.denx.de/pipermail/u-boot/>", "List-Post": "<mailto:u-boot@lists.denx.de>", "List-Help": "<mailto:u-boot-request@lists.denx.de?subject=help>", "List-Subscribe": "<http://lists.denx.de/mailman/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>", "Content-Type": "text/plain; charset=\"us-ascii\"", "Content-Transfer-Encoding": "7bit", "Errors-To": "u-boot-bounces@lists.denx.de", "Sender": "\"U-Boot\" <u-boot-bounces@lists.denx.de>" }, "content": "When determining the region of memory to allow for use by bootm, using\nbi_memstart & adding bi_memsize can cause problems if that leads to an\ninteger overflow. For example on some MIPS systems bi_memstart would be\n0xffffffff80000000 (ie. the start of the MIPS ckseg0 region) and if the\nsystem has 2GB of memory then the addition would wrap around to 0.\n\nThe maximum amount of memory to be used by U-Boot is already accounted\nfor by the ram_top field of struct global_data, so make use of that for\nthe calculation instead.\n\nSigned-off-by: Paul Burton <paul.burton@imgtec.com>\n---\n\n common/image.c | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)", "diff": "diff --git a/common/image.c b/common/image.c\nindex 1581022..25e632b 100644\n--- a/common/image.c\n+++ b/common/image.c\n@@ -489,7 +489,7 @@ phys_size_t getenv_bootm_size(void)\n \tsize = gd->bd->bi_dram[0].size;\n #else\n \tstart = (ulong)phys_to_virt(gd->bd->bi_memstart);\n-\tsize = gd->bd->bi_memsize;\n+\tsize = gd->ram_top - start;\n #endif\n \n \ts = getenv(\"bootm_low\");\n", "prefixes": [ "U-Boot", "20/27" ] }