Re: What's the staging review and acceptance process?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2010-04-21 at 20:45 -0700, Greg KH wrote:
> That caused the huge backlog staring at
> me right now.

Which likely discouraged the new contributors who
submitted stuff still in that backlog.

> I just went through 100 patches, and only 40 of them
> were "valid" and able to be applied, so it is a high rejection rate
> which requires a lot of attention to be paid to them.
> 
> > What is your current review/notify/accept/reject workflow?
> 
> Like it's always been:
>   - patch comes in
>   - I get around to reviewing it
>   - if valid, I apply and you get an email
>   - if invalid, I reject and say why in email

Unfortunately, your process has been opaque until you
personally handle it.

Does the driverdev list or any list always receive a copy
or the feedback?

> > Perhaps a patchwork queue for staging might help track these
> > patches and with more feedback or reviewers, get them in
> > shape to be applied.

> patchwork doesn't work well for my patch flow, but maybe that is because
> I haven't spent enough time with it.  Right now I have all the patches,
> it's just a matter of getting through them.

Maybe you should get some advice on using patchwork use
from somebody like David Miller, who gets rather more
patches for networking than staging gets.  The patchwork
queue for networking always seems managed and it can use
delegation, which your process doesn't seem capable of
doing.  You're a voluntary bottleneck for staging, I
think you either need to find personal cycles or find
some other suckers willing to volunteer who'd make up
more overall cycles.

cheers, Joe

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux