RE: [PATCH 1/2] net/hyperv: Fix data corruption inrndis_filter_receive()

From: Haiyang Zhang
Date: Mon Mar 12 2012 - 11:39:51 EST




> -----Original Message-----
> From: David Miller [mailto:davem@xxxxxxxxxxxxx]
> Sent: Sunday, March 11, 2012 6:51 PM
> To: Haiyang Zhang
> Cc: KY Srinivasan; netdev@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
> devel@xxxxxxxxxxxxxxxxxxxxxx
> Subject: Re: [PATCH 1/2] net/hyperv: Fix data corruption in
> rndis_filter_receive()
>
>
> It is very tiring explaining how to submit patches properly, and here you've
> made a poor submission once again.
>
> You haven't indicated what tree you want these changes applied to.
>
> You also mixed bug fixes and feature patches into the same patch set.
>
> So the patch set isn't appropriate for the 'net' tree, because only the most
> critical bug fixes can go there now.
>
> And this patch set is also not appropriate for the 'net-next' tree because I can
> only presume you want the important bug fix propagated upstream as fast as
> possible but it cannot go in with the feature patch since the feature patch is
> absolutely not appropriate for the 'net' tree right now.
>
> I really don't see why it's such a hard thing to simply say where you want
> patches applied when you submit them. Do you want me to simply guess?
> How else am I supposed to figure out what you want?

I want both patches go to the 'net-next' tree. The patch [1/2] is required by the
vlan trunking feature [2/2]. So they should go together.

The bug in rndis_filter_receive() does NOT happen until we start to use
'per-packet-data' by the vlan trunking feature, so the bug fix [1/2] isn't urgent
to be applied into 'net' tree.

And, thanks for the reminder, I will specify the target tree in future submissions.

Thanks,
- Haiyang


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