Development discussion of WireGuard
 help / color / mirror / Atom feed
* Pre 5.1-rc1 build issue
@ 2019-03-13 18:38 Bruno Wolff III
  2019-03-13 18:52 ` Bruno Wolff III
  0 siblings, 1 reply; 4+ messages in thread
From: Bruno Wolff III @ 2019-03-13 18:38 UTC (permalink / raw)
  To: wireguard

Commit 
 https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=d2c5c103b1337f590b7edf1509a6e294bdf22402
removed net/netfilter/nf_nat_core.h, which breaks wireguard builds for 
5.1.0-0.rc0.git5.2.fc31.x86_64.

It sounds like it isn't needed any more from the commit message.
_______________________________________________
WireGuard mailing list
WireGuard@lists.zx2c4.com
https://lists.zx2c4.com/mailman/listinfo/wireguard

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Pre 5.1-rc1 build issue
  2019-03-13 18:38 Pre 5.1-rc1 build issue Bruno Wolff III
@ 2019-03-13 18:52 ` Bruno Wolff III
  2019-03-13 20:44   ` Jason A. Donenfeld
  0 siblings, 1 reply; 4+ messages in thread
From: Bruno Wolff III @ 2019-03-13 18:52 UTC (permalink / raw)
  To: wireguard

On Wed, Mar 13, 2019 at 13:38:18 -0500,
  Bruno Wolff III <bruno@wolff.to> wrote:
>Commit https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=d2c5c103b1337f590b7edf1509a6e294bdf22402
>removed net/netfilter/nf_nat_core.h, which breaks wireguard builds for 
>5.1.0-0.rc0.git5.2.fc31.x86_64.
>
>It sounds like it isn't needed any more from the commit message.

There are also some rcu changes that I just guessed on fixes for. I think 
this change was covered in Paul McKenney's LCA talk in January.

I got something that seems to be working. So I'm good until the real fixes 
are ready.
_______________________________________________
WireGuard mailing list
WireGuard@lists.zx2c4.com
https://lists.zx2c4.com/mailman/listinfo/wireguard

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Pre 5.1-rc1 build issue
  2019-03-13 18:52 ` Bruno Wolff III
@ 2019-03-13 20:44   ` Jason A. Donenfeld
  2019-03-14  4:08     ` Bruno Wolff III
  0 siblings, 1 reply; 4+ messages in thread
From: Jason A. Donenfeld @ 2019-03-13 20:44 UTC (permalink / raw)
  To: Bruno Wolff III; +Cc: WireGuard mailing list

On Wed, Mar 13, 2019 at 12:53 PM Bruno Wolff III <bruno@wolff.to> wrote:
> I got something that seems to be working. So I'm good until the real fixes
> are ready.

Perhaps your [fake?!] fixes could become the real fixes if you ran
`git send-email` to the mailing list?

Jason
_______________________________________________
WireGuard mailing list
WireGuard@lists.zx2c4.com
https://lists.zx2c4.com/mailman/listinfo/wireguard

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Pre 5.1-rc1 build issue
  2019-03-13 20:44   ` Jason A. Donenfeld
@ 2019-03-14  4:08     ` Bruno Wolff III
  0 siblings, 0 replies; 4+ messages in thread
From: Bruno Wolff III @ 2019-03-14  4:08 UTC (permalink / raw)
  To: Jason A. Donenfeld; +Cc: WireGuard mailing list

On Wed, Mar 13, 2019 at 14:44:33 -0600,
  "Jason A. Donenfeld" <Jason@zx2c4.com> wrote:
>On Wed, Mar 13, 2019 at 12:53 PM Bruno Wolff III <bruno@wolff.to> wrote:
>> I got something that seems to be working. So I'm good until the real fixes
>> are ready.
>
>Perhaps your [fake?!] fixes could become the real fixes if you ran
>`git send-email` to the mailing list?

I'll look at what I did to make sure I didn't make an unwanted rcu change. 
From Paul's talk it looks like there is a bit of a merger of a couple 
of rcu types to keep developers from getting confused and I think the 
unknown functions (that used to be known) should now call the non _bh 
version.

For the network include file I didn't include it any more. The correct 
fix is probably an ifdef based on kernel version or something like 
that.

So it might be useful to look at the changes I did, but I would be wary 
of committing them as is. It'll take me maybe an hour to double check 
and try it again on another system.
_______________________________________________
WireGuard mailing list
WireGuard@lists.zx2c4.com
https://lists.zx2c4.com/mailman/listinfo/wireguard

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2019-03-14  4:09 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-13 18:38 Pre 5.1-rc1 build issue Bruno Wolff III
2019-03-13 18:52 ` Bruno Wolff III
2019-03-13 20:44   ` Jason A. Donenfeld
2019-03-14  4:08     ` Bruno Wolff III

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).