[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1 01/22] s390: use FILE instead of QEMUFile for
From: |
Cornelia Huck |
Subject: |
Re: [Qemu-devel] [PATCH v1 01/22] s390: use FILE instead of QEMUFile for creating text file |
Date: |
Tue, 12 Jan 2016 12:58:10 +0100 |
On Tue, 12 Jan 2016 11:43:55 +0000
"Daniel P. Berrange" <address@hidden> wrote:
> The s390 skeys monitor command needs to write out a plain text
> file. Currently it is using the QEMUFile class for this. There
> is no real benefit to this, and the downside is that it needs to
> snprintf via an intermediate buffer. Switching to regular FILE
> objects simplifies the code.
>
> Signed-off-by: Daniel P. Berrange <address@hidden>
> ---
> hw/s390x/s390-skeys.c | 19 +++++++------------
> 1 file changed, 7 insertions(+), 12 deletions(-)
> @@ -124,7 +119,7 @@ void qmp_dump_skeys(const char *filename, Error **errp)
> return;
> }
>
> - f = qemu_fopen(filename, "wb");
> + f = fopen(filename, "wb");
> if (!f) {
> error_setg_file_open(errp, errno, filename);
> return;
Hmm...
https://marc.info/?l=qemu-devel&m=143740278908660&w=2
We'd lose the benefits from qemu_fopen() here again, or am I missing
something?
- [Qemu-devel] [PATCH v1 00/22] Convert migration to QIOChannel & support TLS, Daniel P. Berrange, 2016/01/12
- [Qemu-devel] [PATCH v1 02/22] migration: remove use of qemu_bufopen from vmstate tests, Daniel P. Berrange, 2016/01/12
- [Qemu-devel] [PATCH v1 03/22] migration: ensure qemu_fflush() always writes full data amount, Daniel P. Berrange, 2016/01/12
- [Qemu-devel] [PATCH v1 04/22] migration: split migration hooks out of QEMUFileOps, Daniel P. Berrange, 2016/01/12
- [Qemu-devel] [PATCH v1 05/22] migration: introduce set_blocking function in QEMUFileOps, Daniel P. Berrange, 2016/01/12