Re: [PATCH v3 0/2] Eliminate race between LOCK and FREE_STATEID

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

 



> On Aug 8, 2016, at 4:06 PM, Chuck Lever <chuck.lever@xxxxxxxxxx> wrote:
> 
>> 
>> On Aug 8, 2016, at 3:59 PM, bfields@xxxxxxxxxxxx wrote:
>> 
>> On Mon, Aug 08, 2016 at 02:59:35PM -0400, Chuck Lever wrote:
>>> This series passes light testing in my lab. If it looks good I will
>>> pass it along to Alexey to confirm it closes the race.
>>> 
>>> To aid distributors and stable kernel maintainers, wondering if a
>>> Fixes: tag should be added. Alexey first observed this issue in v4.1
>>> kernels (UEK4). But looks like the race could have been introduced
>>> as early as v3.17. Maybe this one?
>> 
>> The other reason we didn't see this till now might be client-side
>> changes (maybe b4019c0e219b "NFSv4.1: Allow parallel LOCK/LOCKU
>> calls"?).  (Not trying to dodge responsibility for a server-side bug
>> here, but that might still be useful information for the changelog (not
>> the Fixes: line) if it's correct.)
> 
> I asked Alexey to test as far back as v3.19, where I think the
> LOCK parallelism was added. I need to get some clarification of
> his test results; one set of test runs reproduced the race, and
> a second set of test runs did not.

Alexey reports he tried reproducing this with an Oracle Linux UEK3
client, which is roughly the same as v3.8 stable running on RHEL 6.
He says he was not able to reproduce with that configuration.

However, he was able to reproduce with a stock RHEL 7.2 client, and
with a v3.19 stable kernel running on a RHEL 7 client.


--
Chuck Lever



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



[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux