1. 18 1月, 2008 3 次提交
  2. 16 1月, 2008 3 次提交
    • A
      [libata] core checkpatch fix · 0f757743
      Andrew Morton 提交于
      Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>
      Cc: Jeff Garzik <jeff@garzik.org>
      Cc: Tejun Heo <htejun@gmail.com>
      Signed-off-by: NAndrew Morton <akpm@linux-foundation.org>
      Signed-off-by: NJeff Garzik <jeff@garzik.org>
      0f757743
    • A
      [libata] pata_bf54x: checkpatch fixes · ed722d3d
      Andrew Morton 提交于
      WARNING: line over 80 characters
      #36: FILE: drivers/ata/pata_bf54x.c:1512:
      +	while (bfin_port_info[board_idx].udma_mask>0 && udma_fsclk[udma_mode] > fsclk) {
      
      ERROR: need spaces around that '>' (ctx:VxV)
      #36: FILE: drivers/ata/pata_bf54x.c:1512:
      +	while (bfin_port_info[board_idx].udma_mask>0 && udma_fsclk[udma_mode] > fsclk) {
       	                                          ^
      
      total: 1 errors, 1 warnings, 19 lines checked
      
      Your patch has style problems, please review.  If any of these errors
      are false positives report them to the maintainer, see
      CHECKPATCH in MAINTAINERS.
      
      Please run checkpatch prior to sending patches
      
      Cc: Jeff Garzik <jeff@garzik.org>
      Cc: Sonic Zhang <sonic.zhang@analog.com>
      Cc: Tejun Heo <htejun@gmail.com>
      Cc: sonic zhang <sonic.adi@gmail.com>
      Signed-off-by: NAndrew Morton <akpm@linux-foundation.org>
      Signed-off-by: NJeff Garzik <jeff@garzik.org>
      ed722d3d
    • A
      libata fixes for sparse-found problems · b50e56d8
      Al Viro 提交于
      In pata_legacy and pata_winbond we've got bugs - cpu_to_le16() instead
      of cpu_to_le32().  Fortunately, both affected suckers are VLB, thus
      l-e-only, so we might get away with that unless we hit it with slop == 3
      (hadn't checked if playing with badly aligned sg could trigger that).
      Still buggy...  Moreover, pata_legacy, pata_winbond and pata_qdi forgot to
      initialize pad on the write side of 32bit case in their ->data_xfer().
      Hopefully the hardware does't care, but still, sending uninitialized
      data to it...
      Signed-off-by: NAl Viro <viro@zeniv.linux.org.uk>
      Signed-off-by: NJeff Garzik <jeff@garzik.org>
      b50e56d8
  3. 15 1月, 2008 16 次提交
  4. 14 1月, 2008 1 次提交
  5. 13 1月, 2008 16 次提交
  6. 12 1月, 2008 1 次提交
    • J
      ps3fb: fix deadlock on kexec() · 8dab6376
      Jeremy Kerr 提交于
      Since the introduction of the acquire_console_sem calls in
      0333d835, kexecing can cause the
      kernel to deadlock:
      
       ps3fb_shutdown()
        -> unregister_framebuffer()
        -> fb_notifier_call_chain(FB_EVENT_FB_UNBIND)
        -> fbcon_fb_unbind()
        -> unbind_con_driver()
        -> bind_con_driver()
      	[ acquires console_sem ]
        -> fbcon_deinit()
        -> fbops->fb_release(newinfo, 0)
        -> ps3fb_release()
        -> ps3fb_sync()
      	[ acquires console_sem ]
      
      This change avoids the deadlock by moving the acquire_console_sem()
      out of ps3fb_sync(), and puts it into the two other callsites, leaving
      ps3fb_release() to call ps3fb_sync() without the console semaphore.
      
      [Geert]
        - Corrected call sequence above
        - ps3fb_release() may be called with and without console_sem held. This is an
          inconsistency that should be fixed at the fb level, but for now, try to
          acquire console_sem in ps3fb_release().
      
          I think it's safer to let ps3fb_release() try to acquire console_sem and
          not refresh the screen if it fails, than to call ps3fb_sync() without
          holding console_sem, as ps3fb_par may be modified at the same time, causing
          crashes or lockups.
      
          Besides, ps3fb_release() only calls ps3fb_sync() to refresh the screen
          when display flipping is disabled, which is an uncommon case (except during
          shutdown/kexec).
      Signed-off-by: NJeremy Kerr <jk@ozlabs.org>
      Signed-off-by: NGeert Uytterhoeven <Geert.Uytterhoeven@sonycom.com>
      Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
      Signed-off-by: NLinus Torvalds <torvalds@linux-foundation.org>
      8dab6376