|
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
[Prev in Thread] | Current Thread | [Next in Thread] |