1. 10 12月, 2006 22 次提交
  2. 09 12月, 2006 18 次提交
    • R
      [AX.25]: Fix default address and broadcast address initialization. · 15b1c0e8
      Ralf Baechle 提交于
      Only the callsign but not the SSID part of an AX.25 address is ASCII
      based but Linux by initializes the SSID which should be just a 4-bit
      number from ASCII anyway.
      
      Fix that and convert the code to use a shared constant for both default
      addresses.  While at it, use the same style for null_ax25_address also.
      Signed-off-by: NRalf Baechle <ralf@linux-mips.org>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      15b1c0e8
    • A
      [BNX2]: Add an error check. · 6c1bbcc8
      Adrian Bunk 提交于
      This patch adds a missing error check spotted by the Coverity checker.
      Signed-off-by: NAdrian Bunk <bunk@stusta.de>
      Acked-by: NJeff Garzik <jgarzik@pobox.com>
      Acked-by: NMichael Chan <mchan@broadcom.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      6c1bbcc8
    • J
      [PATCH] Generic HID layer - build · 63f3861d
      Jiri Kosina 提交于
      This modifies Makefiles and Kconfigs to properly reflect the creation of
      generic HID layer.
      
      It also removes the dependency of BROKEN, which was introduced by the
      first patch in series (see the comment). Also updates credits.
      Signed-off-by: NJiri Kosina <jkosina@suse.cz>
      Signed-off-by: NMarcel Holtmann <marcel@holtmann.org>
      Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
      Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
      63f3861d
    • J
      [PATCH] Generic HID layer - pb_fnmode · 4c2ae844
      Jiri Kosina 提交于
      pb_fnmode parameter has to be passed to usbhid, both for compatibility reasons
      and also because it logically belongs there.
      
      Also removes empty hid-input.c file in drivers/usb/input.
      Signed-off-by: NJiri Kosina <jkosina@suse.cz>
      Signed-off-by: NMarcel Holtmann <marcel@holtmann.org>
      Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
      Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
      4c2ae844
    • J
      [PATCH] Generic HID layer - input and event reporting · aa8de2f0
      Jiri Kosina 提交于
      hid_input_report() was needlessly USB-specific in USB HID. This patch
      makes the function independent of HID implementation and fixes all
      the current users. Bluetooth patches comply with this prototype.
      Signed-off-by: NJiri Kosina <jkosina@suse.cz>
      Signed-off-by: NMarcel Holtmann <marcel@holtmann.org>
      Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
      Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
      aa8de2f0
    • J
      [PATCH] Generic HID layer - hiddev · aa938f79
      Jiri Kosina 提交于
      - hiddev is USB-only (agreed with Marcel Holtmann that Bluetooth currently
        doesn't need it, and future planned interface (rawhid) will be more flexible
        and usable)
      - both HID and USB-hid can be now compiled as modules (wasn't possible before
        hiddev was fully separated from generic HID layer)
      Signed-off-by: NJiri Kosina <jkosina@suse.cz>
      Signed-off-by: NMarcel Holtmann <marcel@holtmann.org>
      Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
      Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
      aa938f79
    • J
      [PATCH] Generic HID layer - USB API · 4916b3a5
      Jiri Kosina 提交于
      - 'dev' in struct hid_device changed from struct usb_device to
        struct device and fixed all the users
      - renamed functions which are part of USB HID API from 'hid_*' to
        'usbhid_*'
      - force feedback initialization moved from common part into USB-specific
        driver
      - added usbhid.h header for USB HID API users
      - removed USB-specific fields from struct hid_device and moved them
        to new usbhid_device, which is pointed to by hid_device->driver_data
      - fixed all USB users to use this new structure
      Signed-off-by: NJiri Kosina <jkosina@suse.cz>
      Signed-off-by: NMarcel Holtmann <marcel@holtmann.org>
      Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
      Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
      4916b3a5
    • J
      [PATCH] Generic HID layer - API · 229695e5
      Jiri Kosina 提交于
      - fixed generic API (added neccessary EXPORT_SYMBOL, fixed hid.h to provide correct
        prototypes)
      - extended hid_device with open/close/event function pointers to driver-specific
        functions
      - added driver specific driver_data to hid_device
      Signed-off-by: NJiri Kosina <jkosina@suse.cz>
      Signed-off-by: NMarcel Holtmann <marcel@holtmann.org>
      Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
      Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
      229695e5
    • J
      [PATCH] Generic HID layer - code split · dde5845a
      Jiri Kosina 提交于
      The "big main" split of USB HID code into generic HID code and
      USB-transport specific HID handling.
      Signed-off-by: NJiri Kosina <jkosina@suse.cz>
      Signed-off-by: NMarcel Holtmann <marcel@holtmann.org>
      Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
      Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
      dde5845a
    • J
      [PATCH] Generic HID layer - disable USB HID · 64bb67b1
      Jiri Kosina 提交于
      This patch is a part of generic HID layer introduction. USB HID is
      disabled, so that the code split and changes could be introduced in a
      way that is reviewable (i.e. separate patches), but not to break git
      bisect by uncompilable kernel throughout different stages of the code
      splitup and changes. The last patch of this series enables HID again.
      Signed-off-by: NJiri Kosina <jkosina@suse.cz>
      Signed-off-by: NMarcel Holtmann <marcel@holtmann.org>
      Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
      Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
      64bb67b1
    • A
      c642f9e0
    • J
      [PATCH] dm: raid1: reset sync_search on resume · 88b20a1a
      Jonathan E Brassow 提交于
      Reset sync_search on resume.  The effect is to retry syncing all out-of-sync
      regions when a mirror is resumed, including ones that previously failed.
      Signed-off-by: NJonathan E Brassow <jbrassow@redhat.com>
      Signed-off-by: NAlasdair G Kergon <agk@redhat.com>
      Cc: dm-devel@redhat.com
      Signed-off-by: NAndrew Morton <akpm@osdl.org>
      Signed-off-by: NLinus Torvalds <torvalds@osdl.org>
      88b20a1a
    • J
      [PATCH] dm: log: rename complete_resync_work · f3ee6b2f
      Jonathan E Brassow 提交于
      The complete_resync_work function only provides the ability to change an
      out-of-sync region to in-sync.  This patch enhances the function to allow us
      to change the status from in-sync to out-of-sync as well, something that is
      needed when a mirror write to one of the devices or an initial resync on a
      given region fails.
      Signed-off-by: NJonathan E Brassow <jbrassow@redhat.com>
      Signed-off-by: NAlasdair G Kergon <agk@redhat.com>
      Cc: dm-devel@redhat.com
      Signed-off-by: NAndrew Morton <akpm@osdl.org>
      Signed-off-by: NLinus Torvalds <torvalds@osdl.org>
      f3ee6b2f
    • M
      [PATCH] dm: snapshot: abstract memory release · 31c93a0c
      Milan Broz 提交于
      Move the code that releases memory used by a snapshot into a separate function.
      Signed-off-by: NMilan Broz <mbroz@redhat.com>
      Signed-off-by: NAlasdair G Kergon <agk@redhat.com>
      Cc: dm-devel@redhat.com
      Signed-off-by: NAndrew Morton <akpm@osdl.org>
      Signed-off-by: NLinus Torvalds <torvalds@osdl.org>
      31c93a0c
    • K
      [PATCH] dm: mpath: use noflush suspending · 45e15720
      Kiyoshi Ueda 提交于
      Implement the pushback feature for the multipath target.
      
      The pushback request is used when:
        1) there are no valid paths;
        2) queue_if_no_path was set;
        3) a suspend is being issued with the DMF_NOFLUSH_SUSPENDING flag.
           Otherwise bios are returned to applications with -EIO.
      
      To check whether queue_if_no_path is specified or not, you need to check
      both queue_if_no_path and saved_queue_if_no_path, because presuspend saves
      the original queue_if_no_path value to saved_queue_if_no_path.
      
      The check for 1 already exists in both map_io() and do_end_io().
      So this patch adds __must_push_back() to check 2 and 3.
      
      Test results:
      See the test results in the preceding patch.
      Signed-off-by: NKiyoshi Ueda <k-ueda@ct.jp.nec.com>
      Signed-off-by: NJun'ichi Nomura <j-nomura@ce.jp.nec.com>
      Signed-off-by: NAlasdair G Kergon <agk@redhat.com>
      Cc: dm-devel@redhat.com
      Signed-off-by: NAndrew Morton <akpm@osdl.org>
      Signed-off-by: NLinus Torvalds <torvalds@osdl.org>
      45e15720
    • K
      [PATCH] dm: suspend: add noflush pushback · 2e93ccc1
      Kiyoshi Ueda 提交于
      In device-mapper I/O is sometimes queued within targets for later processing.
      For example the multipath target can be configured to store I/O when no paths
      are available instead of returning it -EIO.
      
      This patch allows the device-mapper core to instruct a target to transfer the
      contents of any such in-target queue back into the core.  This frees up the
      resources used by the target so the core can replace that target with an
      alternative one and then resend the I/O to it.  Without this patch the only
      way to change the target in such circumstances involves returning the I/O with
      an error back to the filesystem/application.  In the multipath case, this
      patch will let us add new paths for existing I/O to try after all the existing
      paths have failed.
      
          DMF_NOFLUSH_SUSPENDING
          ----------------------
      
      If the DM_NOFLUSH_FLAG ioctl option is specified at suspend time, the
      DMF_NOFLUSH_SUSPENDING flag is set in md->flags during dm_suspend().  It
      is always cleared before dm_suspend() returns.
      
      The flag must be visible while the target is flushing pending I/Os so it
      is set before presuspend where the flush starts and unset after the wait
      for md->pending where the flush ends.
      
      Target drivers can check this flag by calling dm_noflush_suspending().
      
          DM_MAPIO_REQUEUE / DM_ENDIO_REQUEUE
          -----------------------------------
      
      A target's map() function can now return DM_MAPIO_REQUEUE to request the
      device mapper core queue the bio.
      
      Similarly, a target's end_io() function can return DM_ENDIO_REQUEUE to request
      the same.  This has been labelled 'pushback'.
      
      The __map_bio() and clone_endio() functions in the core treat these return
      values as errors and call dec_pending() to end the I/O.
      
          dec_pending
          -----------
      
      dec_pending() saves the pushback request in struct dm_io->error.  Once all
      the split clones have ended, dec_pending() will put the original bio on
      the md->pushback list.  Note that this supercedes any I/O errors.
      
      It is possible for the suspend with DM_NOFLUSH_FLAG to be aborted while
      in progress (e.g. by user interrupt).  dec_pending() checks for this and
      returns -EIO if it happened.
      
          pushdback list and pushback_lock
          --------------------------------
      
      The bio is queued on md->pushback temporarily in dec_pending(), and after
      all pending I/Os return, md->pushback is merged into md->deferred in
      dm_suspend() for re-issuing at resume time.
      
      md->pushback_lock protects md->pushback.
      The lock should be held with irq disabled because dec_pending() can be
      called from interrupt context.
      
      Queueing bios to md->pushback in dec_pending() must be done atomically
      with the check for DMF_NOFLUSH_SUSPENDING flag.  So md->pushback_lock is
      held when checking the flag.  Otherwise dec_pending() may queue a bio to
      md->pushback after the interrupted dm_suspend() flushes md->pushback.
      Then the bio would be left in md->pushback.
      
      Flag setting in dm_suspend() can be done without md->pushback_lock because
      the flag is checked only after presuspend and the set value is already
      made visible via the target's presuspend function.
      
      The flag can be checked without md->pushback_lock (e.g. the first part of
      the dec_pending() or target drivers), because the flag is checked again
      with md->pushback_lock held when the bio is really queued to md->pushback
      as described above.  So even if the flag is cleared after the lockless
      checkings, the bio isn't left in md->pushback but returned to applications
      with -EIO.
      
          Other notes on the current patch
          --------------------------------
      
      - md->pushback is added to the struct mapped_device instead of using
        md->deferred directly because md->io_lock which protects md->deferred is
        rw_semaphore and can't be used in interrupt context like dec_pending(),
        and md->io_lock protects the DMF_BLOCK_IO flag of md->flags too.
      
      - Don't issue lock_fs() in dm_suspend() if the DM_NOFLUSH_FLAG
        ioctl option is specified, because I/Os generated by lock_fs() would be
        pushed back and never return if there were no valid devices.
      
      - If an error occurs in dm_suspend() after the DMF_NOFLUSH_SUSPENDING
        flag is set, md->pushback must be flushed because I/Os may be queued to
        the list already.  (flush_and_out label in dm_suspend())
      
          Test results
          ------------
      
      I have tested using multipath target with the next patch.
      
      The following tests are for regression/compatibility:
        - I/Os succeed when valid paths exist;
        - I/Os fail when there are no valid paths and queue_if_no_path is not
          set;
        - I/Os are queued in the multipath target when there are no valid paths and
          queue_if_no_path is set;
        - The queued I/Os above fail when suspend is issued without the
          DM_NOFLUSH_FLAG ioctl option.  I/Os spanning 2 multipath targets also
          fail.
      
      The following tests are for the normal code path of new pushback feature:
        - Queued I/Os in the multipath target are flushed from the target
          but don't return when suspend is issued with the DM_NOFLUSH_FLAG
          ioctl option;
        - The I/Os above are queued in the multipath target again when
          resume is issued without path recovery;
        - The I/Os above succeed when resume is issued after path recovery
          or table load;
        - Queued I/Os in the multipath target succeed when resume is issued
          with the DM_NOFLUSH_FLAG ioctl option after table load. I/Os
          spanning 2 multipath targets also succeed.
      
      The following tests are for the error paths of the new pushback feature:
        - When the bdget_disk() fails in dm_suspend(), the
          DMF_NOFLUSH_SUSPENDING flag is cleared and I/Os already queued to the
          pushback list are flushed properly.
        - When suspend with the DM_NOFLUSH_FLAG ioctl option is interrupted,
            o I/Os which had already been queued to the pushback list
              at the time don't return, and are re-issued at resume time;
            o I/Os which hadn't been returned at the time return with EIO.
      Signed-off-by: NKiyoshi Ueda <k-ueda@ct.jp.nec.com>
      Signed-off-by: NJun'ichi Nomura <j-nomura@ce.jp.nec.com>
      Signed-off-by: NAlasdair G Kergon <agk@redhat.com>
      Cc: dm-devel@redhat.com
      Signed-off-by: NAndrew Morton <akpm@osdl.org>
      Signed-off-by: NLinus Torvalds <torvalds@osdl.org>
      2e93ccc1
    • K
      [PATCH] dm: ioctl: add noflush suspend · 81fdb096
      Kiyoshi Ueda 提交于
      Provide a dm ioctl option to request noflush suspending.  (See next patch for
      what this is for.) As the interface is extended, the version number is
      incremented.
      
      Other than accepting the new option through the interface, There is no change
      to existing behaviour.
      
      Test results:
      Confirmed the option is given from user-space correctly.
      Signed-off-by: NKiyoshi Ueda <k-ueda@ct.jp.nec.com>
      Signed-off-by: NJun'ichi Nomura <j-nomura@ce.jp.nec.com>
      Signed-off-by: NAlasdair G Kergon <agk@redhat.com>
      Cc: dm-devel@redhat.com
      Signed-off-by: NAndrew Morton <akpm@osdl.org>
      Signed-off-by: NLinus Torvalds <torvalds@osdl.org>
      81fdb096
    • K
      [PATCH] dm: map and endio symbolic return codes · d2a7ad29
      Kiyoshi Ueda 提交于
      Update existing targets to use the new symbols for return values from target
      map and end_io functions.
      
      There is no effect on behaviour.
      
      Test results:
      Done build test without errors.
      Signed-off-by: NKiyoshi Ueda <k-ueda@ct.jp.nec.com>
      Signed-off-by: NJun'ichi Nomura <j-nomura@ce.jp.nec.com>
      Signed-off-by: NAlasdair G Kergon <agk@redhat.com>
      Cc: dm-devel@redhat.com
      Signed-off-by: NAndrew Morton <akpm@osdl.org>
      Signed-off-by: NLinus Torvalds <torvalds@osdl.org>
      d2a7ad29