Re: [PATCH 04/16] qemu-file: Don't call qemu

qemu-devel

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 04/16] qemu-file: Don't call qemu_fflush() for read only file

From:	Richard Henderson
Subject:	Re: [PATCH 04/16] qemu-file: Don't call qemu_fflush() for read only files
Date:	Tue, 30 May 2023 10:01:43 -0700
User-agent:	Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.11.0

On 5/30/23 05:28, Juan Quintela wrote:

This was the only caller for read only files.  So change the test for
an assert in qemu_fflush().



Not a fan, as fflush(stdin) is well-defined.


r~

[Prev in Thread]

Current Thread

[Next in Thread]

[PATCH 14/16] migration: Use migration_transferred_bytes(), (continued)
- [PATCH 14/16] migration: Use migration_transferred_bytes(), Juan Quintela, 2023/05/30
- [PATCH 16/16] qemu-file: Make qemu_fflush() return errors, Juan Quintela, 2023/05/30
  - Re: [PATCH 16/16] qemu-file: Make qemu_fflush() return errors, Philippe Mathieu-Daudé, 2023/05/30
- [PATCH 11/16] migration: migration_transferred_bytes() don't need the QEMUFile, Juan Quintela, 2023/05/30
  - Re: [PATCH 11/16] migration: migration_transferred_bytes() don't need the QEMUFile, Philippe Mathieu-Daudé, 2023/05/30
- [PATCH 12/16] migration: migration_rate_limit_reset() don't need the QEMUFile, Juan Quintela, 2023/05/30
  - Re: [PATCH 12/16] migration: migration_rate_limit_reset() don't need the QEMUFile, Philippe Mathieu-Daudé, 2023/05/30
- [PATCH 15/16] migration: Remove transferred atomic counter, Juan Quintela, 2023/05/30
- [PATCH 04/16] qemu-file: Don't call qemu_fflush() for read only files, Juan Quintela, 2023/05/30
  - Re: [PATCH 04/16] qemu-file: Don't call qemu_fflush() for read only files, Fabiano Rosas, 2023/05/30
  - Re: [PATCH 04/16] qemu-file: Don't call qemu_fflush() for read only files, Richard Henderson <=
    - Re: [PATCH 04/16] qemu-file: Don't call qemu_fflush() for read only files, Juan Quintela, 2023/05/30
    - Re: [PATCH 04/16] qemu-file: Don't call qemu_fflush() for read only files, Richard Henderson, 2023/05/30
    - Re: [PATCH 04/16] qemu-file: Don't call qemu_fflush() for read only files, Juan Quintela, 2023/05/30
  - Re: [PATCH 04/16] qemu-file: Don't call qemu_fflush() for read only files, Juan Quintela, 2023/05/30
- [PATCH 05/16] qemu-file: We only call qemu_file_transferred_* on the sending side, Juan Quintela, 2023/05/30
- [PATCH 09/16] qemu_file: Remove unused qemu_file_transferred(), Juan Quintela, 2023/05/30
- [PATCH 01/16] qemu-file: Rename qemu_file_transferred_ fast -> noflush, Juan Quintela, 2023/05/30
  - Re: [PATCH 01/16] qemu-file: Rename qemu_file_transferred_ fast -> noflush, Philippe Mathieu-Daudé, 2023/05/30
    - Re: [PATCH 01/16] qemu-file: Rename qemu_file_transferred_ fast -> noflush, Philippe Mathieu-Daudé, 2023/05/30
- [PATCH 10/16] qemu-file: Remove _noflush from qemu_file_transferred_noflush(), Juan Quintela, 2023/05/30

Prev by Date: Re: [RFC PATCH] target/arm: use x86 intrinsics to implement AES instructions
Next by Date: Re: [RFC PATCH] target/arm: use x86 intrinsics to implement AES instructions
Previous by thread: Re: [PATCH 04/16] qemu-file: Don't call qemu_fflush() for read only files
Next by thread: Re: [PATCH 04/16] qemu-file: Don't call qemu_fflush() for read only files
Index(es):
- Date
- Thread