xfnw/plan9fox
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

devsd: always page align sd buffers

Browse source 
sdbio() tests if it can pass the buffer pointer directly to
the driver when it is already in kernel memory. we also need
to check if the buffer is properly aligned but alignment
requirement is handled in system specific sdmalloc() and
was not known to devsd.

to solve this, we *always* page align sd buffers and get rid
of the system specific sdmalloc() macro (was only used in bcm
kernel).
This commit is contained in:
cinap_lenrek 2015-03-06 16:16:45 +01:00
parent fadbb92afa
commit 374d4ec2c1
3 changed files with 3 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
sys/src/9/bcm/fns.h
View file

@ -58,8 +58,6 @@ extern void procsave(Proc*);
extern void procfork(Proc*);
extern void procsetup(Proc*);
extern void screeninit(void);
#define sdfree(p) free(p)
#define sdmalloc(n) mallocalign(n, CACHELINESZ, 0, 0)
extern void setpower(int, int);
extern void setr13(int, u32int*);
extern int splfhi(void);

2
sys/src/9/port/devsd.c
View file

@ -843,7 +843,7 @@ sdbio(Chan* c, int write, char* a, long len, uvlong off)
len = nb*unit->secsize - offset;
hard = offset || write && len%unit->secsize;
if(iskaddr(a) && !hard) {
if(iskaddr(a) && ((uintptr)a & (BY2PG-1))==0 && !hard) {
b = (uchar*)a;
allocd = 0;
}else{

9
sys/src/9/port/sd.h
View file

@ -145,15 +145,10 @@ enum {
};
/*
* Allow the default #defines for sdmalloc & sdfree to be overridden by
* system-specific versions. This can be used to avoid extra copying
* by making sure sd buffers are cache-aligned (some ARM systems) or
* page-aligned (xen) for DMA.
* Avoid extra copying by making sd buffers page-aligned for DMA.
*/
#ifndef sdmalloc
#define sdmalloc(n) malloc(n)
#define sdmalloc(n) mallocalign(n, BY2PG, 0, 0)
#define sdfree(p) free(p)
#endif
/*
* mmc/sd/sdio host controller interface