1. 24 2月, 2016 11 次提交
  2. 10 11月, 2015 1 次提交
  3. 03 11月, 2015 1 次提交
  4. 29 10月, 2015 9 次提交
  5. 01 6月, 2015 1 次提交
    • S
      megaraid_sas : Modify return value of megasas_issue_blocked_cmd() and... · 2be2a988
      Sumit.Saxena@avagotech.com 提交于
      megaraid_sas : Modify return value of megasas_issue_blocked_cmd() and wait_and_poll() to consider command status returned by firmware
      
      This patch is rebased on top of recently sent 18 patches(submitted by me) for
      megaraid_sas driver.
      
      Change the return value of wait_and_poll() and megsas_issue_blocked_cmd()
      based on MFI_STAT returned by firmware for that command. Earlier driver always
      send return type based on command completion (but never check MFI_STAT_OK for
      that command), so even if command is failed by firmware still driver will
      return SUCCESS status from these functions wait_and_poll() and
      megsas_issue_blocked_cmd() and if caller of these functions does not check
      command status (MFI_STAT), then it may endup using invalid data returned in
      DMA buffers(one of the example is megasas_ld_list_query DCMD). Best thing to
      avoid this type of issue is do error handling and set proper return type from
      caller function wait_and_poll() and megsas_issue_blocked_cmd().
      
      The change proposed in this patch will fix the regression introduced in patch-
      "90dc9d98 megaraid_sas : MFI MPT linked list corruption fix" inside function
      megasas_ld_list_query().  Prior to this MFI MPT linked list corruption fix
      patch, megasas_ld_list_query() function used to check DCMD status(returned by
      firmware) but with this linked list corruption fix patch, DCMD status will not
      be checked inside function megasas_ld_list_query() and introduced this issue
      of wrong data being used by function megasas_ld_list_query().
      
      Cc: <stable@vger.kernel.org>
      Signed-off-by: NKashyap Desai <kashyap.desai@avagotech.com>
      Signed-off-by: NSumit Saxena <sumit.saxena@avagotech.com>
      Reviewed-by: NTomas Henzl <thenzl@redhat.com>
      Signed-off-by: NJames Bottomley <JBottomley@Odin.com>
      2be2a988
  6. 25 5月, 2015 7 次提交
  7. 09 1月, 2015 3 次提交
  8. 24 11月, 2014 3 次提交
  9. 17 9月, 2014 4 次提交
    • S
      megaraid_sas : Driver version update · 0d9d8b9f
      Sumit.Saxena@avagotech.com 提交于
      Resending the patch. Addressed the review comments by Tomas Henzl.
      
      Driver version upgrade patch.
      Signed-off-by: NSumit Saxena <sumit.saxena@avagotech.com>
      Signed-off-by: NKashyap Desai <kashyap.desai@avagotech.com>
      Reviewed-by: NTomas Henzl <thenzl@redhat.com>
      Signed-off-by: NChristoph Hellwig <hch@lst.de>
      0d9d8b9f
    • S
      megaraid_sas : MFI MPT linked list corruption fix · 90dc9d98
      Sumit.Saxena@avagotech.com 提交于
      Resending the patch. Addressed the review comments from Tomas Henzl.
      Added comment for to-do work.
      
      Problem statement:
      MFI link list in megaraid_sas driver is used from mfi-mpt pass-through commands.
      This list can be corrupted due to many possible race conditions in driver and
      eventually we may see kernel panic.
      
      One example -
      MFI frame is freed from calling process as driver send command via polling method and interrupt
      for that command comes after driver free mfi frame (actually even after some other context reuse
      the mfi frame). When driver receive MPT frame in ISR, driver will be using the index of MFI and
      access that MFI frame and finally in-used MFI frame’s list will be corrupted.
      
      High level description of new solution -
      Free MFI and MPT command from same context.
      Free both the command either from process (from where mfi-mpt pass-through was called) or from
      ISR context. Do not split freeing of MFI and MPT, because it creates the race condition which
      will do MFI/MPT list corruption.
      
      Renamed the cmd_pool_lock which is used in instance as well as fusion with below name.
      mfi_pool_lock and mpt_pool_lock to add more code readability.
      Signed-off-by: NSumit Saxena <sumit.saxena@avagotech.com>
      Signed-off-by: NKashyap Desai <kashyap.desai@avagotech.com>
      Reviewed-by: NTomas Henzl <thenzl@redhat.com>
      Signed-off-by: NChristoph Hellwig <hch@lst.de>
      90dc9d98
    • S
      megaraid_sas : N-drive primary raid level 1 load balancing · d2552ebe
      Sumit.Saxena@avagotech.com 提交于
      Resending the patch. Addressed the review comments from Tomas Henzl.
      
      Current driver does fast path read load balancing between arm and mirror disk
      for two Drive Raid-1 configuration only.
      
      Now, Driver support fast path read load balancing for all (any number of disk) Raid-1 configuration.
      Signed-off-by: NSumit Saxena <sumit.saxena@avagotech.com>
      Signed-off-by: NKashyap Desai <kashyap.desai@avagotech.com>
      Reviewed-by: NTomas Henzl <thenzl@redhat.com>
      Signed-off-by: NChristoph Hellwig <hch@lst.de>
      d2552ebe
    • S
      megaraid_sas : Extended VD support · 51087a86
      Sumit.Saxena@avagotech.com 提交于
      Resending the patch. Addressed the review comments from Tomas Henzl.
      reserved1 field(part of union) of Raid map struct was not required so it is removed.
      
      Current MegaRAID firmware and hence the driver only supported 64VDs.
      E.g: If the user wants to create more than 64VD on a controller,
          it is not possible on current firmware/driver.
      
      New feature and requirement to support upto 256VD, firmware/driver/apps need changes.
      In addition to that there must be a backward compatibility of the new driver with the
      older firmware and vice versa.
      
      RAID map is the interface between Driver and FW to fetch all required
      fields(attributes) for each Virtual Drives.
      In the earlier design driver was using the FW copy of RAID map where as
      in the new design the Driver will keep the RAID map copy of its own; on which
      it will operate for any raid map access in fast path.
      
      Local driver raid map copy will provide ease of access through out the code
      and provide generic interface for future FW raid map changes.
      
      For the backward compatibility driver will notify FW that it supports 256VD
      to the FW in driver capability field.
      Based on the controller properly returned by the FW, the Driver will know
      whether it supports 256VD or not and will copy the RAID map accordingly.
      
      At any given time, driver will always have old or new Raid map.
      So with this changes, driver can also work in host lock less mode. Please
      see next patch which enable host lock less mode for megaraid_sas driver.
      Signed-off-by: NSumit Saxena <sumit.saxena@avagotech.com>
      Signed-off-by: NKashyap Desai <kashyap.desai@avagotech.com>
      Reviewed-by: NTomas Henzl <thenzl@redhat.com>
      Signed-off-by: NChristoph Hellwig <hch@lst.de>
      51087a86