mbox

pull request net: 2013-08-17b

Message ID 1376737486-1863-1-git-send-email-ordex@autistici.org
State Superseded, archived
Delegated to: David Miller
Headers show

Pull-request

git://git.open-mesh.org/linux-merge.git tags/batman-adv-fix-for-davem

Message

Antonio Quartulli Aug. 17, 2013, 11:04 a.m. UTC
Hello David,

here is the correct pull request for net/linux-3.11.

As described earlier this patchset contains a change which properly ends a
seq_read cycle and another change that takes care of aborting TX operations if
the skb preparation should faile.

Please pull or let me know if there is any problem.

(about merge of net into net-next?)

Thanks a lot!
	Antonio


The following changes since commit 0f7dd1aa8f959216f1faa71513b9d3c1a9065e5a:

  Merge tag 'clk-fixes-for-linus' of git://git.linaro.org/people/mturquette/linux (2013-08-16 10:00:18 -0700)

are available in the git repository at:


  git://git.open-mesh.org/linux-merge.git tags/batman-adv-fix-for-davem

for you to fetch changes up to d823c33c0717cffea2dc73180ef780bf626908c7:

  batman-adv: check return type of unicast packet preparations (2013-08-17 12:52:27 +0200)

----------------------------------------------------------------
Included change:
- Add a missing skb_abort_seq_read() to properly close seq_read session
- Check if the skb has been correctly prepared before going on

----------------------------------------------------------------
Antonio Quartulli (1):
      batman-adv: properly end seq_read session

Linus Lüssing (1):
      batman-adv: check return type of unicast packet preparations

 net/batman-adv/main.c    |  1 +
 net/batman-adv/unicast.c | 10 +++++++---
 2 files changed, 8 insertions(+), 3 deletions(-)

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

Comments

Antonio Quartulli Aug. 17, 2013, 5:31 p.m. UTC | #1
On Sat, Aug 17, 2013 at 01:04:44PM +0200, Antonio Quartulli wrote:
> Hello David,
> 
> here is the correct pull request for net/linux-3.11.
> 
> As described earlier this patchset contains a change which properly ends a
> seq_read cycle and another change that takes care of aborting TX operations if
> the skb preparation should faile.

Hi David,

sorry for the noise but after analysing the code once more we realised that
skb_abort_seq_read() is not needed anymore for >=linux-3.11.

Therefore one of the patches in this pull request is not needed.

I will send a third pull request with the correct patch only.


Thanks,
	Antonio
David Miller Aug. 20, 2013, 11:56 p.m. UTC | #2
From: Antonio Quartulli <ordex@autistici.org>
Date: Sat, 17 Aug 2013 13:04:44 +0200

>   git://git.open-mesh.org/linux-merge.git tags/batman-adv-fix-for-davem
 ...
> Antonio Quartulli (1):
>       batman-adv: properly end seq_read session
> 
> Linus Lüssing (1):
>       batman-adv: check return type of unicast packet preparations

The change by Linus is not in that tree.

When I pulled I only got the seq_rea fix.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
David Miller Aug. 20, 2013, 11:58 p.m. UTC | #3
From: David Miller <davem@davemloft.net>
Date: Tue, 20 Aug 2013 16:56:49 -0700 (PDT)

> From: Antonio Quartulli <ordex@autistici.org>
> Date: Sat, 17 Aug 2013 13:04:44 +0200
> 
>>   git://git.open-mesh.org/linux-merge.git tags/batman-adv-fix-for-davem
>  ...
>> Antonio Quartulli (1):
>>       batman-adv: properly end seq_read session
>> 
>> Linus Lüssing (1):
>>       batman-adv: check return type of unicast packet preparations
> 
> The change by Linus is not in that tree.
> 
> When I pulled I only got the seq_rea fix.

Nevermind, in fact, I see that only Linus's unicast fix was in there,
which is what you intended.

Sorry for the false alarm.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html