[lkml]   [2021]   [Jul]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH 0/3] gve: Fixes and clean-up
Le 01/07/2021 à 18:20, Jeroen de Borst a écrit :
> On Wed, Jun 30, 2021 at 10:58 PM Christophe JAILLET
> <> wrote:
>> This serie is part of the effort to axe the wrappers in
>> include/linux/pci-dma-compat.h
>> While looking at it, I spotted:
>> - a resource leak in an error handling path (patch 1)
>> - an error code that could be propagated. (patch 2)
>> This patch could be ignored. It's only goal is to be more consistent
>> with other drivers.
>> These 2 paches are not related to the 'pci-dma-compat.h' stuff, which can
>> be found in patch 3.
>> Christophe JAILLET (3):
>> gve: Fix an error handling path in 'gve_probe()'
>> gve: Propagate error codes to caller
>> gve: Simplify code and axe the use of a deprecated API
> Thanks for these patches.
> Can split this into 2 patch series;


> one for net (with the first 2
> patches) and one for net-next (with the cleanup one)?

I've never worked with net and net-next directly.
If just adding net and net-next after [PATCH] in the subject of the
mail, yes, I can do it if it helps.

BTW, I gave a look at
The patch 1/3 is marked as failed because "1 blamed authors not CCed:; 1 maintainers not CCed:"

This author/blame was not spotted by Is it something
I should worry about?

> Also the label in the first patch should probably read
> 'abort_with_gve_init' instead of 'abort_with_vge_init'.

Good catch. Sorry about that.

> Jeroen


 \ /
  Last update: 2021-07-01 19:43    [W:0.075 / U:0.324 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site