* [Qemu-devel] [PATCH v1] Changed malloc to g_malloc, free to g_free in bsd-user/qemu.h @ 2016-04-05 13:09 Md Haris Iqbal 2016-05-03 9:45 ` haris iqbal 0 siblings, 1 reply; 6+ messages in thread From: Md Haris Iqbal @ 2016-04-05 13:09 UTC (permalink / raw) To: qemu-devel; +Cc: qemu-trivial, pbonzini, Md Haris Iqbal, armbru, peter.maydell Signed-off-by: Md Haris Iqbal <haris.phnx@gmail.com> --- bsd-user/qemu.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/bsd-user/qemu.h b/bsd-user/qemu.h index 03b502a..5fac42e 100644 --- a/bsd-user/qemu.h +++ b/bsd-user/qemu.h @@ -357,7 +357,7 @@ static inline void *lock_user(int type, abi_ulong guest_addr, long len, int copy #ifdef DEBUG_REMAP { void *addr; - addr = malloc(len); + addr = g_malloc(len); if (copy) memcpy(addr, g2h(guest_addr), len); else @@ -383,7 +383,7 @@ static inline void unlock_user(void *host_ptr, abi_ulong guest_addr, return; if (len > 0) memcpy(g2h(guest_addr), host_ptr, len); - free(host_ptr); + g_free(host_ptr); #endif } -- 1.9.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH v1] Changed malloc to g_malloc, free to g_free in bsd-user/qemu.h 2016-04-05 13:09 [Qemu-devel] [PATCH v1] Changed malloc to g_malloc, free to g_free in bsd-user/qemu.h Md Haris Iqbal @ 2016-05-03 9:45 ` haris iqbal 2016-05-03 17:32 ` Sean Bruno 0 siblings, 1 reply; 6+ messages in thread From: haris iqbal @ 2016-05-03 9:45 UTC (permalink / raw) To: QEMU Developers Cc: QEMU Trivial, Paolo Bonzini, Peter Maydell, Markus Armbruster, Md Haris Iqbal Just a reminder. waiting for a review for the previous patch. On Tue, Apr 5, 2016 at 6:39 PM, Md Haris Iqbal <haris.phnx@gmail.com> wrote: > Signed-off-by: Md Haris Iqbal <haris.phnx@gmail.com> > --- > bsd-user/qemu.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/bsd-user/qemu.h b/bsd-user/qemu.h > index 03b502a..5fac42e 100644 > --- a/bsd-user/qemu.h > +++ b/bsd-user/qemu.h > @@ -357,7 +357,7 @@ static inline void *lock_user(int type, abi_ulong guest_addr, long len, int copy > #ifdef DEBUG_REMAP > { > void *addr; > - addr = malloc(len); > + addr = g_malloc(len); > if (copy) > memcpy(addr, g2h(guest_addr), len); > else > @@ -383,7 +383,7 @@ static inline void unlock_user(void *host_ptr, abi_ulong guest_addr, > return; > if (len > 0) > memcpy(g2h(guest_addr), host_ptr, len); > - free(host_ptr); > + g_free(host_ptr); > #endif > } > > -- > 1.9.1 > -- With regards, Md Haris Iqbal, Placement Coordinator, MTech IT NITK Surathkal, Contact: +91 8861996962 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH v1] Changed malloc to g_malloc, free to g_free in bsd-user/qemu.h 2016-05-03 9:45 ` haris iqbal @ 2016-05-03 17:32 ` Sean Bruno 2016-05-03 18:10 ` haris iqbal 0 siblings, 1 reply; 6+ messages in thread From: Sean Bruno @ 2016-05-03 17:32 UTC (permalink / raw) To: haris iqbal, QEMU Developers Cc: QEMU Trivial, Paolo Bonzini, Markus Armbruster, Peter Maydell -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA512 On 05/03/16 02:45, haris iqbal wrote: > Just a reminder. waiting for a review for the previous patch. > > On Tue, Apr 5, 2016 at 6:39 PM, Md Haris Iqbal > <haris.phnx@gmail.com> wrote: >> Signed-off-by: Md Haris Iqbal <haris.phnx@gmail.com> --- >> bsd-user/qemu.h | 4 ++-- 1 file changed, 2 insertions(+), 2 >> deletions(-) >> >> diff --git a/bsd-user/qemu.h b/bsd-user/qemu.h index >> 03b502a..5fac42e 100644 --- a/bsd-user/qemu.h +++ >> b/bsd-user/qemu.h @@ -357,7 +357,7 @@ static inline void >> *lock_user(int type, abi_ulong guest_addr, long len, int copy >> #ifdef DEBUG_REMAP { void *addr; - addr = malloc(len); + >> addr = g_malloc(len); if (copy) memcpy(addr, g2h(guest_addr), >> len); else @@ -383,7 +383,7 @@ static inline void >> unlock_user(void *host_ptr, abi_ulong guest_addr, return; if (len >> > 0) memcpy(g2h(guest_addr), host_ptr, len); - >> free(host_ptr); + g_free(host_ptr); #endif } >> >> -- 1.9.1 >> > > > If this is the only patch needing review for the bsd-user code, please put me down as Reviewed by. Reviewed-by: Sean Bruno <sbruno@freebsd.org> -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQF8BAEBCgBmBQJXKOCTXxSAAAAAAC4AKGlzc3Vlci1mcHJAbm90YXRpb25zLm9w ZW5wZ3AuZmlmdGhob3JzZW1hbi5uZXRCQUFENDYzMkU3MTIxREU4RDIwOTk3REQx MjAxRUZDQTFFNzI3RTY0AAoJEBIB78oecn5kZE0H/06zauEdcv8Y0K3KI3r2ioFS M6nqr/UXjqazqIeArBKDC+cnv47cnwH/x6Udn2on+CY9kR91QyUeG/DKUJdEp6Hs idUuTf3L0W8nhNUUXE9bwVI7H2KWpDzHJ52dnHuSXaQZnBE6Q1AFk0ge+hg+XcCU BsOqEkTAcxchOnvQJgNpMyXofI35QvSXqqaijTozMdGT7iHG0nKpO6KNW11ash2U YEVNs9nkr9I+B+qKPhhgX+v4dAuHSekaEstWbkqqSa4YsmqIylL1MNTpRTbEUSEG fScXKCOy3lANGV6u1sTzLIffR6N3WhkaYCuzYsQ7TQL/TAuYpBrQq7TQt8lb0Do= =qmUg -----END PGP SIGNATURE----- ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH v1] Changed malloc to g_malloc, free to g_free in bsd-user/qemu.h 2016-05-03 17:32 ` Sean Bruno @ 2016-05-03 18:10 ` haris iqbal 2016-05-03 19:02 ` Peter Maydell 0 siblings, 1 reply; 6+ messages in thread From: haris iqbal @ 2016-05-03 18:10 UTC (permalink / raw) To: Sean Bruno Cc: QEMU Developers, QEMU Trivial, Paolo Bonzini, Markus Armbruster, Peter Maydell On Tue, May 3, 2016 at 11:02 PM, Sean Bruno <sbruno@freebsd.org> wrote: > -----BEGIN PGP SIGNED MESSAGE----- > Hash: SHA512 > > > > On 05/03/16 02:45, haris iqbal wrote: >> Just a reminder. waiting for a review for the previous patch. >> >> On Tue, Apr 5, 2016 at 6:39 PM, Md Haris Iqbal >> <haris.phnx@gmail.com> wrote: >>> Signed-off-by: Md Haris Iqbal <haris.phnx@gmail.com> --- >>> bsd-user/qemu.h | 4 ++-- 1 file changed, 2 insertions(+), 2 >>> deletions(-) >>> >>> diff --git a/bsd-user/qemu.h b/bsd-user/qemu.h index >>> 03b502a..5fac42e 100644 --- a/bsd-user/qemu.h +++ >>> b/bsd-user/qemu.h @@ -357,7 +357,7 @@ static inline void >>> *lock_user(int type, abi_ulong guest_addr, long len, int copy >>> #ifdef DEBUG_REMAP { void *addr; - addr = malloc(len); + >>> addr = g_malloc(len); if (copy) memcpy(addr, g2h(guest_addr), >>> len); else @@ -383,7 +383,7 @@ static inline void >>> unlock_user(void *host_ptr, abi_ulong guest_addr, return; if (len >>> > 0) memcpy(g2h(guest_addr), host_ptr, len); - >>> free(host_ptr); + g_free(host_ptr); #endif } >>> >>> -- 1.9.1 >>> >> >> >> > > If this is the only patch needing review for the bsd-user code, please > put me down as Reviewed by. Sorry, I am a newbie. Just to confirm, should I send the patch again with "Reviewed by : Sean Bruno" line? > > Reviewed-by: Sean Bruno <sbruno@freebsd.org> > -----BEGIN PGP SIGNATURE----- > Version: GnuPG v2 > > iQF8BAEBCgBmBQJXKOCTXxSAAAAAAC4AKGlzc3Vlci1mcHJAbm90YXRpb25zLm9w > ZW5wZ3AuZmlmdGhob3JzZW1hbi5uZXRCQUFENDYzMkU3MTIxREU4RDIwOTk3REQx > MjAxRUZDQTFFNzI3RTY0AAoJEBIB78oecn5kZE0H/06zauEdcv8Y0K3KI3r2ioFS > M6nqr/UXjqazqIeArBKDC+cnv47cnwH/x6Udn2on+CY9kR91QyUeG/DKUJdEp6Hs > idUuTf3L0W8nhNUUXE9bwVI7H2KWpDzHJ52dnHuSXaQZnBE6Q1AFk0ge+hg+XcCU > BsOqEkTAcxchOnvQJgNpMyXofI35QvSXqqaijTozMdGT7iHG0nKpO6KNW11ash2U > YEVNs9nkr9I+B+qKPhhgX+v4dAuHSekaEstWbkqqSa4YsmqIylL1MNTpRTbEUSEG > fScXKCOy3lANGV6u1sTzLIffR6N3WhkaYCuzYsQ7TQL/TAuYpBrQq7TQt8lb0Do= > =qmUg > -----END PGP SIGNATURE----- -- With regards, Md Haris Iqbal, Placement Coordinator, MTech IT NITK Surathkal, Contact: +91 8861996962 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH v1] Changed malloc to g_malloc, free to g_free in bsd-user/qemu.h 2016-05-03 18:10 ` haris iqbal @ 2016-05-03 19:02 ` Peter Maydell 2016-05-03 19:08 ` haris iqbal 0 siblings, 1 reply; 6+ messages in thread From: Peter Maydell @ 2016-05-03 19:02 UTC (permalink / raw) To: haris iqbal Cc: Sean Bruno, QEMU Developers, QEMU Trivial, Paolo Bonzini, Markus Armbruster On 3 May 2016 at 19:10, haris iqbal <haris.phnx@gmail.com> wrote: > On Tue, May 3, 2016 at 11:02 PM, Sean Bruno <sbruno@freebsd.org> wrote: >> If this is the only patch needing review for the bsd-user code, please >> put me down as Reviewed by. > > Sorry, I am a newbie. Just to confirm, should I send the patch again > with "Reviewed by : Sean Bruno" line? No, you don't need to resend. When bsd-user patches are collected up to be applied then the person who does that will apply the reviewed-by tag. (If you needed to resend a patch for some other reason, then if the patch hasn't changed then you should add the reviewed-by tag when you resend it. But there's no need to resend purely to add the tag.) thanks -- PMM ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH v1] Changed malloc to g_malloc, free to g_free in bsd-user/qemu.h 2016-05-03 19:02 ` Peter Maydell @ 2016-05-03 19:08 ` haris iqbal 0 siblings, 0 replies; 6+ messages in thread From: haris iqbal @ 2016-05-03 19:08 UTC (permalink / raw) To: Peter Maydell Cc: Sean Bruno, QEMU Developers, QEMU Trivial, Paolo Bonzini, Markus Armbruster On Wed, May 4, 2016 at 12:32 AM, Peter Maydell <peter.maydell@linaro.org> wrote: > On 3 May 2016 at 19:10, haris iqbal <haris.phnx@gmail.com> wrote: >> On Tue, May 3, 2016 at 11:02 PM, Sean Bruno <sbruno@freebsd.org> wrote: >>> If this is the only patch needing review for the bsd-user code, please >>> put me down as Reviewed by. >> >> Sorry, I am a newbie. Just to confirm, should I send the patch again >> with "Reviewed by : Sean Bruno" line? > > No, you don't need to resend. When bsd-user patches are collected > up to be applied then the person who does that will apply the > reviewed-by tag. > > (If you needed to resend a patch for some other reason, then if > the patch hasn't changed then you should add the reviewed-by > tag when you resend it. But there's no need to resend purely to > add the tag.) Thanks for the reply, much appreciated. > > thanks > -- PMM -- With regards, Md Haris Iqbal, Placement Coordinator, MTech IT NITK Surathkal, Contact: +91 8861996962 ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-05-03 19:09 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-04-05 13:09 [Qemu-devel] [PATCH v1] Changed malloc to g_malloc, free to g_free in bsd-user/qemu.h Md Haris Iqbal 2016-05-03 9:45 ` haris iqbal 2016-05-03 17:32 ` Sean Bruno 2016-05-03 18:10 ` haris iqbal 2016-05-03 19:02 ` Peter Maydell 2016-05-03 19:08 ` haris iqbal
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).