Re: [PATCH for kernel v4.6] IB/srpt: Revert "Convert to percpu_ida tag allocation"

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 04/02/16 20:56, Nicholas A. Bellinger wrote:
Also, I don't know why none of your ib_srpt patches ever make it to
target-devel, but can you please stop trying to push target driver
changes upstream without first notifying target-devel..?

Beyond that, are you going to send an bug-fix to address this regression
in v4.6 code..?  If not, I'll add this to the queue and just do it
myself.

Hello Nic,

Everyone who is interested in Linux and RDMA and also in the ib_srpt target driver is already reading the linux-rdma mailing list as far as I know. But I can CC target-devel for future ib_srpt patch submissions if you prefer this.

The kernel crash introduced by "Convert to percpu_ida tag allocation" occurs as soon as the first SCSI command is received by the ib_srpt target driver. This means that that patch had not been tested at all. It should have been mentioned in the description of that patch that that patch was untested. And untested patches should not be sent to Linus.

Regarding when and how to fix this regression: aligning the ib_srpt target driver with what other target drivers are doing would introduce new code. New code should be introduced during the merge window. Since the v4.6 merge window has been closed I propose to send in the revert for v4.6-rc<n> and to send in the new version of this patch for the v4.7 merge window.

Bart.


--
To unsubscribe from this list: send the line "unsubscribe target-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux SCSI]     [Kernel Newbies]     [Linux SCSI Target Infrastructure]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Device Mapper]

  Powered by Linux