[Qemu-devel] [Patch v2 1/1] blizzard: Remove support for DEPTH != 32

2016-03-18 Thread Pooja Dhannawat
On Friday, March 18, 2016, Gerd Hoffmann <kra...@redhat.com> wrote: > > On Mo, 2016-03-14 at 15:22 +, Peter Maydell wrote: > > On 14 March 2016 at 15:04, Pooja Dhannawat <dhannawatpoo...@gmail.com> wrote: > > > Removing support for DEPTH != 32 from blizza

Re: [Qemu-devel] [PATCH v4] socket: Allocating Large sized arrays to heap

2016-03-18 Thread Pooja Dhannawat
On Friday, March 18, 2016, Paolo Bonzini <pbonz...@redhat.com> wrote: > > > On 18/03/2016 14:49, Pooja Dhannawat wrote: >> >> >> If it is greater than the accepted level, the on-stack >> buffer is not >> used

[Qemu-devel] [PATCH v2 1/1] socket: Allocating Large sized arrays to heap

2016-03-14 Thread Pooja Dhannawat
net_socket_send has a huge stack usage of 69712 bytes approx. Moving large arrays to heap to reduce stack usage. Signed-off-by: Pooja Dhannawat <dhannawatpoo...@gmail.com> --- net/socket.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/net/socket.c b/net/so

[Qemu-devel] [Patch v2 1/1] blizzard: Remove support for DEPTH != 32

2016-03-14 Thread Pooja Dhannawat
Removing support for DEPTH != 32 from blizzard template header and file that includes it, as macro DEPTH == 32 only used. Signed-off-by: Pooja Dhannawat <dhannawatpoo...@gmail.com> --- hw/display/blizzard.c | 24 hw/display/blizzard_template.

Re: [Qemu-devel] [Patch 1/1] net/net: Allocating Large sized arrays to heap

2016-03-14 Thread Pooja Dhannawat
On Mon, Mar 14, 2016 at 7:41 PM, Eric Blake <ebl...@redhat.com> wrote: > On 03/12/2016 01:39 AM, Pooja Dhannawat wrote: > > Signed-off-by: Pooja Dhannawat <dhannawatpoo...@gmail.com> > > --- > > net/net.c | 8 ++-- > > 1 file changed, 6 insertions(+),

[Qemu-devel] [Patch 1/1] net/socket: Allocating Large sized arrays to heap

2016-03-11 Thread Pooja Dhannawat
Signed-off-by: Pooja Dhannawat <dhannawatpoo...@gmail.com> --- net/socket.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/net/socket.c b/net/socket.c index e32e3cb..483dcac 100644 --- a/net/socket.c +++ b/net/socket.c @@ -147,10 +147,10 @@ static void net_socke

Re: [Qemu-devel] [PATCH v2 1/1] socket: Allocating Large sized arrays to heap

2016-03-14 Thread Pooja Dhannawat
On Mon, Mar 14, 2016 at 10:20 PM, Daniel P. Berrange <berra...@redhat.com> wrote: > On Mon, Mar 14, 2016 at 10:07:53PM +0530, Pooja Dhannawat wrote: > > net_socket_send has a huge stack usage of 69712 bytes approx. > > Moving large arrays to heap to reduce stack usag

[Qemu-devel] [PATCH v3] blizzard: Remove support for DEPTH != 32

2016-03-14 Thread Pooja Dhannawat
Removing support for DEPTH != 32 from blizzard template header and file that includes it, as macro DEPTH == 32 only used. Signed-off-by: Pooja Dhannawat <dhannawatpoo...@gmail.com> --- hw/display/blizzard.c | 24 hw/display/blizzard_template.

[Qemu-devel] [PATCH v3] blizzard: Remove support for DEPTH != 32

2016-03-14 Thread Pooja Dhannawat
Removing support for DEPTH != 32 from blizzard template header and file that includes it, as macro DEPTH == 32 only used. Signed-off-by: Pooja Dhannawat <dhannawatpoo...@gmail.com> --- hw/display/blizzard.c | 24 hw/display/blizzard_template.

[Qemu-devel] [PATCH v2] net: Allocating Large sized arrays to heap

2016-03-14 Thread Pooja Dhannawat
nc_sendv_compat has a huge stack usage of 69680 bytes approx. Moving large arrays to heap to reduce stack usage. Signed-off-by: Pooja Dhannawat <dhannawatpoo...@gmail.com> --- net/net.c | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/net/net.c b/net

[Qemu-devel] [PATCH v3] socket: Allocating Large sized arrays to heap

2016-03-14 Thread Pooja Dhannawat
net_socket_send has a huge stack usage of 69712 bytes approx. Moving large arrays to heap to reduce stack usage. Signed-off-by: Pooja Dhannawat <dhannawatpoo...@gmail.com> --- net/socket.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/net/socket.c b/net/so

[Qemu-devel] [PATCH v4] socket: Allocating Large sized arrays to heap

2016-03-15 Thread Pooja Dhannawat
net_socket_send has a huge stack usage of 69712 bytes approx. Moving large arrays to heap to reduce stack usage. Signed-off-by: Pooja Dhannawat <dhannawatpoo...@gmail.com> --- net/socket.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/net/socket.c b/net/so

[Qemu-devel] [Patch 1/1] omap_lcdc: Remove support for DEPTH != 32

2016-03-12 Thread Pooja Dhannawat
As only DEPTH ==32 case is used, removing other dead code which is based on DEPTH !== 32 Signed-off-by: Pooja Dhannawat <dhannawatpoo...@gmail.com> --- hw/display/omap_lcd_template.h | 10 ++ hw/display/omap_lcdc.c | 21 - 2 files changed, 2 insertions(

[Qemu-devel] [Patch 1/1] net/net: Allocating Large sized arrays to heap

2016-03-12 Thread Pooja Dhannawat
Signed-off-by: Pooja Dhannawat <dhannawatpoo...@gmail.com> --- net/net.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/net/net.c b/net/net.c index b0c832e..5399758 100644 --- a/net/net.c +++ b/net/net.c @@ -709,16 +709,18 @@ ssize_t qemu_send_pack

Re: [Qemu-devel] [PATCH v4] socket: Allocating Large sized arrays to heap

2016-03-19 Thread Pooja Dhannawat
On Thu, Mar 17, 2016 at 8:20 PM, Stefan Hajnoczi <stefa...@gmail.com> wrote: > On Tue, Mar 15, 2016 at 09:29:58PM +0530, Pooja Dhannawat wrote: > > @@ -170,8 +170,8 @@ static void net_socket_send(void *opaque) > > s->index = 0; > > s->packet_len

[Qemu-devel] [PATCH v3] net: Allocating Large sized arrays to heap

2016-03-15 Thread Pooja Dhannawat
nc_sendv_compat has a huge stack usage of 69680 bytes approx. Moving large arrays to heap to reduce stack usage. Signed-off-by: Pooja Dhannawat <dhannawatpoo...@gmail.com> --- net/net.c | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/net/net.c b/net

Re: [Qemu-devel] [PATCH v3] net: Allocating Large sized arrays to heap

2016-03-19 Thread Pooja Dhannawat
On Thu, Mar 17, 2016 at 8:23 PM, Stefan Hajnoczi <stefa...@gmail.com> wrote: > On Tue, Mar 15, 2016 at 11:55:13PM +0530, Pooja Dhannawat wrote: > > nc_sendv_compat has a huge stack usage of 69680 bytes approx. > > Moving large arrays to heap to reduce stack usage. > &g

Re: [Qemu-devel] [PATCH v4] socket: Allocating Large sized arrays to heap

2016-03-19 Thread Pooja Dhannawat
On Fri, Mar 18, 2016 at 4:20 AM, Paolo Bonzini <pbonz...@redhat.com> wrote: > > > On 17/03/2016 16:31, Pooja Dhannawat wrote: > > > > > > On Thu, Mar 17, 2016 at 8:20 PM, Stefan Hajnoczi <stefa...@gmail.com > > <mailto:stefa...@gmail.com>> wr

[Qemu-devel] [PATCH v4] net: Allocating Large sized arrays to heap

2016-03-19 Thread Pooja Dhannawat
nc_sendv_compat has a huge stack usage of 69680 bytes approx. Moving large arrays to heap to reduce stack usage. Signed-off-by: Pooja Dhannawat <dhannawatpoo...@gmail.com> --- net/net.c | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/net/net.c b/net/net.c

Re: [Qemu-devel] [PATCH v4] socket: Allocating Large sized arrays to heap

2016-03-19 Thread Pooja Dhannawat
;> wrote: >> >>> >>> >>> On 18/03/2016 12:24, Pooja Dhannawat wrote: >>> > >>> > >>> > On Fri, Mar 18, 2016 at 4:20 AM, Paolo Bonzini <pbonz...@redhat.com >>> > <mailto:pbonz...@redhat.com>> wrote: >>&

Re: [Qemu-devel] [PATCH v4] socket: Allocating Large sized arrays to heap

2016-03-19 Thread Pooja Dhannawat
On Fri, Mar 18, 2016 at 3:05 PM, Stefan Hajnoczi <stefa...@gmail.com> wrote: > On Thu, Mar 17, 2016 at 11:50:15PM +0100, Paolo Bonzini wrote: > > > > > > On 17/03/2016 16:31, Pooja Dhannawat wrote: > > > > > > > > > On Thu, Mar 17

Re: [Qemu-devel] [PATCH v4] blizzard: Remove support for DEPTH != 32

2016-03-25 Thread Pooja Dhannawat
On Fri, Mar 25, 2016 at 8:39 AM, Fam Zheng <f...@redhat.com> wrote: > On Thu, 03/24 23:48, Pooja Dhannawat wrote: > > Removing support for DEPTH != 32 from blizzard template header > > and file that includes it, as macro DEPTH == 32 only used. > > > > Signed-off-

[Qemu-devel] [PATCH v4] blizzard: Remove support for DEPTH != 32

2016-03-24 Thread Pooja Dhannawat
Removing support for DEPTH != 32 from blizzard template header and file that includes it, as macro DEPTH == 32 only used. Signed-off-by: Pooja Dhannawat <dhannawatpoo...@gmail.com> --- hw/display/blizzard.c | 41 - hw/display/blizzard_temp

[Qemu-devel] [PATCH v4] blizzard: Remove support for DEPTH != 32

2016-03-25 Thread Pooja Dhannawat
Removing support for DEPTH != 32 from blizzard template header and file that includes it, as macro DEPTH == 32 only used. Reviewed-by: Eric Blake <ebl...@redhat.com> Signed-off-by: Pooja Dhannawat <dhannawatpoo...@gmail.com> --- hw/display/blizzard.c

Re: [Qemu-devel] [Patch 1/1] omap_lcdc: Remove support for DEPTH != 32

2016-03-23 Thread Pooja Dhannawat
On Wed, Mar 23, 2016 at 3:55 AM, Eric Blake <ebl...@redhat.com> wrote: > On 03/12/2016 03:23 AM, Pooja Dhannawat wrote: > > As only DEPTH ==32 case is used, removing other dead code > > which is based on DEPTH !== 32 > > > > Signed-off-by: Pooja Dha

Re: [Qemu-devel] [PATCH v2] omap_lcdc: Remove support for DEPTH != 32

2016-04-05 Thread Pooja Dhannawat
CCing Peter Maydell. On Tue, Mar 29, 2016 at 7:32 PM, Pooja Dhannawat <dhannawatpoo...@gmail.com> wrote: > surface_bits_per_pixel() always returns 32 > so, removing other dead code which is > based on DEPTH !== 32 > > Signed-off-by: Pooja Dhannawat <dhannawatpoo.

Re: [Qemu-devel] [PATCH v4] blizzard: Remove support for DEPTH != 32

2016-04-05 Thread Pooja Dhannawat
CCing Eric Blake and Gerd Hoffmann. On Sat, Mar 26, 2016 at 11:27 AM, Pooja Dhannawat <dhannawatpoo...@gmail.com > wrote: > Removing support for DEPTH != 32 from blizzard template header > and file that includes it, as macro DEPTH == 32 only used. > > Reviewed-by: Eric Blake

[Qemu-devel] [PATCH v5] net: Allocating Large sized arrays to heap

2016-03-28 Thread Pooja Dhannawat
nc_sendv_compat has a huge stack usage of 69680 bytes approx. Moving large arrays to heap to reduce stack usage. Reviewed-by: Stefan Hajnoczi <stefa...@redhat.com> Signed-off-by: Pooja Dhannawat <dhannawatpoo...@gmail.com> --- net/net.c | 13 + 1 file changed, 9 inse

Re: [Qemu-devel] [Patch 1/1] omap_lcdc: Remove support for DEPTH != 32

2016-03-28 Thread Pooja Dhannawat
On Wed, Mar 23, 2016 at 11:35 AM, Pooja Dhannawat <dhannawatpoo...@gmail.com > wrote: > > > On Wed, Mar 23, 2016 at 3:55 AM, Eric Blake <ebl...@redhat.com> wrote: > >> On 03/12/2016 03:23 AM, Pooja Dhannawat wrote: >> > As only DEPTH ==32 case is used, remov

[Qemu-devel] [PATCH v2] omap_lcdc: Remove support for DEPTH != 32

2016-03-29 Thread Pooja Dhannawat
surface_bits_per_pixel() always returns 32 so, removing other dead code which is based on DEPTH !== 32 Signed-off-by: Pooja Dhannawat <dhannawatpoo...@gmail.com> --- v2: As surface_bits_per_pixel() always returns 32, so instead of relying on array : draw_line_

Re: [Qemu-devel] [PATCH v5] net: Allocating Large sized arrays to heap

2016-03-31 Thread Pooja Dhannawat
On Thu, Mar 31, 2016 at 3:28 PM, Stefan Hajnoczi <stefa...@gmail.com> wrote: > On Mon, Mar 28, 2016 at 06:04:11PM +0530, Pooja Dhannawat wrote: > > nc_sendv_compat has a huge stack usage of 69680 bytes approx. > > Moving large arrays to heap to reduce stack usage. > &