Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752912AbbG2HcG (ORCPT ); Wed, 29 Jul 2015 03:32:06 -0400 Received: from mail-ob0-f179.google.com ([209.85.214.179]:33918 "EHLO mail-ob0-f179.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751101AbbG2HcE (ORCPT ); Wed, 29 Jul 2015 03:32:04 -0400 MIME-Version: 1.0 In-Reply-To: <1437954348-11859-1-git-send-email-vivien.didelot@savoirfairelinux.com> References: <1437954348-11859-1-git-send-email-vivien.didelot@savoirfairelinux.com> From: Scott Feldman Date: Wed, 29 Jul 2015 00:31:44 -0700 Message-ID: Subject: Re: [PATCH] net: switchdev: restrict vid range abstraction To: Vivien Didelot Cc: Netdev , "David S. Miller" , Jiri Pirko , Florian Fainelli , "linux-kernel@vger.kernel.org" , kernel Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1413 Lines: 30 On Sun, Jul 26, 2015 at 4:45 PM, Vivien Didelot wrote: > This patch replaces the vid_begin and vid_end members of the > switchdev_obj_vlan structure for a single vid member. This way, the VID > range abstraction is restricted to switchdev, not exposed to drivers. > > The main benefice to do so is to allow the prepare phase to be called > for each VID, not only once for the whole range. > > For example, when adding VLANs 2-5, a switch chip may not be able to add > hardware VLAN 2 due to some bridge restriction (thus must return > -EOPNOTSUPP), while VLAN 3-5 are allowed. Hi Vivien, Since the netlink request (for example vlan add) includes the range, I'm not seeing how we can response with success for the satisfied vlans in the range, and also respond with an error for the unsatisfied vlans in the range. In other words, from the netlink msgs perspective, we need to treat a vlan range as all-or-nothing. So in your example, if hw can't add vlan 2, we fail the entire request to add range 2-5. This is where the prepare phase checks to make sure the entire request can be satisfied before committing to hw. -scott -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/