[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/2] i8254: Rework & fix interaction with HPET i
From: |
Jan Kiszka |
Subject: |
Re: [Qemu-devel] [PATCH 2/2] i8254: Rework & fix interaction with HPET in legacy mode |
Date: |
Sat, 10 Dec 2011 17:03:17 +0100 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
On 2011-12-10 16:54, Blue Swirl wrote:
> On Sat, Dec 10, 2011 at 15:51, Jan Kiszka <address@hidden> wrote:
>> On 2011-12-10 16:49, Blue Swirl wrote:
>>>>
>>>> +ISADevice *pit_init(int base, qemu_irq irq)
>>>
>>> Please retain this function in pc.h, or even better, introduce i8254.h.
>>
>> No concerns about i8254.h, but this function does not qualify for static
>> inline.
>
> The function is static inline in a header file not for performance
> reasons, but to keep the instantiation separate from device internals.
Not performance, footprint and header dependencies. You need to pull in
all the stuff the inline function needs for everyone including the
header that contains this function. That's messy.
Even if the instantiation helper should not poke into the device model
internals (and I don't want this to change as well), it belongs to the
module that implements the device. We do the same with other fabric
functions.
Jan
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH 0/2] pit/hpet: Fix legacy mode switching, Jan Kiszka, 2011/12/10
- [Qemu-devel] [PATCH 1/2] hpet: Save/restore cached RTC IRQ level, Jan Kiszka, 2011/12/10
- [Qemu-devel] [PATCH 2/2] i8254: Rework & fix interaction with HPET in legacy mode, Jan Kiszka, 2011/12/10
- Re: [Qemu-devel] [PATCH 2/2] i8254: Rework & fix interaction with HPET in legacy mode, Blue Swirl, 2011/12/10
- Re: [Qemu-devel] [PATCH 2/2] i8254: Rework & fix interaction with HPET in legacy mode, Jan Kiszka, 2011/12/10
- Re: [Qemu-devel] [PATCH 2/2] i8254: Rework & fix interaction with HPET in legacy mode, Blue Swirl, 2011/12/10
- Re: [Qemu-devel] [PATCH 2/2] i8254: Rework & fix interaction with HPET in legacy mode,
Jan Kiszka <=
- Re: [Qemu-devel] [PATCH 2/2] i8254: Rework & fix interaction with HPET in legacy mode, Blue Swirl, 2011/12/10
- Re: [Qemu-devel] [PATCH 2/2] i8254: Rework & fix interaction with HPET in legacy mode, Jan Kiszka, 2011/12/10
- Re: [Qemu-devel] [PATCH 2/2] i8254: Rework & fix interaction with HPET in legacy mode, Blue Swirl, 2011/12/10
Re: [Qemu-devel] [PATCH 0/2] pit/hpet: Fix legacy mode switching, Blue Swirl, 2011/12/10