[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC][PATCH 6/7] blkdebug: Add events and rules
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [RFC][PATCH 6/7] blkdebug: Add events and rules |
Date: |
Mon, 29 Mar 2010 10:00:46 +0200 |
User-agent: |
Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.8) Gecko/20100301 Fedora/3.0.3-1.fc12 Thunderbird/3.0.3 |
Am 28.03.2010 15:12, schrieb Christoph Hellwig:
> On Mon, Mar 15, 2010 at 06:08:34PM +0100, Kevin Wolf wrote:
>> + fprintf(stderr, "bdrv_debug_event: %d\n", event);
>
> Is this supposed to be in the final version or a leftover debugging aid?
It's not there in the final version (which I have already sent out, btw,
so you have reviewed an old version)
>> +#define BLKDBG_EVENT(bs, evt) bdrv_debug_event(bs, evt)
>
> Why not call bdrv_debug_event directly?
Originally I had intended to add a flag to ./configure to enable
blkdebug and #ifdef this out if it's not compiled in. Maybe we still
want to add that, but then it's not really much that you save.
>> + config = strdup(filename);
>> + config[c - filename] = '\0';
>> + ret = read_config(s, config);
>> + if (ret < 0) {
>> + return ret;
>> + }
>> + filename = c + 1;
>> +
>> + /* Open the backing file */
>> + ret = bdrv_file_open(&s->hd, filename, flags);
>> + if (ret < 0) {
>> + return ret;
>> + }
>> +
>> + return 0;
>
> Don't we need to free config somewhere?
Oops, this one is still there in the final version. I'll send another
version of the series.
Kevin
- [Qemu-devel] [RFC][PATCH 1/7] qemu-config: qemu_read_config_file() reads the normal config file, (continued)
- [Qemu-devel] [RFC][PATCH 1/7] qemu-config: qemu_read_config_file() reads the normal config file, Kevin Wolf, 2010/03/15
- [Qemu-devel] [RFC][PATCH 3/7] blkdebug: Basic request passthrough, Kevin Wolf, 2010/03/15
- [Qemu-devel] [RFC][PATCH 4/7] blkdebug: Inject errors, Kevin Wolf, 2010/03/15
- [Qemu-devel] [RFC][PATCH 5/7] Make qemu-config available for tools, Kevin Wolf, 2010/03/15
- [Qemu-devel] [RFC][PATCH 6/7] blkdebug: Add events and rules, Kevin Wolf, 2010/03/15
- [Qemu-devel] [RFC][PATCH 7/7] qcow2: Trigger blkdebug events, Kevin Wolf, 2010/03/15
- Re: [Qemu-devel] [RFC][PATCH 0/7] blkdebug, Blue Swirl, 2010/03/15