linux-kernel - Re: [PATCH v2 0/4] Style fixes: open code obfuscating macros

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 for Android: free password hash cracker in your pocket

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

Message-ID: <e5dbac43-6ca0-1adf-251c-b288e9fae814@deltatee.com>
Date:   Wed, 1 Feb 2017 15:11:44 -0700
From:   Logan Gunthorpe <logang@...tatee.com>
To:     Jon Mason <jdmason@...zu.us>
Cc:     Xiangliang Yu <Xiangliang.Yu@....com>,
        Dave Jiang <dave.jiang@...el.com>,
        Allen Hubbe <Allen.Hubbe@....com>,
        Joe Perches <joe@...ches.com>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        linux-ntb@...glegroups.com,
        linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 0/4] Style fixes: open code obfuscating macros

Cool, Thanks.

Let me know if you need anything from me.

Logan

On 01/02/17 01:08 PM, Jon Mason wrote:
> On Thu, Jan 26, 2017 at 3:00 PM, Logan Gunthorpe <logang@...tatee.com> wrote:
>> Hi,
>>
>> It's been a couple weeks... Any thoughts on this?
> 
> My apologies for not responding sooner.  A large rework of the core
> NTB code was done prior to your patches (for IDT NTB support).
> Unfortunately, after those changes this series does not apply cleanly.
> I understand the changes being made, and will fix-up the patches to
> make it apply cleanly on the updated code base.
> 
> Thanks,
> Jon
> 
>>
>> Thanks,
>>
>> Logan
>>
>> On 10/01/17 05:33 PM, Logan Gunthorpe wrote:
>>> Hi,
>>>
>>> Here's an updated version of the style fixes patchset. The differences
>>> from v1 are:
>>>
>>> 1) Rebased onto Jon Mason's current NTB branch
>>>
>>> 2) Added two more patches to convert the print lines to use the ntb
>>> device and not the pci one. This seems more sane, shortens a bunch
>>> of lines and removes the need for temporaries.
>>>
>>> Note with (2): I've tried my best to ensure that statements that print
>>> before the ntb device is registered still use the PCI device. However,
>>> someone should probably review and test this as I don't have access
>>> to all types of hardware to do that.
>>>
>>> Thanks,
>>>
>>> Logan
>>>
>>>
>>>
>>> Logan Gunthorpe (4):
>>>   ntb_hw_amd: Style fixes: open code macros that just obfuscate code
>>>   ntb_hw_intel: Style fixes: open code macros that just obfuscate code
>>>   ntb_hw_amd: Print kernel messages with the ntb device not the pci one
>>>   ntb_hw_intel: Print kernel messages with the ntb device not the pci
>>>     one
>>>
>>>  drivers/ntb/hw/amd/ntb_hw_amd.c     |  60 +++++------
>>>  drivers/ntb/hw/amd/ntb_hw_amd.h     |   3 -
>>>  drivers/ntb/hw/intel/ntb_hw_intel.c | 192 +++++++++++++++++-------------------
>>>  drivers/ntb/hw/intel/ntb_hw_intel.h |   3 -
>>>  4 files changed, 124 insertions(+), 134 deletions(-)
>>>
>>> --
>>> 2.1.4
>>>
>>
>> --
>> You received this message because you are subscribed to the Google Groups "linux-ntb" group.
>> To unsubscribe from this group and stop receiving emails from it, send an email to linux-ntb+unsubscribe@...glegroups.com.
>> To post to this group, send email to linux-ntb@...glegroups.com.
>> To view this discussion on the web visit https://groups.google.com/d/msgid/linux-ntb/5ed3be6c-c94e-1ade-e4cd-a8e8293296cd%40deltatee.com.
>> For more options, visit https://groups.google.com/d/optout.