-
Notifications
You must be signed in to change notification settings - Fork 229
docs: add page to describe cache access model #2949
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Co-authored-by: Martin Stefanko <xstefank122@gmail.com>
Co-authored-by: Martin Stefanko <xstefank122@gmail.com>
Co-authored-by: Martin Stefanko <xstefank122@gmail.com>
Co-authored-by: Martin Stefanko <xstefank122@gmail.com>
Co-authored-by: Martin Stefanko <xstefank122@gmail.com>
Co-authored-by: Martin Stefanko <xstefank122@gmail.com>
Co-authored-by: Martin Stefanko <xstefank122@gmail.com>
Co-authored-by: Martin Stefanko <xstefank122@gmail.com>
Co-authored-by: Martin Stefanko <xstefank122@gmail.com>
Co-authored-by: Martin Stefanko <xstefank122@gmail.com>
@xstefank thank you very much, committed your suggestions |
Signed-off-by: Chris Laprun <claprun@redhat.com>
Your last commit seems to have overwritten changes I made earlier. |
Sorry that was not intentional. |
But will run some AI to rephrase it, so see it helps it. |
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Improved the workding and made some other fix. It is ready now. please review. Thank you! |
This would have taken way too long… and it already took me a long time to review so… |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please re-incorporate the changes I made previously. Several of these changes were meant to clarify or correct things that I thought were not accurate enough or misleading. Using an AI tool is not going to address these issues (I see this already in the first sentence) and I'm not going to review this again when I've already done it once. I'll review the new final section again once this is done.
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
I added most of you changes from that commit, they were all good, except one which I commented on. thank you! |
Signed-off-by: Chris Laprun <claprun@redhat.com>
Signed-off-by: Chris Laprun <claprun@redhat.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Much better explanations on a complex subject!
Signed-off-by: Attila Mészáros a_meszaros@apple.com