diff mbox

[v11] megasas: LSI Megaraid SAS HBA emulation

Message ID alpine.LFD.2.02.1202181213260.26683@bbs.intern
State New
Headers show

Commit Message

Gerhard Wiesinger Feb. 18, 2012, 11:15 a.m. UTC 
On Sat, 18 Feb 2012, Avi Kivity wrote:

> On 02/17/2012 07:11 PM, Gerhard Wiesinger wrote:
>>>> gcc --version
>>>> gcc (GCC) 4.6.2 20111027 (Red Hat 4.6.2-1)
>>>>
>>>> Please submit a new patch which removes unused variables, etc.
>>>>
>>>
>>> As already mentioned, this patch is against qemu.git, not qemu-kvm.git.
>>> The latter doesn't (yet) have the infrastructure changes, so
>>> obviously it won't compile there.
>>>
>>> For the unused variables I'll be updating the patch.
>>
>>
>> @Avi: Can you pull from qemu to test also qemu-kvm?
>>
>
> I pulled on Thursday, are you testing the latest?
>
> I something's missing, you can always do a local merge btw.

Yes, latest. With patch below, it works also on qemu-kvm.
So something is still missing ...

Local merge: But then history isn't the same ...
What command do you use?

Ciao,
Gerhard

--
http://www.wiesinger.com/

Comments

Avi Kivity Feb. 18, 2012, 11:53 a.m. UTC | #1 
On 02/18/2012 01:15 PM, Gerhard Wiesinger wrote:
>> I pulled on Thursday, are you testing the latest?
>>
>> I something's missing, you can always do a local merge btw.
>
>
> Yes, latest. With patch below, it works also on qemu-kvm.
> So something is still missing ...
>

You'll need that patch for qemu.git too.

> Local merge: But then history isn't the same ...

For testing, why do you care?

> What command do you use?

git pull
diff mbox

Patch

--- hw/megasas.c        2012-02-18 09:08:17.000000000 +0100
+++ ../qemu/hw/megasas.c        2012-02-17 20:27:23.000000000 +0100
@@ -1862,4 +1862,4 @@ 
      type_register_static(&megasas_info);
  }

-device_init(megaraid1078_register_types);
+type_init(megaraid1078_register_types);