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
| ||
|
Date: Thu, 12 Oct 2006 17:00:24 -0700 From: "Chen, Kenneth W" <kenneth.w.chen@...el.com> To: "'Zach Brown'" <zach.brown@...cle.com>, "'Suparna Bhattacharya'" <suparna@...ibm.com>, "Lahaise, Benjamin C" <benjamin.c.lahaise@...el.com> Cc: <linux-kernel@...r.kernel.org>, "'linux-aio'" <linux-aio@...ck.org> Subject: [patch] clarify AIO_EVENTS_OFFSET constant A clean up patch: I think it is a lot easier to read AIO_EVENTS_OFFSET as an offset because of aio_ring at the beginning of a head page, instead of doing arithmetic of (event on 2nd page - event on 1st page). Signed-off-by: Ken Chen <kenneth.w.chen@...el.com> diff -Nurp linux-2.6.18/fs/aio.c linux-2.6.18.ken/fs/aio.c --- linux-2.6.18/fs/aio.c 2006-09-19 20:42:06.000000000 -0700 +++ linux-2.6.18.ken/fs/aio.c 2006-10-12 13:33:09.000000000 -0700 @@ -173,9 +173,8 @@ static int aio_setup_ring(struct kioctx /* aio_ring_event: returns a pointer to the event at the given index from * kmap_atomic(, km). Release the pointer with put_aio_ring_event(); */ -#define AIO_EVENTS_PER_PAGE (PAGE_SIZE / sizeof(struct io_event)) -#define AIO_EVENTS_FIRST_PAGE ((PAGE_SIZE - sizeof(struct aio_ring)) / sizeof(struct io_event)) -#define AIO_EVENTS_OFFSET (AIO_EVENTS_PER_PAGE - AIO_EVENTS_FIRST_PAGE) +#define AIO_EVENTS_PER_PAGE (PAGE_SIZE / sizeof(struct io_event)) +#define AIO_EVENTS_OFFSET (sizeof(struct aio_ring) / sizeof(struct io_event)) #define aio_ring_event(info, nr, km) ({ \ unsigned pos = (nr) + AIO_EVENTS_OFFSET; \ - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists