|To:||Michael Chan <mchan@xxxxxxxxxxxx>|
|Subject:||Re: [PATCH 2.6.12-rc5 9/9] tg3: Fix bug in tg3_load_firmware_cpu|
|From:||Jeff Garzik <jgarzik@xxxxxxxxx>|
|Date:||Thu, 26 May 2005 15:13:47 -0400|
|User-agent:||Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.6) Gecko/20050328 Fedora/1.7.6-1.2.5|
Michael Chan wrote:
Add tg3_nvram_lock() and tg3_nvram_unlock() calls around tg3_halt_cpu(). It is possible that the bootcode may be loading code from nvram during this call and stopping the cpu without getting the lock may cause uncompleted nvram data to be left in the nvram data register. Subsequent calls to read/write nvram data will fail. Signed-off-by: Michael Chan <mchan@xxxxxxxxxxxx>
All the patches seem sane to me.To touch a bit on patch order, I would suggest ordering bug fixes before feature patches. Your patch #9 could have been better ordered as patch #1.
The patch order did not matter today, but sometime in the future, if an add-a-feature patch is rejected or delayed, you would still likely want DaveM to apply other bugfix patches.
|<Prev in Thread]||Current Thread||[Next in Thread>|
|Previous by Date:||[NET] 2.6.x build warning, Jeff Garzik|
|Next by Date:||Re: user space checksum, Stephen Hemminger|
|Previous by Thread:||[PATCH 2.6.12-rc5 9/9] tg3: Fix bug in tg3_load_firmware_cpu, Michael Chan|
|Next by Thread:||Re: [PATCH 2.6.12-rc5 9/9] tg3: Fix bug in tg3_load_firmware_cpu, David S. Miller|
|Indexes:||[Date] [Thread] [Top] [All Lists]|