From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Jones Subject: Re: [PATCH 2/3] drivers/base: devres.c: Add block copy func. for managed devices Date: Wed, 18 Jun 2014 16:52:20 +0100 Message-ID: <53A1B5B4.2020704@codethink.co.uk> References: <1402411308-14182-1-git-send-email-rob.jones@codethink.co.uk> <1402411308-14182-3-git-send-email-rob.jones@codethink.co.uk> <1713451.AmufNoybT7@diego> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: <1713451.AmufNoybT7@diego> Sender: linux-kernel-owner@vger.kernel.org To: =?ISO-8859-1?Q?Heiko_St=FCbner?= Cc: linus.walleij@linaro.org, gnurou@gmail.com, lgirdwood@gmail.com, broonie@kernel.org, linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org, ian.molton@codethink.co.uk, ben.dooks@codethink.co.uk List-Id: linux-gpio@vger.kernel.org On 18/06/14 14:05, Heiko St=FCbner wrote: > Am Dienstag, 10. Juni 2014, 15:41:47 schrieb Rob Jones: >> Add function devm_kmemdup(). >> Rationalise with devm_kstrdup() to avoid code duplication. >> >> Reviewed-by: Ian Molton >> Signed-off-by: Rob Jones >> --- >> drivers/base/devres.c | 42 ++++++++++++++++++++++++++++++++++--= ------ >> include/linux/device.h | 2 ++ >> 2 files changed, 36 insertions(+), 8 deletions(-) >> >> diff --git a/drivers/base/devres.c b/drivers/base/devres.c >> index db4e264..978bbe2 100644 >> --- a/drivers/base/devres.c >> +++ b/drivers/base/devres.c >> @@ -791,6 +791,20 @@ void * devm_kmalloc(struct device *dev, size_t = size, >> gfp_t gfp) EXPORT_SYMBOL_GPL(devm_kmalloc); >> >> /** >> + * static function to avoid code duplication when duplicating buffe= rs >> + */ >> +static void *devm_kmemdup_nochecks(struct device *dev, const void *= s, >> + size_t size, gfp_t gfp) >> +{ >> + void *buf; >> + >> + buf =3D devm_kmalloc(dev, size, gfp); >> + if (buf) >> + memcpy(buf, s, size); >> + return buf; >> +} >> + >> +/** >> * devm_kstrdup - Allocate resource managed space and >> * copy an existing string into that. >> * @dev: Device to allocate memory for >> @@ -802,21 +816,33 @@ EXPORT_SYMBOL_GPL(devm_kmalloc); >> */ >> char *devm_kstrdup(struct device *dev, const char *s, gfp_t gfp) >> { >> - size_t size; >> - char *buf; >> - >> if (!s) >> return NULL; >> >> - size =3D strlen(s) + 1; >> - buf =3D devm_kmalloc(dev, size, gfp); >> - if (buf) >> - memcpy(buf, s, size); >> - return buf; >> + return devm_kmemdup_nochecks(dev, s, (strlen(s)+1), gfp); >> } >> EXPORT_SYMBOL_GPL(devm_kstrdup); > hmm, this looks like an unrelated change to me and may warrant a sepa= rate > patch for the devm_kstrdup change. On reflection, I agree, I'll separate out the change to devm_kstrdup()=20 and resubmit. > > >> /** >> + * devm_kmemdup - Allocate resource managed space and >> + * copy existing data into that. >> + * @dev: Device to allocate memory for >> + * @s: the memory block to duplicate >> + * @gfp: the GFP mask used in the devm_kmalloc() call when >> + * allocating memory >> + * RETURNS: >> + * Pointer to allocated string on success, NULL on failure. >> + */ >> +void *devm_kmemdup(struct device *dev, const void *s, size_t size, = gfp_t >> gfp) +{ >> + if (!s || (size =3D=3D 0)) >> + return NULL; >> + >> + return devm_kmemdup_nochecks(dev, s, size, gfp); >> +} >> +EXPORT_SYMBOL_GPL(devm_kmemdup); >> + >> +/** >> * devm_kfree - Resource-managed kfree >> * @dev: Device this memory belongs to >> * @p: Memory to free >> diff --git a/include/linux/device.h b/include/linux/device.h >> index d1d1c05..7ace116 100644 >> --- a/include/linux/device.h >> +++ b/include/linux/device.h >> @@ -623,6 +623,8 @@ static inline void *devm_kcalloc(struct device *= dev, >> } >> extern void devm_kfree(struct device *dev, void *p); >> extern char *devm_kstrdup(struct device *dev, const char *s, gfp_t= gfp); >> +extern void *devm_kmemdup(struct device *dev, >> + const void *s, size_t size, gfp_t gfp); >> >> void __iomem *devm_ioremap_resource(struct device *dev, struct res= ource >> *res); void __iomem *devm_request_and_ioremap(struct device *dev, > > --=20 Rob Jones Project Manager Codethink Ltd mailto:rob.jones@codethink.co.uk tel:+44 161 236 5575