Messages in this thread | | | Date | Thu, 12 Nov 2015 19:57:14 +0100 | From | Matias Bjørling <> | Subject | Re: [PATCH] lightnvm: change max_phys_sect to ushort |
| |
On 11/12/2015 07:42 PM, Geert Uytterhoeven wrote: > On Thu, Nov 12, 2015 at 7:35 PM, Matias Bjørling <m@bjorling.me> wrote: >> The max_phys_sect variable is defined as a char. We do a boundary check >> to maximally allow 256 physical page descriptors per command. As we are >> not indexing from zero. This expression is always in false. Bump the >> max_phys_sect to an unsigned short to support the range check. > > unsigned int?
Grah, I need to be more careful. I sent the wrong patch after I had fixed it to unsigned short.
| |