[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/2] Keep track of ISA ports ISA device is using
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH 1/2] Keep track of ISA ports ISA device is using in qdev. |
Date: |
Mon, 25 Oct 2010 20:01:13 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux) |
Gleb Natapov <address@hidden> writes:
> On Mon, Oct 25, 2010 at 05:06:51PM +0200, Markus Armbruster wrote:
>> Gleb Natapov <address@hidden> writes:
>>
>> > Prevent two devices from claiming the same io port.
>>
>> Really?
>>
>> Your new check for double-claim is in the new isa_init_ioport(), which
>> is for ISADevice only. Thus, only qdevified ISA devices can opt for
>> this protection, by calling isa_init_ioport(). It doesn't protect from
>> devices who don't or can't opt in, such as PCI devices.
>>
> I didn't claim different. This obviously works only for ISA qdev
> devices.
I read "Prevent two devices from claiming the same io port" as such.
>> Anyway, we already check for double-claim in
>> register_ioport_{read,write}(). The check has issues --- hw_error() is
>> wrong there for hot plug. But it's where the check should be, isn't it?
> You don't like double-claim checking? Forget about it (all 3 lines
> of code). The real point of the patch is to have ISA resources used
> by devices to be stored in common place (ISADevice) which allows
> get_dev_path() to be implemented.
We already track I/O port use, in ioport.c. I don't like that
duplicated. Even less so if the dupe catches fewer double-claims than
the original.
Perhaps your new function should wrap around register_ioport_*() instead
of supplementing it.
Re: [Qemu-devel] [PATCH 0/2] Add get_dev_path callback to ISA bus., Markus Armbruster, 2010/10/25
- Re: [Qemu-devel] [PATCH 0/2] Add get_dev_path callback to ISA bus., Gleb Natapov, 2010/10/25
- Re: [Qemu-devel] [PATCH 0/2] Add get_dev_path callback to ISA bus., Markus Armbruster, 2010/10/25
- Re: [Qemu-devel] [PATCH 0/2] Add get_dev_path callback to ISA bus., Gleb Natapov, 2010/10/25
- Re: [Qemu-devel] [PATCH 0/2] Add get_dev_path callback to ISA bus., Markus Armbruster, 2010/10/26
- Re: [Qemu-devel] [PATCH 0/2] Add get_dev_path callback to ISA bus., Gleb Natapov, 2010/10/26
- Re: [Qemu-devel] [PATCH 0/2] Add get_dev_path callback to ISA bus., Markus Armbruster, 2010/10/26