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] [thread-next>] [day] [month] [year] [list]
Message-ID: <1f337af9f52ac4c3128e61c971cf0e32068a71ab.camel@linux.ibm.com>
Date:   Wed, 04 Dec 2019 15:26:50 -0300
From:   Leonardo Bras <leonardo@...ux.ibm.com>
To:     Ram Pai <linuxram@...ibm.com>, linuxppc-dev@...ts.ozlabs.org,
        mpe@...erman.id.au
Cc:     andmike@...ibm.com, mst@...hat.com, aik@...abs.ru,
        mdroth@...ux.vnet.ibm.com, linux-kernel@...r.kernel.org,
        ram.n.pai@...il.com, cai@....pw, tglx@...utronix.de,
        sukadev@...ux.vnet.ibm.com, hch@....de, bauerman@...ux.ibm.com,
        david@...son.dropbear.id.au
Subject: Re: [PATCH v4 1/2] powerpc/pseries/iommu: Share the per-cpu TCE
 page with the hypervisor.

On Sun, 2019-12-01 at 22:45 -0800, Ram Pai wrote:
> @@ -206,8 +224,7 @@ static int tce_buildmulti_pSeriesLP(struct iommu_table *tbl, long tcenum,
>          * from iommu_alloc{,_sg}()
>          */
>         if (!tcep) {
> -               tcep = (__be64 *)__get_free_page(GFP_ATOMIC);
> -               /* If allocation fails, fall back to the loop implementation */
> +               tcep = alloc_tce_page();
>                 if (!tcep) {
>                         local_irq_restore(flags);
>                         return tce_build_pSeriesLP(tbl, tcenum, npages, uaddr,

The comment about failing allocation was removed, but I see no chage of
behaviour here. 

Can you please explain what/where it changes? 

Best regards,

Leonardo

Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ