1. 03 10月, 2006 3 次提交
    • A
      [PATCH] isdn: more pr_debug() fixes · 3879b6b6
      Andrew Morton 提交于
      drivers/isdn/sc/event.c: In function 'indicate_status':
      drivers/isdn/sc/event.c:49: error: 'events' undeclared (first use in this function)
      drivers/isdn/sc/event.c:49: error: (Each undeclared identifier is reported only once
      drivers/isdn/sc/event.c:49: error: for each function it appears in.)
      drivers/isdn/sc/event.c:49: warning: format '%d' expects type 'int', but argument 4 has type 'ulong'
      
      drivers/isdn/sc/interrupt.c: In function 'interrupt_handler':
      drivers/isdn/sc/interrupt.c:97: warning: format '%x' expects type 'unsigned int', but argument 4 has type 'long unsigned int'
      
      drivers/isdn/sc/timer.c: In function 'check_reset':
      drivers/isdn/sc/timer.c:80: warning: format '%d' expects type 'int', but argument 3 has type 'long unsigned int'
      Acked-by: NKarsten Keil <kkeil@suse.de>
      Cc: Zach Brown <zach.brown@oracle.com>
      Signed-off-by: NAndrew Morton <akpm@osdl.org>
      Signed-off-by: NLinus Torvalds <torvalds@osdl.org>
      3879b6b6
    • A
      [PATCH] isdn-debug-build-fix · 20f43535
      Andrew Morton 提交于
      When pr_debug() is enabled:
      
      drivers/isdn/sc/command.c: In function 'command':
      drivers/isdn/sc/command.c:107: error: 'commands' undeclared (first use in this function)
      drivers/isdn/sc/command.c:107: error: (Each undeclared identifier is reported only once
      drivers/isdn/sc/command.c:107: error: for each function it appears in.)
      drivers/isdn/sc/command.c:122: warning: format '%x' expects type 'unsigned int', but argument 3 has type 'long unsigned int'
      drivers/isdn/sc/command.c: In function 'dial':
      drivers/isdn/sc/command.c:199: warning: format '%d' expects type 'int', but argument 4 has type 'long unsigned int'
      drivers/isdn/sc/command.c: In function 'answer':
      drivers/isdn/sc/command.c:221: warning: format '%s' expects type 'char *', but argument 3 has type 'long unsigned int'
      drivers/isdn/sc/command.c: In function 'hangup':
      drivers/isdn/sc/command.c:244: warning: format '%d' expects type 'int', but argument 3 has type 'long unsigned int'
      drivers/isdn/sc/command.c: In function 'setl2':
      drivers/isdn/sc/command.c:265: error: 'l2protos' undeclared (first use in this function)
      drivers/isdn/sc/command.c: In function 'setl3':
      drivers/isdn/sc/command.c:297: error: 'l3protos' undeclared (first use in this function)
      drivers/isdn/sc/command.c:297: warning: format '%d' expects type 'int', but argument 3 has type 'long unsigned int'
      drivers/isdn/sc/command.c: In function 'acceptb':
      drivers/isdn/sc/command.c:315: warning: format '%d' expects type 'int', but argument 3 has type 'long unsigned int'
      drivers/isdn/sc/command.c: In function 'clreaz':
      drivers/isdn/sc/command.c:330: warning: format '%d' expects type 'int', but argument 3 has type 'long unsigned int'
      drivers/isdn/sc/command.c: In function 'seteaz':
      drivers/isdn/sc/command.c:345: warning: format '%d' expects type 'int', but argument 3 has type 'long unsigned int'
      
      Cc: Zach Brown <zach.brown@oracle.com>
      Acked-by: NKarsten Keil <kkeil@suse.de>
      Signed-off-by: NAndrew Morton <akpm@osdl.org>
      Signed-off-by: NLinus Torvalds <torvalds@osdl.org>
      20f43535
    • A
      [PATCH] ISDN warning fixes · 17a4506d
      Alan Cox 提交于
      Clean up warnings in drivers/isdn by using long not int for the values
      where we pass void * and cast to integer types.  The code is ok (ok passing
      the stuff this way isn't pretty but the code is valid).  In all the cases I
      checked out the right thing happens anyway but this removes all the
      warnings.
      Signed-off-by: NAlan Cox <alan@redhat.com>
      Cc: Jeff Garzik <jeff@garzik.org>
      Acked-by: NKarsten Keil <kkeil@suse.de>
      Signed-off-by: NAndrew Morton <akpm@osdl.org>
      Signed-off-by: NLinus Torvalds <torvalds@osdl.org>
      17a4506d
  2. 02 10月, 2006 3 次提交
  3. 01 10月, 2006 1 次提交
  4. 29 9月, 2006 1 次提交
  5. 28 9月, 2006 1 次提交
    • P
      USB: Dealias -110 code (more complete) · 38e2bfc9
      Pete Zaitcev 提交于
      The purpose of this patch is to split off the case when a device does
      not reply on the lower level (which is reported by HC hardware), and
      a case when the device accepted the request, but does not reply at
      upper level. This redefinition allows to diagnose issues easier,
      without asking the user if the -110 happened "immediately".
      
      The usbmon splits such cases already thanks to its timestamp, but
      it's not always available.
      
      I adjusted all drivers which I found affected (by searching for "urb").
      Out of tree drivers may suffer a little bit, but I do not expect much
      breakage. At worst they may print a few messages.
      Signed-off-by: NPete Zaitcev <zaitcev@redhat.com>
      Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
      38e2bfc9
  6. 27 9月, 2006 1 次提交
  7. 06 8月, 2006 1 次提交
  8. 20 7月, 2006 1 次提交
  9. 11 7月, 2006 2 次提交
  10. 03 7月, 2006 1 次提交
  11. 01 7月, 2006 1 次提交
  12. 30 6月, 2006 3 次提交
  13. 28 6月, 2006 4 次提交
  14. 27 6月, 2006 11 次提交
    • K
      [PATCH] fix processing of the last byte in isdn_readbchan_tty() · 916d1544
      Karsten Keil 提交于
      The changes in the tty handling contain a bug while accessing
      the last byte in the skb. Since special sequence for control of
      DTMF and FAX via ttyI* devices handled via this path, these services
      do not work anymore.
      Signed-off-by: NKarsten Keil <kkeil@suse.de>
      Signed-off-by: NLinus Torvalds <torvalds@osdl.org>
      916d1544
    • G
      [PATCH] devfs: Rename TTY_DRIVER_NO_DEVFS to TTY_DRIVER_DYNAMIC_DEV · 331b8319
      Greg Kroah-Hartman 提交于
      I've always found this flag confusing.  Now that devfs is no longer around, it
      has been renamed, and the documentation for when this flag should be used has
      been updated.
      
      Also fixes all drivers that use this flag.
      Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
      331b8319
    • G
      [PATCH] devfs: Remove the tty_driver devfs_name field as it's no longer needed · f4eaa370
      Greg Kroah-Hartman 提交于
      Also fixes all drivers that set this field.
      Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
      f4eaa370
    • G
      [PATCH] devfs: Remove the devfs_fs_kernel.h file from the tree · ff23eca3
      Greg Kroah-Hartman 提交于
      Also fixes up all files that #include it.
      Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
      ff23eca3
    • G
      [PATCH] devfs: Remove devfs_remove() function from the kernel tree · 8ab5e4c1
      Greg Kroah-Hartman 提交于
      Removes the devfs_remove() function and all callers of it.
      Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
      8ab5e4c1
    • G
      [PATCH] devfs: Remove devfs_mk_cdev() function from the kernel tree · 7c69ef79
      Greg Kroah-Hartman 提交于
      Removes the devfs_mk_cdev() function and all callers of it.
      Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
      7c69ef79
    • T
      [PATCH] isdn4linux: Gigaset driver cleanup · 698e3ed9
      Tilman Schmidt 提交于
      The following patch to the common part of the Siemens Gigaset driver
      prevents it from trying to send the +++ break sequence if the device has
      been disconnected, and removes a couple of assignments which didn't have
      any effect.
      Signed-off-by: NTilman Schmidt <tilman@imap.cc>
      Acked-by: NHansjoerg Lipp <hjlipp@web.de>
      Cc: Karsten Keil <kkeil@suse.de>
      Signed-off-by: NAndrew Morton <akpm@osdl.org>
      Signed-off-by: NLinus Torvalds <torvalds@osdl.org>
      698e3ed9
    • T
      [PATCH] isdn4linux: Gigaset base driver: improve error recovery · 06163f86
      Tilman Schmidt 提交于
      The following patch to the Siemens Gigaset base driver adds graceful
      recovery for some frequently encountered error conditions, by retrying
      failed control requests (eg.  stalled control pipe), and by closing and
      reopening the AT command channel when it appears to be stuck.
      Signed-off-by: NTilman Schmidt <tilman@imap.cc>
      Acked-by: NHansjoerg Lipp <hjlipp@web.de>
      Cc: Karsten Keil <kkeil@suse.de>
      Signed-off-by: NAndrew Morton <akpm@osdl.org>
      Signed-off-by: NLinus Torvalds <torvalds@osdl.org>
      06163f86
    • E
      [PATCH] Fix typo in drivers/isdn/hisax/q931.c · 9f13fae2
      Eric Sesterhenn 提交于
      This fixes coverity bug #517.
      
      Since IESIZE is greater than IESIZE_NI1 we might run past the end of
      ielist_ni1.  This fixes it by using the proper IESIZE_NI1 define.
      Signed-off-by: NEric Sesterhenn <snakebyte@gmx.de>
      Acked-by: NKarsten Keil <kkeil@suse.de>
      Signed-off-by: NAndrew Morton <akpm@osdl.org>
      Signed-off-by: NLinus Torvalds <torvalds@osdl.org>
      9f13fae2
    • M
      [PATCH] CAPI crash / race condition · 6aa65472
      Michael Buesch 提交于
      I am getting more or less reproducible crashes from the CAPI subsystem
      using the fcdsl driver:
      
      Unable to handle kernel NULL pointer dereference at virtual address 00000010
       printing eip:
      c39bbca4
      *pde = 00000000
      Oops: 0000 [#1]
      Modules linked in: netconsole capi capifs 3c59x mii fcdsl kernelcapi uhci_hcd usbcore ide_cd cdrom
      CPU:    0
      EIP:    0060:[<c39bbca4>]    Tainted: P      VLI
      EFLAGS: 00010202   (2.6.16.11 #3)
      EIP is at handle_minor_send+0x17a/0x241 [capi]
      eax: c24abbc0   ebx: c0b4c980   ecx: 00000010   edx: 00000010
      esi: c1679140   edi: c2783016   ebp: 0000c28d   esp: c0327e24
      ds: 007b   es: 007b   ss: 0068
      Process swapper (pid: 0, threadinfo=c0326000 task=c02e1300)
      Stack: <0>000005b4 c1679180 00000000 c28d0000 c1ce04e0 c2f69654 c221604e c1679140
             c39bc19a 00000038 c20c0400 c075c560 c1f2f800 00000000 c01dc9b5 c1e96a40
             c075c560 c2ed64c0 c1e96a40 c01dcd3b c2fb94e8 c075c560 c0327f00 c1e96a40
      Call Trace:
       [<c39bc19a>] capinc_tty_write+0xda/0xf3 [capi]
       [<c01dc9b5>] ppp_sync_push+0x52/0xfe
       [<c01dcd3b>] ppp_sync_send+0x1f5/0x204
       [<c01d9bc1>] ppp_push+0x3e/0x9c
       [<c01dacd4>] ppp_xmit_process+0x422/0x4cc
       [<c01daf3f>] ppp_start_xmit+0x1c1/0x1f6
       [<c0213ea5>] qdisc_restart+0xa7/0x135
       [<c020b112>] dev_queue_xmit+0xba/0x19e
       [<c0223f69>] ip_output+0x1eb/0x236
       [<c0220907>] ip_forward+0x1c1/0x21a
       [<c021fa6c>] ip_rcv+0x38e/0x3ea
       [<c020b4c2>] netif_receive_skb+0x166/0x195
       [<c020b55e>] process_backlog+0x6d/0xd2
       [<c020a30f>] net_rx_action+0x6a/0xff
       [<c0112909>] __do_softirq+0x35/0x7d
       [<c0112973>] do_softirq+0x22/0x26
       [<c0103a9d>] do_IRQ+0x1e/0x25
       [<c010255a>] common_interrupt+0x1a/0x20
       [<c01013c5>] default_idle+0x2b/0x53
       [<c0101426>] cpu_idle+0x39/0x4e
       [<c0328386>] start_kernel+0x20b/0x20d
      Code: c0 e8 b3 b6 77 fc 85 c0 75 10 68 d8 c8 9b c3 e8 82 3d 75 fc 8b 43 60 5a eb 50 8d 56 50 c7 00 00 00 00 00 66 89 68 04 eb 02 89
      ca <8b> 0a 85 c9 75 f8 89 02 89 da ff 46 54 8b 46 10 e8 30 79 fd ff
       <0>Kernel panic - not syncing: Fatal exception in interrupt
      
      That oops took me to the "ackqueue" implementation in capi.c.  The crash
      occured in capincci_add_ack() (auto-inlined by the compiler).
      
      I read the code a bit and finally decided to replace the custom linked list
      implementation (struct capiminor->ackqueue) by a struct list_head.  That
      did not solve the crash, but produced the following interresting oops:
      
      Unable to handle kernel paging request at virtual address 00200200
       printing eip:
      c39bb1f5
      *pde = 00000000
      Oops: 0002 [#1]
      Modules linked in: netconsole capi capifs 3c59x mii fcdsl kernelcapi uhci_hcd usbcore ide_cd cdrom
      CPU:    0
      EIP:    0060:[<c39bb1f5>]    Tainted: P      VLI
      EFLAGS: 00010246   (2.6.16.11 #3)
      EIP is at capiminor_del_ack+0x18/0x49 [capi]
      eax: 00200200   ebx: c18d41a0   ecx: c1385620   edx: 00100100
      esi: 0000d147   edi: 00001103   ebp: 0000d147   esp: c1093f3c
      ds: 007b   es: 007b   ss: 0068
      Process events/0 (pid: 3, threadinfo=c1092000 task=c1089030)
      Stack: <0>c2a17580 c18d41a0 c39bbd16 00000038 c18d41e0 00000000 d147c640 c29e0b68
             c29e0b90 00000212 c29e0b68 c39932b2 c29e0bb0 c10736a0 c0119ef0 c399326c
             c10736a8 c10736a0 c10736b0 c0119f93 c011a06e 00000001 00000000 00000000
      Call Trace:
       [<c39bbd16>] handle_minor_send+0x1af/0x241 [capi]
       [<c39932b2>] recv_handler+0x46/0x5f [kernelcapi]
       [<c0119ef0>] run_workqueue+0x5e/0x8d
       [<c399326c>] recv_handler+0x0/0x5f [kernelcapi]
       [<c0119f93>] worker_thread+0x0/0x10b
       [<c011a06e>] worker_thread+0xdb/0x10b
       [<c010c998>] default_wake_function+0x0/0xc
       [<c011c399>] kthread+0x90/0xbc
       [<c011c309>] kthread+0x0/0xbc
       [<c0100a65>] kernel_thread_helper+0x5/0xb
      Code: 7e 02 89 ee 89 f0 5a f7 d0 c1 f8 1f 5b 21 f0 5e 5f 5d c3 56 53 8b 48 50 89 d6 89 c3 8b 11 eb 2f 66 39 71 08 75 25 8b 41 04 8b 11 <89> 10 89 42 04 c7 01 00 01 10 00 89 c8 c7 41 04 00 02 20 00 e8
      
      The interresting part of it is the "virtual address 00200200", which is
      LIST_POISON2.  I thought about some race condition, but as this is an UP
      system, it leads to questions on how it can happen.  If we look at EFLAGS:
      00010202, we see that interrupts are enabled at the time of the crash
      (eflags & 0x200).
      
      Finally, I don't understand all the capi code, but I think that
      handle_minor_send() is racing somehow against capi_recv_message(), which
      call both capiminor_del_ack().  So if an IRQ occurs in the middle of
      capiminor_del_ack() and another instance of it is invoked, it leads to
      linked list corruption.
      
      I came up with the following patch.  With this, I could not reproduce the
      crash anymore.  Clearly, this is not the correct fix for the issue.  As this
      seems to be some locking issue, there might be more locking issues in that
      code.  For example, doesn't the whole struct capiminor have to be locked
      somehow?
      
      Cc: Carsten Paeth <calle@calle.de>
      Cc: Kai Germaschewski <kai.germaschewski@gmx.de>
      Cc: Karsten Keil <kkeil@suse.de>
      Signed-off-by: NAndrew Morton <akpm@osdl.org>
      Signed-off-by: NLinus Torvalds <torvalds@osdl.org>
      6aa65472
    • A
      spelling fixes · d6e05edc
      Andreas Mohr 提交于
      acquired (aquired)
      contiguous (contigious)
      successful (succesful, succesfull)
      surprise (suprise)
      whether (weather)
      some other misspellings
      Signed-off-by: NAndreas Mohr <andi@lisas.de>
      Signed-off-by: NAdrian Bunk <bunk@stusta.de>
      d6e05edc
  15. 23 6月, 2006 3 次提交
    • J
      [PATCH] ISDN: correctly handle isdn_writebuf_stub() errors · d20d04bc
      Jesper Juhl 提交于
      isdn_writebuf_stub() forgets to detect memory allocation and uaccess errors.
      And when that's fixed, if a error happens the caller will just keep on
      looping.
      
      So change the caller to detect the error, and to return it.
      Signed-off-by: NJesper Juhl <jesper.juhl@gmail.com>
      Cc: Karsten Keil <kkeil@suse.de>
      Signed-off-by: NTilman Schmidt <tilman@imap.cc>
      Signed-off-by: NAndrew Morton <akpm@osdl.org>
      Signed-off-by: NLinus Torvalds <torvalds@osdl.org>
      d20d04bc
    • J
      [PATCH] i4l: memory leak fix for sc_ioctl(). · 94a6735c
      Jesper Juhl 提交于
      Fix leak of `rcvmsg' in sc_ioctl().
      
      There are two returns in the switch in sc_ioctl (the SCIOCSTART case) that
      may leak `rcvmsg'.  This patch fixes that by adding a kfree() call at the
      beginning of that case.
      
      Bug found by the coverity checker as #1098
      
      Eric Sesterhenn send me a patch to fix the leak(s) by adding 2 kfree()
      calls before the returns, I changed that into just a single call at the
      beginning.
      Signed-off-by: NJesper Juhl <jesper.juhl@gmail.com>
      Cc: Karsten Keil <kkeil@suse.de>
      Signed-off-by: NAndrew Morton <akpm@osdl.org>
      Signed-off-by: NLinus Torvalds <torvalds@osdl.org>
      94a6735c
    • D
      [PATCH] VFS: Permit filesystem to override root dentry on mount · 454e2398
      David Howells 提交于
      Extend the get_sb() filesystem operation to take an extra argument that
      permits the VFS to pass in the target vfsmount that defines the mountpoint.
      
      The filesystem is then required to manually set the superblock and root dentry
      pointers.  For most filesystems, this should be done with simple_set_mnt()
      which will set the superblock pointer and then set the root dentry to the
      superblock's s_root (as per the old default behaviour).
      
      The get_sb() op now returns an integer as there's now no need to return the
      superblock pointer.
      
      This patch permits a superblock to be implicitly shared amongst several mount
      points, such as can be done with NFS to avoid potential inode aliasing.  In
      such a case, simple_set_mnt() would not be called, and instead the mnt_root
      and mnt_sb would be set directly.
      
      The patch also makes the following changes:
      
       (*) the get_sb_*() convenience functions in the core kernel now take a vfsmount
           pointer argument and return an integer, so most filesystems have to change
           very little.
      
       (*) If one of the convenience function is not used, then get_sb() should
           normally call simple_set_mnt() to instantiate the vfsmount. This will
           always return 0, and so can be tail-called from get_sb().
      
       (*) generic_shutdown_super() now calls shrink_dcache_sb() to clean up the
           dcache upon superblock destruction rather than shrink_dcache_anon().
      
           This is required because the superblock may now have multiple trees that
           aren't actually bound to s_root, but that still need to be cleaned up. The
           currently called functions assume that the whole tree is rooted at s_root,
           and that anonymous dentries are not the roots of trees which results in
           dentries being left unculled.
      
           However, with the way NFS superblock sharing are currently set to be
           implemented, these assumptions are violated: the root of the filesystem is
           simply a dummy dentry and inode (the real inode for '/' may well be
           inaccessible), and all the vfsmounts are rooted on anonymous[*] dentries
           with child trees.
      
           [*] Anonymous until discovered from another tree.
      
       (*) The documentation has been adjusted, including the additional bit of
           changing ext2_* into foo_* in the documentation.
      
      [akpm@osdl.org: convert ipath_fs, do other stuff]
      Signed-off-by: NDavid Howells <dhowells@redhat.com>
      Acked-by: NAl Viro <viro@zeniv.linux.org.uk>
      Cc: Nathan Scott <nathans@sgi.com>
      Cc: Roland Dreier <rolandd@cisco.com>
      Signed-off-by: NAndrew Morton <akpm@osdl.org>
      Signed-off-by: NLinus Torvalds <torvalds@osdl.org>
      454e2398
  16. 22 6月, 2006 1 次提交
  17. 22 5月, 2006 1 次提交
  18. 16 5月, 2006 1 次提交