From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1MsxVR-0001yr-W5
	for qemu-devel@nongnu.org; Wed, 30 Sep 2009 07:38:02 -0400
Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43)
	id 1MsxVN-0001x7-UL
	for qemu-devel@nongnu.org; Wed, 30 Sep 2009 07:38:01 -0400
Received: from [199.232.76.173] (port=42454 helo=monty-python.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1MsxVN-0001wf-BE
	for qemu-devel@nongnu.org; Wed, 30 Sep 2009 07:37:57 -0400
Received: from mx1.redhat.com ([209.132.183.28]:30805)
	by monty-python.gnu.org with esmtp (Exim 4.60)
	(envelope-from <quintela@redhat.com>) id 1MsxVM-0008HJ-Rx
	for qemu-devel@nongnu.org; Wed, 30 Sep 2009 07:37:57 -0400
From: Juan Quintela <quintela@redhat.com>
In-Reply-To: <Pine.LNX.4.64.0909301435150.2147@linmac.oyster.ru> (malc's
	message of "Wed, 30 Sep 2009 14:37:16 +0400 (MSD)")
References: <cover.1254255997.git.quintela@redhat.com>
	<d93032e20d443aeb567c2dcee9c6d4d166b98d58.1254255997.git.quintela@redhat.com>
	<Pine.LNX.4.64.0909301435150.2147@linmac.oyster.ru>
Date: Wed, 30 Sep 2009 13:37:34 +0200
Message-ID: <m3zl8cwtmp.fsf@neno.mitica>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Subject: [Qemu-devel] Re: [PATCH 27/49] ac97: add active to the state
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/pipermail/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: malc <av1474@comtv.ru>
Cc: qemu-devel@nongnu.org

malc <av1474@comtv.ru> wrote:
> On Tue, 29 Sep 2009, Juan Quintela wrote:
>
>> This simplifies reset_voices, that only takes one argument now.
>> 
>> Signed-off-by: Juan Quintela <quintela@redhat.com>
>> ---
>>  hw/ac97.c |   42 ++++++++++++++++++++----------------------
>>  1 files changed, 20 insertions(+), 22 deletions(-)
>> 
>> diff --git a/hw/ac97.c b/hw/ac97.c
>> index 610ca60..da6cb2d 100644
>> --- a/hw/ac97.c
>> +++ b/hw/ac97.c
>> @@ -146,6 +146,13 @@ typedef struct AC97BusMasterRegs {
>>      BD bd;
>>  } AC97BusMasterRegs;
>> 
>> +enum {
>> +    PI_INDEX = 0,
>> +    PO_INDEX,
>> +    MC_INDEX,
>> +    LAST_INDEX
>> +};
>
> And this was moved becasue...?

Because I need LAST_INDEX in next part that you quoted.

>> +
>>  typedef struct AC97LinkState {
>>      PCIDevice dev;
>>      QEMUSoundCard card;
>> @@ -162,6 +169,7 @@ typedef struct AC97LinkState {
>>      uint8_t silence[128];
>>      uint32_t base[2];
>>      int bup_flag;
>> +    uint8_t active[LAST_INDEX];
>
> This doesn't belong here, cause the only purpose i can see is to hack
> around defficiencies of the new load/savevm APIs.

That was supposed to be one of the features, not deficiences.  You can't
sent stuff that it is not in the state. It is "by design" that you can't
sent arbitrary variables.

Later, Juan.