linux-kernel - Re: [PATCH V2] lib/scatterlist: Add SG_CHAIN and SG

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Mon, 19 Feb 2018 14:25:44 +0200
From:   Andy Shevchenko <andy.shevchenko@...il.com>
To:     Tvrtko Ursulin <tursulin@...ulin.net>
Cc:     Anshuman Khandual <khandual@...ux.vnet.ibm.com>,
        kbuild test robot <lkp@...el.com>, kbuild-all@...org,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Bart Van Assche <bart.vanassche@....com>,
        Johannes Thumshirn <jthumshirn@...e.de>,
        Jens Axboe <axboe@...nel.dk>,
        Chris Wilson <chris@...is-wilson.co.uk>,
        Tvrtko Ursulin <tvrtko.ursulin@...el.com>
Subject: Re: [PATCH V2] lib/scatterlist: Add SG_CHAIN and SG_END macros for
 LSB encodings

On Mon, Feb 19, 2018 at 12:10 PM, Tvrtko Ursulin <tursulin@...ulin.net> wrote:
> On 19/02/18 04:12, Anshuman Khandual wrote:
>> On 02/18/2018 12:58 AM, kbuild test robot wrote:


>>>      #define SG_END   0x02
>>>          In file included from include/linux/blkdev.h:28:0,
>>>                      from drivers/staging/rts5208/rtsx.c:23:
>>>     include/linux/scatterlist.h:69:0: note: this is the location of the
>>> previous definition
>>>      #define SG_END  0x02UL
>>>
>>> vim +/SG_END +343 drivers/staging/rts5208/rtsx_chip.h
>>
>>
>> SG_END is already defined in a driver.

In some driver which is by the way staging one.

> Hence shall we change back
>> 'SG_END' definition as 'SG_EMARK' or any other suggestions ?
>
>
> Maybe SG_LAST? Going by existence of sg_is_chain and sg_is_last.

The mentioned driver should be cleaned up as well (and perhaps to start with).
It allows us to rectify constants in global namespace.

So, from my point of view the driver code is ugly in a sense it
doesn't have its own namespace for constants.

After we fix the driver, the rest would be whatever we get agreed on.

> And also would probably be good to add something like:
>
> #define SG_PAGE_BITS (SG_CHAIN | SG_LAST)
> #define SG_PAGE_MASK (~SG_PAGE_BITS)
>
> ?

Good suggestion,

-- 
With Best Regards,
Andy Shevchenko