linux-kernel - Re: [PATCH][v3.19.y-ckt] IB/ipoib: change init sequence ordering

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 01 Jul 2015 13:42:28 -0400
From:	Doug Ledford <dledford@...hat.com>
To:	Joseph Salisbury <joseph.salisbury@...onical.com>,
	Kamal Mostafa <kamal.mostafa@...onical.com>
CC:	sean.hefty@...el.com, hal.rosenstock@...il.com,
	roland@...estorage.com, ogerlitz@...lanox.com, erezsh@...lanox.com,
	alex.estrin@...el.com, linux-rdma@...r.kernel.org,
	LKML <linux-kernel@...r.kernel.org>,
	"stable@...r.kernel.org" <stable@...r.kernel.org>,
	kamalh@...lanox.com
Subject: Re: [PATCH][v3.19.y-ckt] IB/ipoib: change init sequence ordering

On 07/01/2015 01:27 PM, Joseph Salisbury wrote:
> Hello,
> 
> Please consider including upstream commits
> be7aa663fc1d9156798f5af3c60e6df45e1fe5de and
> e135106fac9525352feb8e49077c8f46c3eaf288 in the next v3.19.y-ckt
> release.  They were included mainline as of v4.1-rc1.  These commits
> have been tested and confirmed to resolve
> http://bugs.launchpad.net/bugs/1467912 .
> 
> 
> commit be7aa663fc1d9156798f5af3c60e6df45e1fe5de
> Author: Doug Ledford <dledford@...hat.com>
> Date:   Sat Feb 21 19:27:00 2015 -0500
> 
>     IB/ipoib: change init sequence ordering
> 
> commit e135106fac9525352feb8e49077c8f46c3eaf288
> Author: Doug Ledford <dledford@...hat.com>
> Date:   Sat Feb 21 19:26:59 2015 -0500
> 
>     IB/ipoib: factor out ah flushing
> 
> The bug reporter states this bug is only happening with the 3.19 kernel,
> these commits may not be needed in other stable kernels.

I have no objection to these two patches going into a 3.19 stable
kernel.  They are both stand alone, safe patches.

However, there is no difference in the IPoIB code between 3.18 and 3.19,
and I didn't think this bug existed in 3.19 (my commit logs called the
work preparatory for upcoming patches).  If your testing does in fact
show that this is only a 3.19 bug, and 3.18 tests clear, then that means
the bug was introduced by some other change in the core kernel (probably
in how work queues are scheduled and we likely had a situation where our
code worked based upon how they used to be scheduled, but with the
change we now need explicit flushes).

In any case, ack from me for these two patches.

-- 
Doug Ledford <dledford@...hat.com>
              GPG KeyID: 0E572FDD

Download attachment "signature.asc" of type "application/pgp-signature" (885 bytes)