Implement copy for volatile?

So I have been following the 'Writing an OS in Rust' guide. And while I was experimenting and reading through the code I noticed that Copy hasn't been implemented for Volatile.

This could be useful when writing code:

impl<T: Copy> Copy for Volatile<T> {
    // Empty.
}

After this the new line function for the VGA buffer can be implemented as follows:

fn new_line(&mut self) {
    &self.buffer.chars.copy_within(1.., 0);
        
    self.clear_row(BUFFER_HEIGHT - 1);
    self.column_position = 0;
}

Now I did notice that copy_within uses the normal ptr::copy instead of volatile_copy_memory. Is this a potential issue?

Anyway let me know if you want to add this, if so I will make a short MR.

@phil-opp

See #6 (comment) and #5 (comment) as to why this hasn't been done yet.

I would like to see VolatileArray be implemented, is that possible now? @phil-opp

@phil-opp

See #6 (comment) and #5 (comment) as to why this hasn't been done yet.

I would like to see VolatileArray be implemented, is that possible now? @phil-opp

Oh it seems like I forgot to look at the MRs to check if this was already asked. Sorry about that. Anyway thanks for your answer.

I'd also be interested to know if something like VolatileArray could be useful.

I think it should be possible to implement VolatileArray and VolatileSlice types now that we have basic support for const generics in Rust, but I'm not 100% sure.

The new v0.5.0 release includes both a VolatilePtr type that implements Copy and (unstable) methods for accessing slices and arrays.

Navigation Menu

Repository navigation

Implement copy for volatile? #9

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Implement copy for volatile? #9

Description

Activity

videah commented on Nov 8, 2019

BuggStream commented on Nov 9, 2019

phil-opp commented on Nov 26, 2019

phil-opp commented on Jun 24, 2023

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions