Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1537949

Re: [net-next][PATCH v2 18/18] RDS: IB: add missing connection cache usage info

From Santosh Shilimkar <santosh.shilimkar@oracle.com>
Newsgroups linux.kernel
Subject Re: [net-next][PATCH v2 18/18] RDS: IB: add missing connection cache usage info
Date 2016-12-07 18:30 +0100
Message-ID <sLOcy-31p-19@gated-at.bofh.it> (permalink)
References <sLBIl-3hz-11@gated-at.bofh.it> <sLMNs-1Yv-25@gated-at.bofh.it> <sLNzQ-2yE-29@gated-at.bofh.it> <sLNTb-2UC-19@gated-at.bofh.it>
Organization Oracle Corporation

Show all headers | View raw


On 12/7/2016 9:05 AM, David Miller wrote:
> From: Santosh Shilimkar <santosh.shilimkar@oracle.com>
> Date: Wed, 7 Dec 2016 08:44:04 -0800
>
>> On 12/7/2016 7:55 AM, David Miller wrote:
>>> From: Santosh Shilimkar <santosh.shilimkar@oracle.com>
>>> Date: Tue,  6 Dec 2016 20:01:56 -0800
>>>
>>> What level of compatability exists here?  If we run an old tool on a
>>> new
>>> kernel, or a new tool on an old kernel, does it work properly?
>>>
>> Tools repo carries a copy of the header and thats how the old tool and
>> new tools have been running with older/newer kernels. There are few
>> more
>> bits left before I can start using directly kernel header for newer
>> tools.
>>
>> Moreover this particular parameter is only used for verbose mode which
>> isn't used in default options.
>
> That doesn't really answer my question, I think.
>
Sorry for not being clear.

> Are you saying that one is required to run old tools on old kernels,
> and new tools on new kernels, and that's how you have this setup in
> your repo?
>
No.

> If so, that really isn't acceptable.  Both old and new tools must work
> with all kernel versions.
>
Older version of tools works on either kernel versions. Older tools
don't parse this additional info since its copy of header not
carrying some of these extra verbose fields. Newer/Updated tools which
can parse this extra info in needs newer or an updated kernel which
supports and populates these fields.

As mentioned, this particular option used only in verbose mode so
am ok to drop this change if its still a concern.

Regards,
Santosh

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[net-next][PATCH v2 18/18] RDS: IB: add missing connection cache usage info Santosh Shilimkar <santosh.shilimkar@oracle.com> - 2016-12-07 05:10 +0100
  Re: [net-next][PATCH v2 18/18] RDS: IB: add missing connection  cache usage info David Miller <davem@davemloft.net> - 2016-12-07 17:00 +0100
    Re: [net-next][PATCH v2 18/18] RDS: IB: add missing connection cache  usage info Santosh Shilimkar <santosh.shilimkar@oracle.com> - 2016-12-07 17:50 +0100
      Re: [net-next][PATCH v2 18/18] RDS: IB: add missing connection  cache usage info David Miller <davem@davemloft.net> - 2016-12-07 18:10 +0100
        Re: [net-next][PATCH v2 18/18] RDS: IB: add missing connection cache  usage info Santosh Shilimkar <santosh.shilimkar@oracle.com> - 2016-12-07 18:30 +0100
          Re: [net-next][PATCH v2 18/18] RDS: IB: add missing connection  cache usage info David Miller <davem@davemloft.net> - 2016-12-07 18:40 +0100
            Re: [net-next][PATCH v2 18/18] RDS: IB: add missing connection cache  usage info Santosh Shilimkar <santosh.shilimkar@oracle.com> - 2016-12-07 21:50 +0100

csiph-web