[ovs-dev] [PATCH v2 0/8] Implement OFPMP_TABLE_FEATURES GET

Alexander Wu alexander.wu at huawei.com
Tue Nov 12 07:19:12 UTC 2013


On 12/11/2013 12:34, Ben Pfaff wrote:
> On Tue, Nov 12, 2013 at 11:03:45AM +0800, Alexander Wu wrote:
>> On 12/11/2013 07:55, Ben Pfaff wrote:
>>> These patches seem to be whitespace damaged.  Perhaps you could make
>>> them available through a branch somewhere?
>>>
>>>
>>
>> Sorry for the whitespace. It's caused by direct copy of PDF file.
>
> I am confused.  How was a PDF used to prepare these patches?
>

I copied 5 lines from OpenFlow 1.3.2 spec as [0/8]'s description, they
describe that OFPMP_TABLE_FEATURES consists of two parts:

   Request without body: GET the maximum features of tables.
   Request within body: SET the real features of tables.

My purpose of citing the para in spec is to avoid ambiguity.

>> I've sent a correct version after the patches. You can find this one
>> in emails:
>>    [PATCH v2 0/8] Implement OFPMP_TABLE_FEATURES GET - Fix fi error
>>
>> And you'll see the content below:
>
> I think you're saying that you sent another version.  I will look.
>
>

Yes. Note that the titles are different.

  [PATCH v2 0/8] Implement OFPMP_TABLE_FEATURES GET
   change to
  [PATCH v2 0/8] Implement OFPMP_TABLE_FEATURES GET - Fix fi error

Best regards,
Alexander Wu




More information about the dev mailing list