Skip to content
This repository was archived by the owner on Jun 30, 2025. It is now read-only.

Conversation

@justalittlenoob
Copy link
Contributor

No description provided.

@justalittlenoob justalittlenoob requested a review from a team as a code owner September 20, 2022 11:30
@justalittlenoob justalittlenoob temporarily deployed to intel_workflow September 20, 2022 11:30 Inactive
@justalittlenoob justalittlenoob temporarily deployed to intel_workflow September 20, 2022 11:31 Inactive
@justalittlenoob justalittlenoob temporarily deployed to intel_workflow September 20, 2022 11:31 Inactive
@justalittlenoob justalittlenoob temporarily deployed to intel_workflow September 20, 2022 11:38 Inactive
@justalittlenoob justalittlenoob temporarily deployed to intel_workflow September 20, 2022 11:38 Inactive
@justalittlenoob justalittlenoob temporarily deployed to intel_workflow September 20, 2022 11:38 Inactive
@justalittlenoob
Copy link
Contributor Author

Related to #16

@skmono skmono temporarily deployed to intel_workflow September 21, 2022 00:05 Inactive
@skmono skmono temporarily deployed to intel_workflow September 21, 2022 00:05 Inactive
@skmono skmono temporarily deployed to intel_workflow September 21, 2022 00:05 Inactive
@skmono
Copy link
Contributor

skmono commented Sep 21, 2022

I made some slight changes to the public header ipcl/ipcl.hpp. Technically we can only use ipcl/keygen.hpp in as it includes all the dependencies needed. Can you take a look?

@skmono
Copy link
Contributor

skmono commented Sep 21, 2022

@justalittlenoob if including ipcl/keygen.hpp works perhaps we can rename it to ipcl/ipcl.hpp and see if it works?

@justalittlenoob justalittlenoob temporarily deployed to intel_workflow September 21, 2022 05:13 Inactive
@justalittlenoob justalittlenoob temporarily deployed to intel_workflow September 21, 2022 05:14 Inactive
@justalittlenoob justalittlenoob temporarily deployed to intel_workflow September 21, 2022 05:14 Inactive
Copy link
Contributor

@skmono skmono left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes look great!

#include "ipcl/bignum.h"
#include "ipcl/ciphertext.hpp"
#include "ipcl/common.hpp"
#include "ipcl/keygen.hpp"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested this and seems like we can include only #include "ipcl/keygen.hpp" as it contains all the info needed

@skmono skmono merged commit 505b8ac into development Sep 21, 2022
@skmono skmono deleted the zpf/feature-pub-header branch September 21, 2022 16:09
skmono pushed a commit that referenced this pull request Nov 15, 2022
* initial release

* Update CODEOWNERS (#1)

* Minor updates (#4)

* Minor updates
- Updated contributors
- Added code of conduct
- Updated CI/CD

* ModExp function: remove padding & code clean (#6)

- Remove padding operation in ippsModExp function
- Let ippsMBModExp function support modulus of different bit size(in one vector)
- Code clean

* ippsModExp & unittests: corner case fix (#7)

* Improve RNG security (#9)

* Fix insecure prng (#3)
- Add RDSEED and RDRAND instruction check in compile time
- Prioritize RDSEED/RDRAND based RNG to produce random big number

* Add RNG support for non-RDRAND, non-RDSEED systems (#5)
- Use IPP-Crypto pseudo random number generator if none of those instructions are supported

* Removing seed setup and replacing rng function for PrimeGen_BN (#8)
- Remove seed setup for prime number generator
- Add support to TRNGen_RDSEED and PRNGen_RDRAND for prime number generator

Co-authored-by: Pengfei Zhao <pengfei.zhao@intel.com>

* Refactor apply obfuscator (#10)

- Refactor apply_obfuscator
- minor typo fix

* Update ipp-crypto version to use ippcp_2021.6 (#12)

- Minor update to use IPP-Crypto v2021.6

* 13 errors building installing questions about docs (#15)

* Minor fixes
- Fix gbenchmark build error on other platforms
- Fixed IPCLTargets typo
- Update version to 1.1.4

* Update to 1.1.4 (#17) (#20)

* Improve RNG security (#9)
* Fix insecure prng (#3)
- Add RDSEED and RDRAND instruction check in compile time
- Prioritize RDSEED/RDRAND based RNG to produce random big number
* Add RNG support for non-RDRAND, non-RDSEED systems (#5)
- Use IPP-Crypto pseudo random number generator if none of those instructions are supported
* Removing seed setup and replacing rng function for PrimeGen_BN (#8)
- Remove seed setup for prime number generator
- Add support to TRNGen_RDSEED and PRNGen_RDRAND for prime number generator

* Refactor apply obfuscator (#10)
- Refactor apply_obfuscator
- minor typo fix

* Update version for 1.1.4
* Update ipp-crypto version to use ippcp_2021.6 (#12)
- Minor update to use IPP-Crypto v2021.6

* 13 errors building installing questions about docs (#15)
* Minor fixes
- Fix gbenchmark build error on other platforms
- Fixed IPCLTargets typo
- Update version to 1.1.4

Co-authored-by: Pengfei Zhao <pengfei.zhao@intel.com>

* Add ipcl.hpp public header & cleanup ipcl/CMakeLists.txt (#22)

* Cleanup ipcl/CMakeLists.txt
* Refactor header - rename keygen.hpp with ipcl.hpp, use it as public header
* Unit test & benchmark: replace header files with public header - ipcl.hpp

* Improve installation and support runtime IFMA detection (#23)

* Add option to determine AVX512IFMA during runtime (#18)
- Add CMake flag to enable runtime version (```IPCL_DETECT_IFMA_RUNTIME```)
- Add ```cpu_features``` dependency
- Add manual IFMA disabling with environment variable (```IPCL_DISABLE_AVX512IFMA=ON```)
- Updated README to include AVX512IFMA runtime detection option

* Fixed shared library build
- Refactor cmake configurations
- Added example code

* Added build and usage documentation
- Fixed minor typos and updated flag names to be more consistent
- Cleaned up example CMake file

* Added examples documentation
- Build and install instruction
- Linking and compiling instruction
- Usage examples

* Fixed typo in root README (#24)

Co-authored-by: Pengfei Zhao <pengfei.zhao@intel.com>
skmono pushed a commit that referenced this pull request Nov 16, 2022
…nding-buffers

Fdiasmor/distributed outstanding buffers
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants