[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 33/41] qemu-file: simplify and export qemu_ftell
From: |
Orit Wasserman |
Subject: |
Re: [Qemu-devel] [PATCH 33/41] qemu-file: simplify and export qemu_ftell |
Date: |
Thu, 21 Feb 2013 10:14:30 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130110 Thunderbird/17.0.2 |
On 02/15/2013 07:47 PM, Paolo Bonzini wrote:
> Force a flush when qemu_ftell is called. This simplifies the buffer magic
> (it also breaks qemu_ftell for input QEMUFiles, but we never use it).
>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> savevm.c | 20 ++++++++------------
> 1 files changed, 8 insertions(+), 12 deletions(-)
>
> diff --git a/savevm.c b/savevm.c
> index b6cf415..e4dc01d 100644
> --- a/savevm.c
> +++ b/savevm.c
> @@ -119,8 +119,8 @@ struct QEMUFile {
> void *opaque;
> int is_write;
>
> - int64_t buf_offset; /* start of buffer when writing, end of buffer
> - when reading */
> + int64_t pos; /* start of buffer when writing, end of buffer
> + when reading */
> int buf_index;
> int buf_size; /* 0 when writing */
> uint8_t buf[IO_BUF_SIZE];
> @@ -505,9 +505,9 @@ static void qemu_fflush(QEMUFile *f)
> return;
> }
> if (f->is_write && f->buf_index > 0) {
> - ret = f->ops->put_buffer(f->opaque, f->buf, f->buf_offset,
> f->buf_index);
> + ret = f->ops->put_buffer(f->opaque, f->buf, f->pos, f->buf_index);
> if (ret >= 0) {
> - f->buf_offset += f->buf_index;
> + f->pos += f->buf_index;
> }
> f->buf_index = 0;
> }
> @@ -534,11 +534,11 @@ static void qemu_fill_buffer(QEMUFile *f)
> f->buf_index = 0;
> f->buf_size = pending;
>
> - len = f->ops->get_buffer(f->opaque, f->buf + pending, f->buf_offset,
> + len = f->ops->get_buffer(f->opaque, f->buf + pending, f->pos,
> IO_BUF_SIZE - pending);
> if (len > 0) {
> f->buf_size += len;
> - f->buf_offset += len;
> + f->pos += len;
> } else if (len == 0) {
> qemu_file_set_error(f, -EIO);
> } else if (len != -EAGAIN)
> @@ -718,12 +718,8 @@ int qemu_get_byte(QEMUFile *f)
>
> int64_t qemu_ftell(QEMUFile *f)
> {
> - /* buf_offset excludes buffer for writing but includes it for reading */
> - if (f->is_write) {
> - return f->buf_offset + f->buf_index;
> - } else {
> - return f->buf_offset - f->buf_size + f->buf_index;
> - }
> + qemu_fflush(f);
> + return f->pos;
> }
>
> int qemu_file_rate_limit(QEMUFile *f)
>
Reviewed-by: Orit Wasserman <address@hidden>
- Re: [Qemu-devel] [PATCH 13/41] migration: prepare to access s->state outside critical sections, (continued)
- [Qemu-devel] [PATCH 16/41] block-migration: small preparatory changes for locking, Paolo Bonzini, 2013/02/15
- [Qemu-devel] [PATCH 19/41] migration: reorder SaveVMHandlers members, Paolo Bonzini, 2013/02/15
- [Qemu-devel] [PATCH 37/41] migration: small changes around rate-limiting, Paolo Bonzini, 2013/02/15
- [Qemu-devel] [PATCH 33/41] qemu-file: simplify and export qemu_ftell, Paolo Bonzini, 2013/02/15
- [Qemu-devel] [PATCH 15/41] block-migration: remove variables that are never read, Paolo Bonzini, 2013/02/15
- [Qemu-devel] [PATCH 06/41] qemu-file: pass errno from qemu_fflush via f->last_error, Paolo Bonzini, 2013/02/15
- [Qemu-devel] [PATCH 32/41] qemu-file: add writable socket QEMUFile, Paolo Bonzini, 2013/02/15