linux-kernel - Re: [PATCH v3 0/2] add regmap-spi-avmm & Intel Max10 BMC chip support

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:   Tue, 18 Aug 2020 23:07:38 +0800
From:   Xu Yilun <yilun.xu@...el.com>
To:     Tom Rix <trix@...hat.com>
Cc:     Lee Jones <lee.jones@...aro.org>, broonie@...nel.org,
        linux-kernel@...r.kernel.org, matthew.gerlach@...ux.intel.com,
        russell.h.weight@...el.com, lgoncalv@...hat.com, hao.wu@...el.com
Subject: Re: [PATCH v3 0/2] add regmap-spi-avmm & Intel Max10 BMC chip support

I think that's a good optimization. I'll include this change and send a
v4.

Thanks.

On Tue, Aug 18, 2020 at 07:40:45AM -0700, Tom Rix wrote:
> Yilun,
> 
> I was looking at the tx side a bit and think the padding function could be moved into the pkt/phy function.  The pky/phy function already is looking for the eop's so reuse it and remove the search for eop and exchange the loops that do char moving and padding to mem* functions.  The logic is something like.                                                          
> if (tb == tb_end - 1 && !eop_inserted) {
>         *pb = PKT_EOP;
> 
>         p_eop = pb;
> 
> ...
> 
>     dst_pb = &br->phy_buf[aligned_len];
>     size_t s = pb - p_eop + 1;
>     /* move EOP and bytes after EOP to the end of aligned size */
>     memmove(dst_pb - s, p_eop, s);
>     /* fill the hole with PHY_IDLEs */
>     memset(p_eop, PHY_IDLE, aligned_len - br->phy_len);
>     /* update the phy data length */
> 
> Tom
> 
>  
> 
> On 8/18/20 1:36 AM, Xu Yilun wrote:
> > On Mon, Aug 17, 2020 at 10:12:52AM +0100, Lee Jones wrote:
> >> On Mon, 17 Aug 2020, Xu Yilun wrote:
> >>
> >>> Hi Brown & jones:
> >>>
> >>> I tried to refacor the regmap code and add comments in this patchset. I
> >>> made big changes to the rx flow and remove some tricky parts in it.
> >>>
> >>> Would it be more understandable than last version? I'm expecting your
> >>> comments on it when you have time, thanks in advance.
> >> Just resubmit please.  We can review the code itself.
> > Ok. I'll rebase it to 5.9-rc1 and resubmit it.
> >
> > Thanks,
> > Yilun
> >
> >> -- 
> >> Lee Jones [李琼斯]
> >> Senior Technical Lead - Developer Services
> >> Linaro.org │ Open source software for Arm SoCs
> >> Follow Linaro: Facebook | Twitter | Blog