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: Sun, 3 Aug 2014 13:30:44 -0400 From: Nick Krause <xerofoify@...il.com> To: Sergei Shtylyov <sergei.shtylyov@...entembedded.com> Cc: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, USB list <linux-usb@...r.kernel.org> Subject: Re: [PATCH] scatterlist.h: Change CONFIG_DEBUG_SG for ifdef statement in sg_set_bf On Sun, Aug 3, 2014 at 8:28 AM, Sergei Shtylyov <sergei.shtylyov@...entembedded.com> wrote: > Hello. > > > On 03-08-2014 6:56, Nicholas Krause wrote: > >> This changes the ifdef statement in sg_set_bg to !CONFIG_DEBUG_SG in >> order >> to avoid a bug with xhci dequence/enquence functions. > > > dequeue/enqueue? > > >> Signed-off-by: Nicholas Krause <xerofoify@...il.com> >> --- >> include/linux/scatterlist.h | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) > > >> diff --git a/include/linux/scatterlist.h b/include/linux/scatterlist.h >> index adae88f..62de7b3 100644 >> --- a/include/linux/scatterlist.h >> +++ b/include/linux/scatterlist.h >> @@ -111,7 +111,7 @@ static inline struct page *sg_page(struct scatterlist >> *sg) >> static inline void sg_set_buf(struct scatterlist *sg, const void *buf, >> unsigned int buflen) >> { >> -#ifdef CONFIG_DEBUG_SG >> +#ifdef !CONFIG_DEBUG_SG > > > Didn't you mean #ifndef instead? I guess you didn't even try to > build-test this. > > >> BUG_ON(!virt_addr_valid(buf)); >> #endif >> sg_set_page(sg, virt_to_page(buf), buflen, offset_in_page(buf)); > > > WBR, Sergei > I am going to stay around and learn more but am going to check my patches better as this is my fault. Regards Nick -- 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