Re: [PATCH 0/6] mka cleanups

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Jouni,

2016-08-28, 22:40:35 +0300, Jouni Malinen wrote:
> On Mon, Aug 15, 2016 at 11:40:37AM +0200, Sabrina Dubroca wrote:
> > This patchset contains a variety of cleanups for the MKA code (src/pae).
> 
> > Sabrina Dubroca (6):
> >   mka: some cleanups
> >   mka: cleanups, part 2
> >   mka: cleanups, part 3
> >   mka: cleanups, part 4
> >   mka: move structs {transmit,receive}_{sa,sc} to a common header
> >   mka: pass full structures down the stack in ops callbacks
> 
> Thanks, I applied patches 1..4, but left patches 5 and 6 waiting for
> updates as noted in the earlier messages.

Thanks, I'll update and resend these two.


> Please note that I split the
> patches in significantly smaller independent changes (total of 29
> patches instead of just 4) to make it easier to review them. I did some
> cleanup and fixes, e.g., sci_equal() was using sizeof(a) where a is a
> pointer while this was supposed to use the size of the data structure;
> it happened to be 8 so no difference on 64-bit builds, but for 32-bit
> builds, this could have resulted in incorrect behavior. Please check
> that I did not end up breaking any functionality with the changes.

I reviewed the diff between my local branch and your tree, and it
looks okay.  I'll run some tests before I submit more patches and send
fixes if necessary.


> For any followup cleanup, please consider making smaller patches. That
> would make it significantly easier to review the changes by finding the
> obvious changes without having to figure out which parts of the patch
> apply to which item in the commit message (and which parts are missing
> altogether from the commit message).

Okay, sorry about that.  I have a few other rather big patches coming
up, I'll try to split them up.


> I left out one change since I did not find any justification in the
> commit message for it and did not see it as an obvious improvement
> myself. Please resubmit this with more descriptive commit message if you
> still want this included:
> 
> 
> [PATCH] mka: Move kay_ops struct out of ieee802_1x_alloc_kay_sm()

I prefer this style, but it doesn't make a huge difference.  Since you
prefer the current version, I'll just drop this change.


Thanks.

-- 
Sabrina

_______________________________________________
Hostap mailing list
Hostap@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/hostap



[Index of Archives]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux