* [PATCH] Remove unnecessary zero-initialisations from *io.c
@ 2013-01-01 14:58 Colin Watson
2013-01-03 19:23 ` Vladimir 'φ-coder/phcoder' Serbinenko
0 siblings, 1 reply; 3+ messages in thread
From: Colin Watson @ 2013-01-01 14:58 UTC (permalink / raw)
To: grub-devel
This patch removes explicit zero-initialisations from *io.c in favour of
grub_zalloc. Is there any good reason to initialise these explicitly?
This reduces the size of all four of these modules on i386-pc as
follows:
bufio -56
gzio -24
lzopio -20
xzio -64
2013-01-01 Colin Watson <cjwatson@ubuntu.com>
* grub-core/io/bufio.c (grub_bufio_open): Use grub_zalloc instead of
explicitly zeroing elements.
* grub-core/io/gzio.c (grub_gzio_open): Likewise.
* grub-core/io/lzopio.c (grub_lzopio_open): Remove explicit zeroing
of elements in a structure already allocated using grub_zalloc.
* grub-core/io/xzio.c (grub_xzio_open): Likewise.
=== modified file 'grub-core/io/bufio.c'
--- grub-core/io/bufio.c 2011-12-25 14:43:15 +0000
+++ grub-core/io/bufio.c 2013-01-01 14:38:07 +0000
@@ -48,7 +48,7 @@ grub_bufio_open (grub_file_t io, int siz
grub_file_t file;
grub_bufio_t bufio = 0;
- file = (grub_file_t) grub_malloc (sizeof (*file));
+ file = (grub_file_t) grub_zalloc (sizeof (*file));
if (! file)
return 0;
@@ -61,7 +61,7 @@ grub_bufio_open (grub_file_t io, int siz
size = ((io->size > GRUB_BUFIO_MAX_SIZE) ? GRUB_BUFIO_MAX_SIZE :
io->size);
- bufio = grub_malloc (sizeof (struct grub_bufio) + size);
+ bufio = grub_zalloc (sizeof (struct grub_bufio) + size);
if (! bufio)
{
grub_free (file);
@@ -70,14 +70,10 @@ grub_bufio_open (grub_file_t io, int siz
bufio->file = io;
bufio->block_size = size;
- bufio->buffer_len = 0;
- bufio->buffer_at = 0;
file->device = io->device;
- file->offset = 0;
file->size = io->size;
file->data = bufio;
- file->read_hook = 0;
file->fs = &grub_bufio_fs;
file->not_easily_seekable = io->not_easily_seekable;
=== modified file 'grub-core/io/gzio.c'
--- grub-core/io/gzio.c 2012-03-06 13:11:10 +0000
+++ grub-core/io/gzio.c 2013-01-01 14:38:03 +0000
@@ -1130,7 +1130,7 @@ grub_gzio_open (grub_file_t io)
grub_file_t file;
grub_gzio_t gzio = 0;
- file = (grub_file_t) grub_malloc (sizeof (*file));
+ file = (grub_file_t) grub_zalloc (sizeof (*file));
if (! file)
return 0;
@@ -1144,9 +1144,7 @@ grub_gzio_open (grub_file_t io)
gzio->file = io;
file->device = io->device;
- file->offset = 0;
file->data = gzio;
- file->read_hook = 0;
file->fs = &grub_gzio_fs;
file->not_easily_seekable = 1;
=== modified file 'grub-core/io/lzopio.c'
--- grub-core/io/lzopio.c 2012-04-07 17:54:51 +0000
+++ grub-core/io/lzopio.c 2013-01-01 14:38:26 +0000
@@ -428,9 +428,7 @@ grub_lzopio_open (grub_file_t io)
lzopio->file = io;
file->device = io->device;
- file->offset = 0;
file->data = lzopio;
- file->read_hook = 0;
file->fs = &grub_lzopio_fs;
file->size = GRUB_FILE_SIZE_UNKNOWN;
file->not_easily_seekable = 1;
=== modified file 'grub-core/io/xzio.c'
--- grub-core/io/xzio.c 2012-02-12 14:25:25 +0000
+++ grub-core/io/xzio.c 2013-01-01 14:39:38 +0000
@@ -186,12 +186,9 @@ grub_xzio_open (grub_file_t io)
}
xzio->file = io;
- xzio->saved_offset = 0;
file->device = io->device;
- file->offset = 0;
file->data = xzio;
- file->read_hook = 0;
file->fs = &grub_xzio_fs;
file->size = GRUB_FILE_SIZE_UNKNOWN;
file->not_easily_seekable = 1;
@@ -210,10 +207,7 @@ grub_xzio_open (grub_file_t io)
}
xzio->buf.in = xzio->inbuf;
- xzio->buf.in_pos = 0;
- xzio->buf.in_size = 0;
xzio->buf.out = xzio->outbuf;
- xzio->buf.out_pos = 0;
xzio->buf.out_size = XZBUFSIZ;
/* FIXME: don't test footer on not easily seekable files. */
Thanks,
--
Colin Watson [cjwatson@ubuntu.com]
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] Remove unnecessary zero-initialisations from *io.c
2013-01-01 14:58 [PATCH] Remove unnecessary zero-initialisations from *io.c Colin Watson
@ 2013-01-03 19:23 ` Vladimir 'φ-coder/phcoder' Serbinenko
2013-01-07 10:45 ` Colin Watson
0 siblings, 1 reply; 3+ messages in thread
From: Vladimir 'φ-coder/phcoder' Serbinenko @ 2013-01-03 19:23 UTC (permalink / raw)
To: grub-devel
[-- Attachment #1: Type: text/plain, Size: 4089 bytes --]
On 01.01.2013 15:58, Colin Watson wrote:
> This patch removes explicit zero-initialisations from *io.c in favour of
> grub_zalloc. Is there any good reason to initialise these explicitly?
> This reduces the size of all four of these modules on i386-pc as
> follows:
>
> bufio -56
> gzio -24
> lzopio -20
> xzio -64
Go ahead.
>
> 2013-01-01 Colin Watson <cjwatson@ubuntu.com>
>
> * grub-core/io/bufio.c (grub_bufio_open): Use grub_zalloc instead of
> explicitly zeroing elements.
> * grub-core/io/gzio.c (grub_gzio_open): Likewise.
> * grub-core/io/lzopio.c (grub_lzopio_open): Remove explicit zeroing
> of elements in a structure already allocated using grub_zalloc.
> * grub-core/io/xzio.c (grub_xzio_open): Likewise.
>
> === modified file 'grub-core/io/bufio.c'
> --- grub-core/io/bufio.c 2011-12-25 14:43:15 +0000
> +++ grub-core/io/bufio.c 2013-01-01 14:38:07 +0000
> @@ -48,7 +48,7 @@ grub_bufio_open (grub_file_t io, int siz
> grub_file_t file;
> grub_bufio_t bufio = 0;
>
> - file = (grub_file_t) grub_malloc (sizeof (*file));
> + file = (grub_file_t) grub_zalloc (sizeof (*file));
> if (! file)
> return 0;
>
> @@ -61,7 +61,7 @@ grub_bufio_open (grub_file_t io, int siz
> size = ((io->size > GRUB_BUFIO_MAX_SIZE) ? GRUB_BUFIO_MAX_SIZE :
> io->size);
>
> - bufio = grub_malloc (sizeof (struct grub_bufio) + size);
> + bufio = grub_zalloc (sizeof (struct grub_bufio) + size);
> if (! bufio)
> {
> grub_free (file);
> @@ -70,14 +70,10 @@ grub_bufio_open (grub_file_t io, int siz
>
> bufio->file = io;
> bufio->block_size = size;
> - bufio->buffer_len = 0;
> - bufio->buffer_at = 0;
>
> file->device = io->device;
> - file->offset = 0;
> file->size = io->size;
> file->data = bufio;
> - file->read_hook = 0;
> file->fs = &grub_bufio_fs;
> file->not_easily_seekable = io->not_easily_seekable;
>
>
> === modified file 'grub-core/io/gzio.c'
> --- grub-core/io/gzio.c 2012-03-06 13:11:10 +0000
> +++ grub-core/io/gzio.c 2013-01-01 14:38:03 +0000
> @@ -1130,7 +1130,7 @@ grub_gzio_open (grub_file_t io)
> grub_file_t file;
> grub_gzio_t gzio = 0;
>
> - file = (grub_file_t) grub_malloc (sizeof (*file));
> + file = (grub_file_t) grub_zalloc (sizeof (*file));
> if (! file)
> return 0;
>
> @@ -1144,9 +1144,7 @@ grub_gzio_open (grub_file_t io)
> gzio->file = io;
>
> file->device = io->device;
> - file->offset = 0;
> file->data = gzio;
> - file->read_hook = 0;
> file->fs = &grub_gzio_fs;
> file->not_easily_seekable = 1;
>
>
> === modified file 'grub-core/io/lzopio.c'
> --- grub-core/io/lzopio.c 2012-04-07 17:54:51 +0000
> +++ grub-core/io/lzopio.c 2013-01-01 14:38:26 +0000
> @@ -428,9 +428,7 @@ grub_lzopio_open (grub_file_t io)
> lzopio->file = io;
>
> file->device = io->device;
> - file->offset = 0;
> file->data = lzopio;
> - file->read_hook = 0;
> file->fs = &grub_lzopio_fs;
> file->size = GRUB_FILE_SIZE_UNKNOWN;
> file->not_easily_seekable = 1;
>
> === modified file 'grub-core/io/xzio.c'
> --- grub-core/io/xzio.c 2012-02-12 14:25:25 +0000
> +++ grub-core/io/xzio.c 2013-01-01 14:39:38 +0000
> @@ -186,12 +186,9 @@ grub_xzio_open (grub_file_t io)
> }
>
> xzio->file = io;
> - xzio->saved_offset = 0;
>
> file->device = io->device;
> - file->offset = 0;
> file->data = xzio;
> - file->read_hook = 0;
> file->fs = &grub_xzio_fs;
> file->size = GRUB_FILE_SIZE_UNKNOWN;
> file->not_easily_seekable = 1;
> @@ -210,10 +207,7 @@ grub_xzio_open (grub_file_t io)
> }
>
> xzio->buf.in = xzio->inbuf;
> - xzio->buf.in_pos = 0;
> - xzio->buf.in_size = 0;
> xzio->buf.out = xzio->outbuf;
> - xzio->buf.out_pos = 0;
> xzio->buf.out_size = XZBUFSIZ;
>
> /* FIXME: don't test footer on not easily seekable files. */
>
> Thanks,
>
--
Regards
Vladimir 'φ-coder/phcoder' Serbinenko
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 294 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] Remove unnecessary zero-initialisations from *io.c
2013-01-03 19:23 ` Vladimir 'φ-coder/phcoder' Serbinenko
@ 2013-01-07 10:45 ` Colin Watson
0 siblings, 0 replies; 3+ messages in thread
From: Colin Watson @ 2013-01-07 10:45 UTC (permalink / raw)
To: grub-devel
On Thu, Jan 03, 2013 at 08:23:19PM +0100, Vladimir 'φ-coder/phcoder' Serbinenko wrote:
> On 01.01.2013 15:58, Colin Watson wrote:
> > This patch removes explicit zero-initialisations from *io.c in favour of
> > grub_zalloc. Is there any good reason to initialise these explicitly?
> > This reduces the size of all four of these modules on i386-pc as
> > follows:
> >
> > bufio -56
> > gzio -24
> > lzopio -20
> > xzio -64
>
> Go ahead.
Committed, thanks.
--
Colin Watson [cjwatson@ubuntu.com]
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-01-07 10:46 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-01-01 14:58 [PATCH] Remove unnecessary zero-initialisations from *io.c Colin Watson
2013-01-03 19:23 ` Vladimir 'φ-coder/phcoder' Serbinenko
2013-01-07 10:45 ` Colin Watson
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.