On 01/23/2018 10:07 AM, Cong Wang wrote: > On Mon, Jan 22, 2018 at 2:44 PM, Cong Wang <xiyou.wangc...@gmail.com> wrote: >> >> Good point, but netdev_for_each_tx_queue() doesn't handle errors >> in current code base, it is not trivial to make it returning int to reflect >> errors, since we have to rollback a partial failure too. > > I think for now we can just check for errors but not rollback, > at least attach_default_qdiscs() does not rollback either. > And I will add a comment saying we need to do it in future. >
Sorry I delayed responding was thinking about some solutions. Unrolling is problematic because we may fail again on the unroll :( so I guess for this to work we would need a two phase operation with a prep and commit phase. So agree for now checking for errors seems reasonable and is at least better than without the patches. And as you say another patch set can cleanup the bigger issue of rollback. Thanks, .John
