qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH][v9] megasas: LSI Megaraid SAS HBA emulation


From: Anthony Liguori
Subject: Re: [Qemu-devel] [PATCH][v9] megasas: LSI Megaraid SAS HBA emulation
Date: Mon, 16 Jan 2012 07:39:21 -0600
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.23) Gecko/20110922 Lightning/1.0b2 Thunderbird/3.1.15

On 01/13/2012 10:54 AM, Andreas Färber wrote:
Am 13.01.2012 12:19, schrieb Hannes Reinecke:
+    if (desc) {
+        trace_megasas_readl_reg("mmio", desc, retval);
+    } else {
+        trace_megasas_readl("mmio", addr, retval);
+    }

Drop _reg version?

Didn't review the SCSI bits. Patch is pretty large, too. ;)

I have the same thought but it looks to be large because of the header and tracing file. I didn't see an obvious place to break it up.

Although Hannes, if there is a way to break it up, that would certainly help 
review.

Regards,

Anthony Liguori


Regards,
Andreas





reply via email to

[Prev in Thread] Current Thread [Next in Thread]