From: "\"Subash Patel (ಸುಭಾಷ್)\"" <subashrp@gmail.com>
To: Olof Johansson <olof@lixom.net>
Cc: linaro-mm-sig@lists.linaro.org,
Subash Patel <subash.rp@samsung.com>,
dri-devel@lists.freedesktop.org, inki.dae@samsung.com,
subash.ramaswamy@linaro.org, airlied@redhat.com
Subject: Re: [PATCH] DRM: exynos: return NULL if exynos_pages_to_sg fails
Date: Fri, 22 Jun 2012 17:54:20 -0700 [thread overview]
Message-ID: <4FE513BC.2020008@gmail.com> (raw)
In-Reply-To: <CALiw-2GBa1w68XaDG+gwCsc4oQz6JJz5b2nrGfJur9QO=KFwgw@mail.gmail.com>
Hi Olof,
I will then rework this patch commit message and send it. I will copy
Samsung-soc list (dri-devel is already in copy)
Regards,
Subash
On 06/22/2012 05:47 PM, Olof Johansson wrote:
> Hi,
>
> On Fri, Jun 22, 2012 at 5:41 PM, Subash Patel <subashrp@gmail.com
> <mailto:subashrp@gmail.com>> wrote:
>
> From: Subash Patel <subash.rp@samsung.com
> <mailto:subash.rp@samsung.com>>
>
> exynos_pages_to_sg() internally calls sg_kmalloc() which can return
> no pages when the system is under high memory crunch. One such instance
> is chromeos-install in the chromeos. This patch adds check for the
> return
> value of the function in subject to return NULL on failure.
>
> BUG=chrome-os-partner:9481
> TEST=built, ran on snow and tried chromeos-install without a crash
>
> Change-Id: I0abda74beaedae002a17de9962d7a462a2a7c2fb
> Signed-off-by: Subash Patel <subash.rp@samsung.com
> <mailto:subash.rp@samsung.com>>
>
>
> No BUG/TEST/Change-Id on a public patch, please. Also, I didn't see this
> posted on neither samsung-soc, nor the dri-devel list?
>
>
> -Olof
>
next prev parent reply other threads:[~2012-06-23 0:54 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-06-23 0:41 [PATCH] DRM: exynos: return NULL if exynos_pages_to_sg fails Subash Patel
2012-06-23 0:47 ` Olof Johansson
2012-06-23 0:54 ` "Subash Patel (ಸುಭಾಷ್)" [this message]
-- strict thread matches above, loose matches on Subject: below --
2012-06-23 1:12 Subash Patel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4FE513BC.2020008@gmail.com \
--to=subashrp@gmail.com \
--cc=airlied@redhat.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=inki.dae@samsung.com \
--cc=linaro-mm-sig@lists.linaro.org \
--cc=olof@lixom.net \
--cc=subash.ramaswamy@linaro.org \
--cc=subash.rp@samsung.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.