[B.A.T.M.A.N.] batman-adv stable patches: information request

[B.A.T.M.A.N.] batman-adv stable patches: information request

[Antonio Quartulli]

Hello Greg,

I'm Antonio Quartulli, a net/batman-adv kernel module contributor, and I'm
contacting you as you are the stable tree maintainer.

On Oct 25, 2011 I sent the following two patches to stable@kernel.org:

- [PATCH 1/2] batman-adv: fix tt_local_reset_flags() function
  which is 31901264511cf20c5ed33b8649a3ca9ce28df60b in David's net-next branch


- [PATCH 2/2] batman-adv: correctly set the data field in the TT_REPONSE packet
  which is 9d8523931f7f5eb8900077f0da0fbe6b8ad0010b in David's net-next branch

Up to now we didn't received any reply, either ACK or NACK, but I saw that
linux-3.1.1 has been released in the meanwhile. As I guessed, this release
doesn't contain our patches.

Did I miss something in the stable-submit procedure?
I'm sorry for any inconvenient but this is the first time that I send patches to
stable.



Best Regards,


-- 
Antonio Quartulli

..each of us alone is worth nothing..
Ernesto "Che" Guevara

Re: [B.A.T.M.A.N.] batman-adv stable patches: information request

[Greg KH]

On Mon, Nov 21, 2011 at 08:27:24PM +0100, Antonio Quartulli wrote:
> Hello Greg,
> 
> I'm Antonio Quartulli, a net/batman-adv kernel module contributor, and I'm
> contacting you as you are the stable tree maintainer.
> 
> On Oct 25, 2011 I sent the following two patches to stable@kernel.org:
> 
> - [PATCH 1/2] batman-adv: fix tt_local_reset_flags() function
>   which is 31901264511cf20c5ed33b8649a3ca9ce28df60b in David's net-next branch
> 
> 
> - [PATCH 2/2] batman-adv: correctly set the data field in the TT_REPONSE packet
>   which is 9d8523931f7f5eb8900077f0da0fbe6b8ad0010b in David's net-next branch
> 
> Up to now we didn't received any reply, either ACK or NACK, but I saw that
> linux-3.1.1 has been released in the meanwhile. As I guessed, this release
> doesn't contain our patches.
> 
> Did I miss something in the stable-submit procedure?
> I'm sorry for any inconvenient but this is the first time that I send patches to
> stable.

Ugh, no, you did everything right, it's just that at that point in time,
stable@kernel.org was down, as it still is, the new address is
stable@vger.kernel.org.

I'll queue these two patches up for the next release.  Just to verify,
these should only go to 3.1-stable, and not 3.0-stable as well?

Also, in the future, you can just add a simple:
	Cc: stable <stable@vger.kernel.org>
to the patch in the signed-off-by: area, and when it hits Linus's tree,
I'll automatically pick it up for the next stable release, you don't
have to do anything extra.

thanks,

greg k-h

Re: [B.A.T.M.A.N.] batman-adv stable patches: information request

[Antonio Quartulli]

Hello Greg,

On Mon, Nov 21, 2011 at 12:54:22 -0800, Greg KH wrote:
> On Mon, Nov 21, 2011 at 08:27:24PM +0100, Antonio Quartulli wrote:
> > Hello Greg,
> > 
> > I'm Antonio Quartulli, a net/batman-adv kernel module contributor, and I'm
> > contacting you as you are the stable tree maintainer.
> > 
> > On Oct 25, 2011 I sent the following two patches to stable@kernel.org:
> > 
> > - [PATCH 1/2] batman-adv: fix tt_local_reset_flags() function
> >   which is 31901264511cf20c5ed33b8649a3ca9ce28df60b in David's net-next branch
> > 
> > 
> > - [PATCH 2/2] batman-adv: correctly set the data field in the TT_REPONSE packet
> >   which is 9d8523931f7f5eb8900077f0da0fbe6b8ad0010b in David's net-next branch
> > 
> > Up to now we didn't received any reply, either ACK or NACK, but I saw that
> > linux-3.1.1 has been released in the meanwhile. As I guessed, this release
> > doesn't contain our patches.
> > 
> > Did I miss something in the stable-submit procedure?
> > I'm sorry for any inconvenient but this is the first time that I send patches to
> > stable.
> 
> Ugh, no, you did everything right, it's just that at that point in time,
> stable@kernel.org was down, as it still is, the new address is
> stable@vger.kernel.org.
> 

Ah ok, thank you for the advice!

At this point we have 3 patches more that we sent to the wrong address after the
two that I already mentioned. The patches are:

- [PATCH 1/3] batman-adv: remove references for global tt entries
  commit id in Dadiv's tree: 531027fcddbcf81c9937dd04f08a7e8f11fd47d2

- [PATCH 2/3] batman-adv: add sanity check when removing global tts
  commit id in Dadiv's tree: 6e8014947d6469df1566e9e253805557c5c0e4e0

- [PATCH 3/3] batman-adv: unify hash_entry field position in
  tt_local/global_entry
  commit id in Dadiv's tree: 93840ac40bb0d0f177ef8af74e64671be67e8c37

> I'll queue these two patches up for the next release.  Just to verify,
> these should only go to 3.1-stable, and not 3.0-stable as well?
> 

Exactly: the 5 patches are for 3.1-stable only.

> Also, in the future, you can just add a simple:
> 	Cc: stable <stable@vger.kernel.org>
> to the patch in the signed-off-by: area, and when it hits Linus's tree,
> I'll automatically pick it up for the next stable release, you don't
> have to do anything extra.
> 

This is what we usually do, but this time we decided to send these patches
*after* they got merged into David's tree. :)

Thank you very much!


-- 
Antonio Quartulli

..each of us alone is worth nothing..
Ernesto "Che" Guevara

Re: [B.A.T.M.A.N.] batman-adv stable patches: information request

[Greg KH]

On Mon, Nov 21, 2011 at 10:27:10PM +0100, Antonio Quartulli wrote:
> Hello Greg,
> 
> On Mon, Nov 21, 2011 at 12:54:22 -0800, Greg KH wrote:
> > On Mon, Nov 21, 2011 at 08:27:24PM +0100, Antonio Quartulli wrote:
> > > Hello Greg,
> > > 
> > > I'm Antonio Quartulli, a net/batman-adv kernel module contributor, and I'm
> > > contacting you as you are the stable tree maintainer.
> > > 
> > > On Oct 25, 2011 I sent the following two patches to stable@kernel.org:
> > > 
> > > - [PATCH 1/2] batman-adv: fix tt_local_reset_flags() function
> > >   which is 31901264511cf20c5ed33b8649a3ca9ce28df60b in David's net-next branch
> > > 
> > > 
> > > - [PATCH 2/2] batman-adv: correctly set the data field in the TT_REPONSE packet
> > >   which is 9d8523931f7f5eb8900077f0da0fbe6b8ad0010b in David's net-next branch
> > > 
> > > Up to now we didn't received any reply, either ACK or NACK, but I saw that
> > > linux-3.1.1 has been released in the meanwhile. As I guessed, this release
> > > doesn't contain our patches.
> > > 
> > > Did I miss something in the stable-submit procedure?
> > > I'm sorry for any inconvenient but this is the first time that I send patches to
> > > stable.
> > 
> > Ugh, no, you did everything right, it's just that at that point in time,
> > stable@kernel.org was down, as it still is, the new address is
> > stable@vger.kernel.org.
> > 
> 
> Ah ok, thank you for the advice!
> 
> At this point we have 3 patches more that we sent to the wrong address after the
> two that I already mentioned. The patches are:
> 
> - [PATCH 1/3] batman-adv: remove references for global tt entries
>   commit id in Dadiv's tree: 531027fcddbcf81c9937dd04f08a7e8f11fd47d2
> 
> - [PATCH 2/3] batman-adv: add sanity check when removing global tts
>   commit id in Dadiv's tree: 6e8014947d6469df1566e9e253805557c5c0e4e0
> 
> - [PATCH 3/3] batman-adv: unify hash_entry field position in
>   tt_local/global_entry
>   commit id in Dadiv's tree: 93840ac40bb0d0f177ef8af74e64671be67e8c37
> 
> > I'll queue these two patches up for the next release.  Just to verify,
> > these should only go to 3.1-stable, and not 3.0-stable as well?
> > 
> 
> Exactly: the 5 patches are for 3.1-stable only.

The 2/3 patch above did apply to the 3.0-stable tree, but I didn't
include it there, let me know if you want me to.

I've queued all of these up now, thanks,

greg k-h

Re: [B.A.T.M.A.N.] batman-adv stable patches: information request

[Greg KH]

On Tue, Nov 22, 2011 at 02:47:00PM -0800, Greg KH wrote:
> On Mon, Nov 21, 2011 at 10:27:10PM +0100, Antonio Quartulli wrote:
> > Hello Greg,
> > 
> > On Mon, Nov 21, 2011 at 12:54:22 -0800, Greg KH wrote:
> > > On Mon, Nov 21, 2011 at 08:27:24PM +0100, Antonio Quartulli wrote:
> > > > Hello Greg,
> > > > 
> > > > I'm Antonio Quartulli, a net/batman-adv kernel module contributor, and I'm
> > > > contacting you as you are the stable tree maintainer.
> > > > 
> > > > On Oct 25, 2011 I sent the following two patches to stable@kernel.org:
> > > > 
> > > > - [PATCH 1/2] batman-adv: fix tt_local_reset_flags() function
> > > >   which is 31901264511cf20c5ed33b8649a3ca9ce28df60b in David's net-next branch
> > > > 
> > > > 
> > > > - [PATCH 2/2] batman-adv: correctly set the data field in the TT_REPONSE packet
> > > >   which is 9d8523931f7f5eb8900077f0da0fbe6b8ad0010b in David's net-next branch
> > > > 
> > > > Up to now we didn't received any reply, either ACK or NACK, but I saw that
> > > > linux-3.1.1 has been released in the meanwhile. As I guessed, this release
> > > > doesn't contain our patches.
> > > > 
> > > > Did I miss something in the stable-submit procedure?
> > > > I'm sorry for any inconvenient but this is the first time that I send patches to
> > > > stable.
> > > 
> > > Ugh, no, you did everything right, it's just that at that point in time,
> > > stable@kernel.org was down, as it still is, the new address is
> > > stable@vger.kernel.org.
> > > 
> > 
> > Ah ok, thank you for the advice!
> > 
> > At this point we have 3 patches more that we sent to the wrong address after the
> > two that I already mentioned. The patches are:
> > 
> > - [PATCH 1/3] batman-adv: remove references for global tt entries
> >   commit id in Dadiv's tree: 531027fcddbcf81c9937dd04f08a7e8f11fd47d2
> > 
> > - [PATCH 2/3] batman-adv: add sanity check when removing global tts
> >   commit id in Dadiv's tree: 6e8014947d6469df1566e9e253805557c5c0e4e0
> > 
> > - [PATCH 3/3] batman-adv: unify hash_entry field position in
> >   tt_local/global_entry
> >   commit id in Dadiv's tree: 93840ac40bb0d0f177ef8af74e64671be67e8c37
> > 
> > > I'll queue these two patches up for the next release.  Just to verify,
> > > these should only go to 3.1-stable, and not 3.0-stable as well?
> > > 
> > 
> > Exactly: the 5 patches are for 3.1-stable only.
> 
> The 2/3 patch above did apply to the 3.0-stable tree, but I didn't
> include it there, let me know if you want me to.

Oh nevermind, it didn't apply properly, I should have looked more
carefully.

greg k-h