Re: [PATCH] net: detect recursive dev_queue_xmit() on RT properly

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

 



This is already fixed by 0f27b2b421a4 ("net: move xmit_recursion to
per-task variable on -RT"), which is in the various stable-rts too.
Apologies for the noise.

>> This also includes "543a589a00e8 net: xmit lock owner cleanup" from
>> v2.6.33.9-rt31 by mingo and tglx, which was just some cleanups to the
>> original patch which make porting to to current upstream easier.
>>
>> Signed-off-by: Mark Beauchemin <mark.beauchemin@xxxxxxxxxxxxxxx>
>> [ ported to latest upstream ]
>> Signed-off-by: Ingo Molnar <mingo@xxxxxxx>
>> Signed-off-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
>> [ ported to 4.8 and used struct task_struct instead of void ]
>> Signed-off-by: Brian Silverman <brian@xxxxxxxxxxxxxxxx>
>
> The signed-off-by chain suggest that Mark passed the patch to Ingo,
> Thomas and to Brian. I doubt that. I can't resolve the sha1 id mentioned
> here.

It's from the rt-history repository. The original commit in the
rt-history repository has that Signed-off-by chain except I added me
at the bottom. I guess it would make more sense to remove the other
people who just cherry-picked it if I run into this again.

> Is this still broken as of v4.8-RT? I remember fixing something similar
> with bridge devices.

Indeed you did it. It was cherry-picked to v4.1.19-rt22 too, which
matches the 4.1 kernel I originally found this on. I thought I checked
for the problem one or both of those, but apparently not... My bad.
--
To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [RT Stable]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]

  Powered by Linux